Refactor modality-specific behavior from ConstrainedWindowViews to WebContentsModalDialogManager

chrome/browser/ui/views/constrained_window_views.cc

 // Copyright (c) 2012 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_views.h"
 
 #include <algorithm>
 
 #include "base/command_line.h"
 #include "base/utf_string_conversions.h"
@@ skipping 151 matching lines @@
 
   DISALLOW_COPY_AND_ASSIGN(VistaWindowResources);
 };
 
 gfx::ImageSkia* XPWindowResources::images_[];
 gfx::ImageSkia* VistaWindowResources::images_[];
 
 class ConstrainedWindowFrameView : public views::NonClientFrameView,
                                    public views::ButtonListener {
  public:
-  explicit ConstrainedWindowFrameView(ConstrainedWindowViews* container);
+  ConstrainedWindowFrameView(ConstrainedWindowViews* container,
+                             bool browser_is_off_the_record);
   virtual ~ConstrainedWindowFrameView();
 
   void UpdateWindowTitle();
 
   // Overridden from views::NonClientFrameView:
   virtual gfx::Rect GetBoundsForClientView() const OVERRIDE;
   virtual gfx::Rect GetWindowBoundsForClientBounds(
       const gfx::Rect& client_bounds) const OVERRIDE;
   virtual int NonClientHitTest(const gfx::Point& point) OVERRIDE;
   virtual void GetWindowMask(const gfx::Size& size,
@@ skipping 35 matching lines @@
   void PaintClientEdge(gfx::Canvas* canvas);
 
   // Layout various sub-components of this view.
   void LayoutWindowControls();
   void LayoutTitleBar();
 
   // Returns the bounds of the client area for the specified view size.
   gfx::Rect CalculateClientAreaBounds(int width, int height) const;
 
   SkColor GetTitleColor() const {
-    return container_->owner()->GetBrowserContext()->IsOffTheRecord()
+    return browser_is_off_the_record_
 #if defined(OS_WIN) && !defined(USE_AURA)
             || !ui::win::IsAeroGlassEnabled()
 #endif
             ? SK_ColorWHITE : SK_ColorBLACK;
   }
 
   // Loads the appropriate set of WindowResources for the frame view.
   void InitWindowResources();
 
   ConstrainedWindowViews* container_;
 
+  bool browser_is_off_the_record_;
+
   scoped_ptr<views::WindowResources> resources_;
 
   gfx::Rect title_bounds_;
 
   views::ImageButton* close_button_;
 
   // The bounds of the ClientView.
   gfx::Rect client_view_bounds_;
 
   // Background painter for the frame.
@@ skipping 29 matching lines @@
 // The title text starts 2 px from the right edge of the left frame border.
 const int kTitleLeftSpacing = 2;
 // There is a 5 px gap between the title text and the caption buttons.
 const int kTitleCaptionSpacing = 5;
 
 const SkColor kContentsBorderShadow = SkColorSetARGB(51, 0, 0, 0);
 
 }  // namespace
 
 ConstrainedWindowFrameView::ConstrainedWindowFrameView(
-    ConstrainedWindowViews* container)
+    ConstrainedWindowViews* container, bool browser_is_off_the_record)
     : NonClientFrameView(),
       container_(container),
+      browser_is_off_the_record_(browser_is_off_the_record),
       close_button_(new views::ImageButton(this)),
       frame_background_(new views::FrameBackground()) {
   InitClass();
   InitWindowResources();
 
   // Constrained windows always use the custom frame - they just have a
   // different set of images.
   container->set_frame_type(views::Widget::FRAME_TYPE_FORCE_CUSTOM);
 
   ui::ResourceBundle& rb = ui::ResourceBundle::GetSharedInstance();
@@ skipping 251 matching lines @@
       container_->CloseWebContentsModalDialog();
   }
 
  private:
   ConstrainedWindowViews* container_;  // not owned
   DISALLOW_COPY_AND_ASSIGN(ConstrainedWindowFrameViewAsh);
 };
 #endif  // defined(USE_ASH)
 
 ConstrainedWindowViews::ConstrainedWindowViews(
-    content::WebContents* web_contents,
+    gfx::NativeView parent,
+    bool off_the_record,
     views::WidgetDelegate* widget_delegate)
