Keep visibility state in sync between the native window and view

chrome/browser/ui/views/constrained_window_win.cc

Index: chrome/browser/ui/views/constrained_window_win.cc
diff --git a/chrome/browser/ui/views/constrained_window_win.cc b/chrome/browser/ui/views/constrained_window_win.cc
index ac1d0d2c779497987bb9e48d8dcfbf07902058c4..ce65a623bee4680b248b70838cce5cada2169bb0 100644
--- a/chrome/browser/ui/views/constrained_window_win.cc
+++ b/chrome/browser/ui/views/constrained_window_win.cc
@@ -571,6 +571,7 @@ void ConstrainedWindowWin::FocusConstrainedWindow() {
 }
 
 void ConstrainedWindowWin::ShowConstrainedWindow() {
+  GetNonClientView()->SetVisible(true);

[sky, 2011/01/19 21:36:51]

Add a comment as to why this is necessary.

[asanka (google), 2011/01/20 14:49:18]

Done.

   if (owner_->delegate())
     owner_->delegate()->WillShowConstrainedWindow(owner_);
   ActivateConstrainedWindow();
@@ -614,6 +615,7 @@ ConstrainedWindowWin::ConstrainedWindowWin(
   set_window_style(WS_CHILD | WS_CLIPSIBLINGS | WS_CLIPCHILDREN | WS_CAPTION |
                    WS_THICKFRAME | WS_SYSMENU);
   set_focus_on_creation(false);
+  GetNonClientView()->SetVisible(false);

[sky, 2011/01/19 21:36:51]

Add a comment as to why this is necessary.

[asanka (google), 2011/01/20 14:49:18]

Done.

 
   WindowWin::Init(owner_->GetNativeView(), gfx::Rect());
 }