Adding new window management

chrome/browser/ui/window_sizer/window_sizer_ash.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/window_sizer/window_sizer.h"
 
 #include "ash/shell.h"
 #include "ash/wm/window_cycle_controller.h"
 #include "ash/wm/window_util.h"
 #include "base/compiler_specific.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/ui/browser.h"
 #include "chrome/browser/ui/browser_list.h"
 #include "chrome/browser/ui/browser_window.h"
 #include "ui/aura/window.h"
 #include "ui/aura/window_delegate.h"
 #include "ui/gfx/screen.h"
 
 namespace {
 
-// Check if the window was not created as popup or as panel.
-bool IsValidToplevelWindow(aura::Window* window) {
+// Check if the window was not created as popup or as panel, it is
+// on the screen defined by |bounds| and visible.
+bool IsValidToplevelWindow(aura::Window* window, const gfx::Rect& bounds) {
   for (BrowserList::const_iterator iter = BrowserList::begin();
        iter != BrowserList::end();
        ++iter) {
     Browser* browser = *iter;
     if (browser && browser->window() &&
         browser->window()->GetNativeWindow() == window) {
-      return (!(browser->is_type_popup() || browser->is_type_panel()));
+      return (!(browser->is_type_popup() || browser->is_type_panel()) &&
+              !browser->window()->IsMinimized() &&
+              bounds.Intersects(browser->window()->GetBounds()));
     }
   }
   // A window which has no browser associated with it is probably not a window
   // of which we want to copy the size from.
   return false;
 }
 
-// Get the first open window in the stack on the screen.
-aura::Window* GetTopWindow() {
+// Get the first open (non minimized) window which is on the screen defined
+// by |bounds| and visible.
+aura::Window* GetTopWindow(const gfx::Rect& bounds) {

[sky, 2012/09/13 15:44:51]

The coordinates you pass in here are in screen coordinates, but you use
window->bounds() which are not necessarily screen coordinates. Same thing for
IsValidToplevelWindow above.

[Mr4D (OOO till 08-26), 2012/09/13 16:45:08]

Right! Done.

   // Get the active window.
   aura::Window* window = ash::wm::GetActiveWindow();
   if (window && window->type() == aura::client::WINDOW_TYPE_NORMAL &&
-      window->IsVisible() && IsValidToplevelWindow(window))
+      window->IsVisible() && IsValidToplevelWindow(window, bounds))
     return window;
 
   // Get a list of all windows.
   const std::vector<aura::Window*> windows =
       ash::WindowCycleController::BuildWindowList(NULL);
 
   if (windows.empty())
     return NULL;
 
   aura::Window::Windows::const_iterator iter = windows.begin();
   // Find the index of the current window.
   if (window)
     iter = std::find(windows.begin(), windows.end(), window);
 
   int index = (iter == windows.end()) ? 0 : (iter - windows.begin());
 
   // Scan the cycle list backwards to see which is the second topmost window
   // (and so on). Note that we might cycle a few indices twice if there is no
   // suitable window. However - since the list is fairly small this should be
   // very fast anyways.
   for (int i = index + windows.size(); i >= 0; i--) {
     aura::Window* window = windows[i % windows.size()];
     if (window && window->type() == aura::client::WINDOW_TYPE_NORMAL &&
-        window->IsVisible() && IsValidToplevelWindow(window))
+        bounds.Intersects(window->bounds()) &&
+        window->IsVisible() && IsValidToplevelWindow(window, bounds))
       return window;
   }
   return NULL;
 }
 
+// Return the number of valid top level windows on the screen defined by
+// the |bounds| rectangle.
+int GetNumberOfValidTopLevelWindows(const gfx::Rect& bounds) {

[sky, 2012/09/13 15:44:51]

This method is named as though it wants all top levels, but it only looks at
browsers. Please name accordingly.

[Mr4D (OOO till 08-26), 2012/09/13 16:45:08]

Done.

+  int count = 0;
+  for (BrowserList::const_iterator iter = BrowserList::begin();
+       iter != BrowserList::end();
+       ++iter) {
+    Browser* browser = *iter;
+
+    if (browser && browser->window() &&

[sky, 2012/09/13 15:44:51]

Can this and IsToplevelWindow share code?

[Mr4D (OOO till 08-26), 2012/09/13 16:45:08]

Done.

+        bounds.Intersects(browser->window()->GetBounds()) &&
+        !browser->window()->IsMinimized() &&
+        !(browser->is_type_popup() || browser->is_type_panel()))
+      count++;
+  }
+  return count;
+}
+
+// Move the given |bounds| on the available |work_area| to the direction.
+// If |move_right| is true, the rectangle gets moved to the right corner.
+// otherwise to the left side.
+bool MoveRect(const gfx::Rect& work_area, gfx::Rect& bounds, bool move_right) {
+  if (move_right) {
+    if (work_area.right() > bounds.right()) {
+      bounds.set_x(work_area.right() - bounds.width());
+      return true;
+    }
+  } else {
+    if (work_area.x() < bounds.x()) {
+      bounds.set_x(work_area.x());
+      return true;
+    }
+  }
+  return false;
+}
+
 }  // namespace
 