-    : web_contents_(web_contents),
-      ALLOW_THIS_IN_INITIALIZER_LIST(native_constrained_window_(
-          NativeConstrainedWindow::CreateNativeConstrainedWindow(this))) {
+    : off_the_record_(off_the_record) {
   views::Widget::InitParams params(views::Widget::InitParams::TYPE_WINDOW);
   params.delegate = widget_delegate;
-  params.native_widget = native_constrained_window_->AsNativeWidget();
   params.child = true;
 
-  params.parent = web_contents_->GetNativeView();
+  params.parent = parent;
 
 #if defined(USE_ASH)
   // Ash window headers can be transparent.
   params.transparent = true;
-  views::corewm::SetChildWindowVisibilityChangesAnimated(params.parent);
-  // No animations should get performed on the window since that will re-order
-  // the window stack which will then cause many problems.
-  if (params.parent && params.parent->parent()) {
-    params.parent->parent()->SetProperty(aura::client::kAnimationsDisabledKey,
-                                         true);
-  }
 #endif
+
   Init(params);
-
-  WebContentsModalDialogManager* web_contents_modal_dialog_manager =
-      WebContentsModalDialogManager::FromWebContents(web_contents_);
-  web_contents_modal_dialog_manager->AddDialog(this);
-#if defined(USE_ASH)
-  GetNativeWindow()->SetProperty(ash::kConstrainedWindowKey, true);
-  views::corewm::SetModalParent(GetNativeWindow(),
-                                web_contents_->GetView()->GetNativeView());
-#endif
 }
 
 ConstrainedWindowViews::~ConstrainedWindowViews() {
 }
 
 void ConstrainedWindowViews::ShowWebContentsModalDialog() {
   Show();
   FocusWebContentsModalDialog();
 }
 
 void ConstrainedWindowViews::CloseWebContentsModalDialog() {
-#if defined(USE_ASH)
-  gfx::NativeView view = web_contents_->GetNativeView();
-  // Allow the parent to animate again.
-  if (view && view->parent())
-    view->parent()->ClearProperty(aura::client::kAnimationsDisabledKey);
-#endif
-  WebContentsModalDialogManager* web_contents_modal_dialog_manager =
-      WebContentsModalDialogManager::FromWebContents(web_contents_);
-  web_contents_modal_dialog_manager->WillClose(this);
   Close();
 }
 
 void ConstrainedWindowViews::FocusWebContentsModalDialog() {
   if (widget_delegate() && widget_delegate()->GetInitiallyFocusedView())
     widget_delegate()->GetInitiallyFocusedView()->RequestFocus();
 #if defined(USE_ASH)
   // We don't necessarily have a RootWindow yet.
   if (GetNativeView()->GetRootWindow())
     GetNativeView()->Focus();
 #endif
 }
 
 void ConstrainedWindowViews::PulseWebContentsModalDialog() {
 }
 
 gfx::NativeWindow ConstrainedWindowViews::GetNativeWindow() {
   return Widget::GetNativeWindow();
 }
 
+ConstrainedWindowViews* ConstrainedWindowViews::Create(
+    content::WebContents* web_contents,
+    views::WidgetDelegate* widget_delegate) {
+  WebContentsModalDialogManager* manager =
+      WebContentsModalDialogManager::FromWebContents(web_contents);
+  ConstrainedWindowViews* dialog = new ConstrainedWindowViews(
+      web_contents->GetNativeView(),
+      web_contents->GetBrowserContext()->IsOffTheRecord(),
+      widget_delegate);
+  manager->AddDialog(dialog);
+  return dialog;
+}
+
 views::NonClientFrameView* ConstrainedWindowViews::CreateNonClientFrameView() {
   if (views::DialogDelegate::UseNewStyle())
     return new views::DialogFrameView(widget_delegate()->GetWindowTitle());
 #if defined(USE_ASH)
   ConstrainedWindowFrameViewAsh* frame = new ConstrainedWindowFrameViewAsh;
   frame->Init(this);
   return frame;
 #endif
-  return new ConstrainedWindowFrameView(this);
+  return new ConstrainedWindowFrameView(this, off_the_record_);
 }
-
-void ConstrainedWindowViews::OnNativeConstrainedWindowDestroyed() {
-  WebContentsModalDialogManager* web_contents_modal_dialog_manager =
-      WebContentsModalDialogManager::FromWebContents(web_contents_);
-  web_contents_modal_dialog_manager->WillClose(this);
-}
-
-void ConstrainedWindowViews::OnNativeConstrainedWindowMouseActivate() {
-  Activate();
-}
-
-views::internal::NativeWidgetDelegate*
-    ConstrainedWindowViews::AsNativeWidgetDelegate() {
-  return this;
-}
-
-int ConstrainedWindowViews::GetNonClientComponent(const gfx::Point& point) {
-  // Prevent a constrained window to be moved by the user.
-  return HTNOWHERE;
-}