The method always returns true or false, and this is always equal to the DialogResult property of the window at the time it closes.
But the DialogResult property is null before the window is closed, and another thread could check the property. So it kind of makes sense that the return value is a nullable boolean to match the property, even though it is never actually null.