Keep visibility state in sync between the native window and view

chrome/browser/ui/views/constrained_window_win.cc

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/ui/views/constrained_window_win.h"
 
 #include <algorithm>
 
 #include "app/resource_bundle.h"
 #include "app/win/hwnd_util.h"
@@ skipping 553 matching lines @@
 
 void ConstrainedWindowWin::FocusConstrainedWindow() {
   if ((!owner_->delegate() ||
        owner_->delegate()->ShouldFocusConstrainedWindow()) &&
       GetDelegate() && GetDelegate()->GetInitiallyFocusedView()) {
     GetDelegate()->GetInitiallyFocusedView()->RequestFocus();
   }
 }
 
 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();
   FocusConstrainedWindow();
 }
 
 void ConstrainedWindowWin::CloseConstrainedWindow() {
   // Broadcast to all observers of NOTIFY_CWINDOW_CLOSED.
   // One example of such an observer is AutomationCWindowTracker in the
   // automation component.
@@ skipping 23 matching lines @@
 ConstrainedWindowWin::ConstrainedWindowWin(
     TabContents* owner,
     views::WindowDelegate* window_delegate)
     : WindowWin(window_delegate),
       owner_(owner) {
   GetNonClientView()->SetFrameView(CreateFrameViewForWindow());
 
   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());
 }
 
 void ConstrainedWindowWin::ActivateConstrainedWindow() {
   // Other pop-ups are simply moved to the front of the z-order.
   SetWindowPos(HWND_TOP, 0, 0, 0, 0, SWP_NOMOVE | SWP_NOSIZE | SWP_SHOWWINDOW);
 }
 
 ////////////////////////////////////////////////////////////////////////////////
@@ skipping 37 matching lines @@
 
 
 // static
 ConstrainedWindow* ConstrainedWindow::CreateConstrainedDialog(
     TabContents* parent,
     views::WindowDelegate* window_delegate) {
   ConstrainedWindowWin* window = new ConstrainedWindowWin(parent,
                                                           window_delegate);
   return window;
 }