-bool WindowSizer::GetBoundsIgnoringPreviousStateAsh(
-    const gfx::Rect& specified_bounds,
-    gfx::Rect* bounds) const {
+bool WindowSizer::GetBoundsOverrideAsh(const gfx::Rect& specified_bounds,
+                                       gfx::Rect* bounds) const {
   *bounds = specified_bounds;
   DCHECK(bounds->IsEmpty());
+
+  if (!GetSavedWindowBounds(bounds))
+    GetDefaultWindowBounds(bounds);
+
   if (browser_ != NULL && browser_->type() == Browser::TYPE_TABBED) {
+    gfx::Rect work_area =
+        monitor_info_provider_->GetMonitorWorkAreaMatching(*bounds);
     // This is a window / app. See if there is no window and try to place it.
-    aura::Window* top_window = GetTopWindow();
-    // If there are no windows we have a special case and try to
-    // maximize which leaves a 'border' which shows the desktop.
-    if (top_window == NULL) {
-      GetDefaultWindowBounds(bounds);
-    } else {
-      *bounds = top_window->GetBoundsInScreen();
-      gfx::Rect work_area =
-          monitor_info_provider_->GetMonitorWorkAreaMatching(*bounds);
-      *bounds = bounds->AdjustToFit(work_area);
+    int count = GetNumberOfValidTopLevelWindows(work_area);
+    aura::Window* top_window = GetTopWindow(work_area);
+
+    // If there is no valid other window we take the coordinates as is.
+    if (!count || !top_window || ash::wm::IsWindowMaximized(top_window))
+      return true;
+
+    gfx::Rect other_bounds = top_window->bounds();
+    bool move_right =
+        other_bounds.CenterPoint().x() < work_area.CenterPoint().x();
+
+    // In case we have only one window, we move the other window fully to the
+    // "other side" - making room for this new window.
+    if (count == 1) {
+      if (MoveRect(work_area, other_bounds, !move_right))
+        top_window->SetBounds(other_bounds);
     }
+    // Use the size of the other window, and mirror the location to the
+    // opposite side. Then make sure that it is inside our work area
+    // (if possible).
+    *bounds = other_bounds;
+    MoveRect(work_area, *bounds, move_right);
+    if (bounds->bottom() > work_area.bottom())
+      bounds->set_y(std::max(work_area.y(),
+                             work_area.bottom() - bounds->height()));
     return true;
-    // If both fail we will continue the default path.
   }
 
   return false;
 }
 
 void WindowSizer::GetDefaultWindowBoundsAsh(gfx::Rect* default_bounds) const {
   DCHECK(default_bounds);
   DCHECK(monitor_info_provider_.get());
 
   gfx::Rect work_area = monitor_info_provider_->GetPrimaryDisplayWorkArea();
@@ skipping 12 matching lines @@
   if (default_width > kMaximumWindowWidth) {
     // The window should get centered on the screen and not follow the grid.
     offset_x = (work_area.width() - kMaximumWindowWidth) / 2;
     default_width = kMaximumWindowWidth;
   }
   default_bounds->SetRect(work_area.x() + offset_x,
                           work_area.y() + kDesktopBorderSize,
                           default_width,
                           default_height);
 }