exo: Improve Maximize and Fullscreen support for ShellSurfaces.

components/exo/shell_surface.cc

 // Copyright 2015 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 "components/exo/shell_surface.h"
 
 #include "ash/shell.h"
 #include "ash/shell_window_ids.h"
 #include "ash/wm/window_state.h"
 #include "base/logging.h"
@@ skipping 36 matching lines @@
   views::Widget* const widget_;
 
   DISALLOW_COPY_AND_ASSIGN(CustomFrameView);
 };
 
 class ShellSurfaceWidget : public views::Widget {
  public:
   explicit ShellSurfaceWidget(ShellSurface* shell_surface)
       : shell_surface_(shell_surface) {}
 
-  static views::Widget::InitParams CreateInitParams(
-      views::WidgetDelegate* delegate) {
-    views::Widget::InitParams params;
-    params.type = views::Widget::InitParams::TYPE_WINDOW;
-    params.ownership = views::Widget::InitParams::WIDGET_OWNS_NATIVE_WIDGET;
-    params.delegate = delegate;
-    params.shadow_type = views::Widget::InitParams::SHADOW_TYPE_NONE;
-    params.opacity = views::Widget::InitParams::TRANSLUCENT_WINDOW;
-    params.show_state = ui::SHOW_STATE_NORMAL;
-    params.parent =
-        ash::Shell::GetContainer(ash::Shell::GetPrimaryRootWindow(),
-                                 ash::kShellWindowId_DefaultContainer);
-    return params;
-  }
-
   // Overridden from views::Widget
   void Close() override { shell_surface_->Close(); }
 
  private:
   ShellSurface* const shell_surface_;
 
   DISALLOW_COPY_AND_ASSIGN(ShellSurfaceWidget);
 };
 
 }  // namespace
@@ skipping 12 matching lines @@
 
 ShellSurface::~ShellSurface() {
   if (surface_) {
     surface_->SetSurfaceDelegate(nullptr);
     surface_->RemoveSurfaceObserver(this);
   }
   if (widget_)
     widget_->CloseNow();
 }
 
-void ShellSurface::SetToplevel() {
-  TRACE_EVENT0("exo", "ShellSurface::SetToplevel");
+void ShellSurface::Init() {
+  TRACE_EVENT0("exo", "ShellSurface::Init");
 
   if (widget_) {
-    DLOG(WARNING) << "Shell surface already mapped";
+    DLOG(WARNING) << "Shell surface already initialized";
     return;
   }
 
-  views::Widget::InitParams params = ShellSurfaceWidget::CreateInitParams(this);
-  params.bounds = gfx::Rect(gfx::Size(1, 1));
+  views::Widget::InitParams params;
+  params.type = views::Widget::InitParams::TYPE_WINDOW;
+  params.ownership = views::Widget::InitParams::WIDGET_OWNS_NATIVE_WIDGET;
+  params.delegate = this;
+  params.shadow_type = views::Widget::InitParams::SHADOW_TYPE_NONE;
+  params.opacity = views::Widget::InitParams::TRANSLUCENT_WINDOW;
+  params.show_state = ui::SHOW_STATE_NORMAL;
+  params.parent = ash::Shell::GetContainer(
+      ash::Shell::GetPrimaryRootWindow(), ash::kShellWindowId_DefaultContainer);
   widget_.reset(new ShellSurfaceWidget(this));
   widget_->Init(params);
   widget_->GetNativeWindow()->set_owned_by_parent(false);
   widget_->GetNativeWindow()->SetName("ExoShellSurface");
   widget_->GetNativeWindow()->AddChild(surface_);
   SetApplicationId(widget_->GetNativeWindow(), &application_id_);
 
-  // The position of a standard top level shell surface is managed by Ash.
+  // The position of a top-level shell surface is managed by Ash.
   ash::wm::GetWindowState(widget_->GetNativeWindow())
       ->set_window_position_managed(true);
 }
 
-void ShellSurface::SetMaximized() {
-  TRACE_EVENT0("exo", "ShellSurface::SetMaximized");
+void ShellSurface::Maximize() {
+  TRACE_EVENT0("exo", "ShellSurface::Maximize");
 
-  if (widget_) {
-    DLOG(WARNING) << "Shell surface already mapped";
-    return;
-  }
+  DCHECK(widget_);
+  widget_->Maximize();
 
-  views::Widget::InitParams params = ShellSurfaceWidget::CreateInitParams(this);
-  params.show_state = ui::SHOW_STATE_MAXIMIZED;
-  widget_.reset(new ShellSurfaceWidget(this));
-  widget_->Init(params);
-  widget_->GetNativeWindow()->set_owned_by_parent(false);
-  widget_->GetNativeWindow()->SetName("ExoShellSurface");
-  widget_->GetNativeWindow()->AddChild(surface_);
-  SetApplicationId(widget_->GetNativeWindow(), &application_id_);
+  if (!configure_callback_.is_null())
+    configure_callback_.Run(widget_->GetWindowBoundsInScreen().size());
 }
 
-void ShellSurface::SetFullscreen() {
-  TRACE_EVENT0("exo", "ShellSurface::SetFullscreen");
+void ShellSurface::SetFullscreen(bool fullscreen) {
+  TRACE_EVENT1("exo", "ShellSurface::SetFullscreen", "fullscreen", fullscreen);
 
-  if (widget_) {
-    DLOG(WARNING) << "Shell surface already mapped";
-    return;
-  }
+  DCHECK(widget_);
+  widget_->SetFullscreen(fullscreen);
 
-  views::Widget::InitParams params = ShellSurfaceWidget::CreateInitParams(this);
-  params.show_state = ui::SHOW_STATE_FULLSCREEN;
-  widget_.reset(new ShellSurfaceWidget(this));
-  widget_->Init(params);
-  widget_->GetNativeWindow()->set_owned_by_parent(false);
-  widget_->GetNativeWindow()->SetName("ExoShellSurface");
-  widget_->GetNativeWindow()->AddChild(surface_);
-  SetApplicationId(widget_->GetNativeWindow(), &application_id_);
+  if (!configure_callback_.is_null())
+    configure_callback_.Run(widget_->GetWindowBoundsInScreen().size());
 }
 
 void ShellSurface::SetTitle(const base::string16& title) {
   TRACE_EVENT1("exo", "ShellSurface::SetTitle", "title",
                base::UTF16ToUTF8(title));
 
   title_ = title;
   if (widget_)
     widget_->UpdateWindowTitle();
 }
@@ skipping 118 matching lines @@
 // views::Views overrides:
 
 gfx::Size ShellSurface::GetPreferredSize() const {
   if (!geometry_.IsEmpty())
     return geometry_.size();
 
   return surface_ ? surface_->GetPreferredSize() : gfx::Size();
 }
 
 }  // namespace exo