Fix fullscreen mode on aura

views/widget/native_widget_aura.cc

Index: views/widget/native_widget_aura.cc
diff --git a/views/widget/native_widget_aura.cc b/views/widget/native_widget_aura.cc
index ee23f426f26c5b6a1d6c38078c1d2759ddc09c7a..c96b644d528df6b7fae5051dca6288224e548f9e 100644
--- a/views/widget/native_widget_aura.cc
+++ b/views/widget/native_widget_aura.cc
@@ -374,7 +374,9 @@ void NativeWidgetAura::Show() {
 }
 
 void NativeWidgetAura::Hide() {
-  window_->Hide();
+  // |window_| may already be deleted by parent window.
+  if (window_)

[Ben Goodger (Google), 2011/11/16 15:48:25]

This seems kind of random. What other functions in NWA might be called after
window_ is randomly deleted?

[oshima, 2011/11/16 16:51:47]

On second thought, I think I should move this to Close, like
if (window_)
 Hide();

This happens when a window that has either child or transient child (similar to
tab contents detach fix i did a copule of weeks ago) is closing. Deleting parent
window deletes child & transient child before deleting child views, and deleting
view may try to close the NWA whose window has already been deleted.
 Since this should happen only when we're closing the window, Close is probably 
better place. This should also be possible when ownership is
WIDGET_OWNS_NATIVE_WIDGET, so I can add DCHECK. Please let me know what you
think.

BTW. my understanding of why this problem doesn't happen on gtk is because gtk
widget is refcounted. Not sure how this works on windows though.

+    window_->Hide();
 }
 
 void NativeWidgetAura::ShowMaximizedWithBounds(