Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(8872)

Unified Diff: ash/wm/window_positioner.cc

Issue 1900443002: Removes aura dependencies from WindowPositioner (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: nuke GetWorkAreaForWindowInParent and fix windows Created 4 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « ash/wm/window_positioner.h ('k') | ash/wm/window_positioner_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ash/wm/window_positioner.cc
diff --git a/ash/wm/window_positioner.cc b/ash/wm/window_positioner.cc
index 1c705fdf7740c7f249b973b2e25d87e1bc9e7356..1b295499a3670e5926d1ae3f55a5b6862235d87a 100644
--- a/ash/wm/window_positioner.cc
+++ b/ash/wm/window_positioner.cc
@@ -4,23 +4,14 @@
#include "ash/wm/window_positioner.h"
-#include "ash/screen_util.h"
-#include "ash/shell.h"
-#include "ash/shell_delegate.h"
-#include "ash/shell_window_ids.h"
-#include "ash/wm/mru_window_tracker.h"
-#include "ash/wm/window_resizer.h"
+#include "ash/wm/common/window_positioning_utils.h"
+#include "ash/wm/common/wm_globals.h"
+#include "ash/wm/common/wm_screen_util.h"
+#include "ash/wm/common/wm_window.h"
#include "ash/wm/window_state.h"
-#include "ash/wm/window_state_aura.h"
-#include "ash/wm/window_util.h"
-#include "ui/aura/window.h"
-#include "ui/aura/window_delegate.h"
-#include "ui/aura/window_event_dispatcher.h"
#include "ui/compositor/layer.h"
-#include "ui/compositor/scoped_layer_animation_settings.h"
+#include "ui/gfx/geometry/insets.h"
#include "ui/gfx/screen.h"
-#include "ui/wm/core/window_animations.h"
-#include "ui/wm/core/window_util.h"
namespace ash {
@@ -54,10 +45,10 @@ static bool disable_auto_positioning = false;
static bool maximize_first_window = false;
// Check if any management should be performed (with a given |window|).
-bool UseAutoWindowManager(const aura::Window* window) {
+bool UseAutoWindowManager(const wm::WmWindow* window) {
if (disable_auto_positioning)
return false;
- const wm::WindowState* window_state = wm::GetWindowState(window);
+ const wm::WindowState* window_state = window->GetWindowState();
return !window_state->is_dragged() && window_state->window_position_managed();
}
@@ -65,33 +56,16 @@ bool UseAutoWindowManager(const aura::Window* window) {
// state is not minimized/maximized/fullscreen/the user has changed
// its size by hand already. It furthermore checks for the
// WindowIsManaged status.
-bool WindowPositionCanBeManaged(const aura::Window* window) {
+bool WindowPositionCanBeManaged(const wm::WmWindow* window) {
if (disable_auto_positioning)
return false;
- const wm::WindowState* window_state = wm::GetWindowState(window);
+ const wm::WindowState* window_state = window->GetWindowState();
return window_state->window_position_managed() &&
!window_state->IsMinimized() && !window_state->IsMaximized() &&
!window_state->IsFullscreen() &&
!window_state->bounds_changed_by_user();
}
-// Get the work area for a given |window| in parent coordinates.
-gfx::Rect GetWorkAreaForWindowInParent(aura::Window* window) {
-#if defined(OS_WIN)
- // On Win 8, the host window can't be resized, so
- // use window's bounds instead.
- // TODO(oshima): Emulate host window resize on win8.
- gfx::Rect work_area = gfx::Rect(window->parent()->bounds().size());
- work_area.Inset(
- gfx::Screen::GetScreen()
- ->GetDisplayMatching(window->parent()->GetBoundsInScreen())
- .GetWorkAreaInsets());
- return work_area;
-#else
- return ScreenUtil::GetDisplayWorkAreaBoundsInParent(window);
-#endif
-}
-
// Move the given |bounds| on the available |work_area| in the direction
// indicated by |move_right|. If |move_right| is true, the rectangle gets moved
// to the right edge, otherwise to the left one.
@@ -115,16 +89,14 @@ bool MoveRectToOneSide(const gfx::Rect& work_area,
// Move a |window| to new |bounds|. Animate if desired by user.
// Moves the transient children of the |window| as well by the same |offset| as
// the parent |window|.
-void SetBoundsAndOffsetTransientChildren(aura::Window* window,
+void SetBoundsAndOffsetTransientChildren(wm::WmWindow* window,
const gfx::Rect& bounds,
const gfx::Rect& work_area,
const gfx::Vector2d& offset) {
- aura::Window::Windows transient_children =
- ::wm::GetTransientChildren(window);
- for (aura::Window::Windows::iterator iter = transient_children.begin();
- iter != transient_children.end(); ++iter) {
- aura::Window* transient_child = *iter;
- gfx::Rect child_bounds = transient_child->bounds();
+ std::vector<wm::WmWindow*> transient_children =
+ window->GetTransientChildren();
+ for (wm::WmWindow* transient_child : transient_children) {
+ gfx::Rect child_bounds = transient_child->GetBounds();
gfx::Rect new_child_bounds = child_bounds + offset;
if ((child_bounds.x() <= work_area.x() &&
new_child_bounds.x() <= work_area.x()) ||
@@ -140,20 +112,13 @@ void SetBoundsAndOffsetTransientChildren(aura::Window* window,
new_child_bounds, work_area, offset);
}
- if (::wm::WindowAnimationsDisabled(window)) {
- window->SetBounds(bounds);
- return;
- }
-
- ui::ScopedLayerAnimationSettings settings(window->layer()->GetAnimator());
- settings.SetTransitionDuration(
- base::TimeDelta::FromMilliseconds(kWindowAutoMoveDurationMS));
- window->SetBounds(bounds);
+ window->SetBoundsWithTransitionDelay(
+ bounds, base::TimeDelta::FromMilliseconds(kWindowAutoMoveDurationMS));
}
// Move a |window| to new |bounds|. Animate if desired by user.
// Note: The function will do nothing if the bounds did not change.
-void SetBoundsAnimated(aura::Window* window,
+void SetBoundsAnimated(wm::WmWindow* window,
const gfx::Rect& bounds,
const gfx::Rect& work_area) {
gfx::Rect old_bounds = window->GetTargetBounds();
@@ -165,14 +130,14 @@ void SetBoundsAnimated(aura::Window* window,
// Move |window| into the center of the screen - or restore it to the previous
// position.
-void AutoPlaceSingleWindow(aura::Window* window, bool animated) {
- gfx::Rect work_area = GetWorkAreaForWindowInParent(window);
- gfx::Rect bounds = window->bounds();
+void AutoPlaceSingleWindow(wm::WmWindow* window, bool animated) {
+ gfx::Rect work_area = GetDisplayWorkAreaBoundsInParent(window);
+ gfx::Rect bounds = window->GetBounds();
const gfx::Rect* user_defined_area =
- wm::GetWindowState(window)->pre_auto_manage_window_bounds();
+ window->GetWindowState()->pre_auto_manage_window_bounds();
if (user_defined_area) {
bounds = *user_defined_area;
- ash::wm::AdjustBoundsToEnsureMinimumWindowVisibility(work_area, &bounds);
+ wm::AdjustBoundsToEnsureMinimumWindowVisibility(work_area, &bounds);
} else {
// Center the window (only in x).
bounds.set_x(work_area.x() + (work_area.width() - bounds.width()) / 2);
@@ -185,42 +150,41 @@ void AutoPlaceSingleWindow(aura::Window* window, bool animated) {
}
// Get the first open (non minimized) window which is on the screen defined.
-aura::Window* GetReferenceWindow(const aura::Window* root_window,
- const aura::Window* exclude,
- bool *single_window) {
+wm::WmWindow* GetReferenceWindow(const wm::WmWindow* root_window,
+ const wm::WmWindow* exclude,
+ bool* single_window) {
if (single_window)
*single_window = true;
// Get the active window.
- aura::Window* active = ash::wm::GetActiveWindow();
+ wm::WmWindow* active = root_window->GetGlobals()->GetActiveWindow();
if (active && active->GetRootWindow() != root_window)
active = NULL;
// Get a list of all windows.
- const std::vector<aura::Window*> windows = ash::Shell::GetInstance()->
- mru_window_tracker()->BuildWindowListIgnoreModal();
+ const std::vector<wm::WmWindow*> windows =
+ root_window->GetGlobals()->GetMruWindowListIgnoreModals();
if (windows.empty())
- return NULL;
+ return nullptr;
- aura::Window::Windows::const_iterator iter = windows.begin();
+ int index = 0;
// Find the index of the current active window.
if (active)
- iter = std::find(windows.begin(), windows.end(), active);
-
- int index = (iter == windows.end()) ? 0 : (iter - windows.begin());
+ index = std::find(windows.begin(), windows.end(), active) - 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.
- aura::Window* found = NULL;
+ wm::WmWindow* found = nullptr;
for (int i = index + windows.size(); i >= 0; i--) {
- aura::Window* window = windows[i % windows.size()];
- while (::wm::GetTransientParent(window))
- window = ::wm::GetTransientParent(window);
- if (window != exclude && window->type() == ui::wm::WINDOW_TYPE_NORMAL &&
- window->GetRootWindow() == root_window && window->TargetVisibility() &&
- wm::GetWindowState(window)->window_position_managed()) {
+ wm::WmWindow* window = windows[i % windows.size()];
+ while (window->GetTransientParent())
+ window = window->GetTransientParent();
+ if (window != exclude && window->GetType() == ui::wm::WINDOW_TYPE_NORMAL &&
+ window->GetRootWindow() == root_window &&
+ window->GetTargetVisibility() &&
+ window->GetWindowState()->window_position_managed()) {
if (found && found != window) {
// no need to check !single_window because the function must have
// been already returned in the "if (!single_window)" below.
@@ -245,24 +209,23 @@ int WindowPositioner::GetForceMaximizedWidthLimit() {
// static
void WindowPositioner::GetBoundsAndShowStateForNewWindow(
- const gfx::Screen* screen,
- const aura::Window* new_window,
+ const wm::WmWindow* new_window,
bool is_saved_bounds,
ui::WindowShowState show_state_in,
gfx::Rect* bounds_in_out,
ui::WindowShowState* show_state_out) {
// Always open new window in the target display.
- aura::Window* target = Shell::GetTargetRootWindow();
+ wm::WmWindow* target = wm::WmGlobals::Get()->GetRootWindowForNewWindows();
- aura::Window* top_window = GetReferenceWindow(target, NULL, NULL);
+ wm::WmWindow* top_window = GetReferenceWindow(target, nullptr, nullptr);
// Our window should not have any impact if we are already on top.
if (top_window == new_window)
- top_window = NULL;
+ top_window = nullptr;
// If there is no valid other window we take and adjust the passed coordinates
// and show state.
if (!top_window) {
- gfx::Rect work_area = screen->GetDisplayNearestWindow(target).work_area();
+ gfx::Rect work_area = target->GetDisplayNearestWindow().work_area();
bounds_in_out->AdjustToFit(work_area);
// Use adjusted saved bounds, if there is one.
@@ -271,14 +234,14 @@ void WindowPositioner::GetBoundsAndShowStateForNewWindow(
if (show_state_in == ui::SHOW_STATE_DEFAULT) {
const bool maximize_first_window_on_first_run =
- Shell::GetInstance()->delegate()->IsForceMaximizeOnFirstRun();
+ target->GetGlobals()->IsForceMaximizeOnFirstRun();
// We want to always open maximized on "small screens" or when policy
// tells us to.
const bool set_maximized =
maximize_first_window ||
((work_area.width() <= GetForceMaximizedWidthLimit() ||
maximize_first_window_on_first_run) &&
- (!new_window || !wm::GetWindowState(new_window)->IsFullscreen()));
+ (!new_window || !new_window->GetWindowState()->IsFullscreen()));
if (set_maximized)
*show_state_out = ui::SHOW_STATE_MAXIMIZED;
@@ -286,7 +249,7 @@ void WindowPositioner::GetBoundsAndShowStateForNewWindow(
return;
}
- wm::WindowState* top_window_state = wm::GetWindowState(top_window);
+ wm::WindowState* top_window_state = top_window->GetWindowState();
bool maximized = top_window_state->IsMaximized();
// We ignore the saved show state, but look instead for the top level
// window's show state.
@@ -306,7 +269,7 @@ void WindowPositioner::GetBoundsAndShowStateForNewWindow(
gfx::Vector2d(kMinimumWindowOffset, kMinimumWindowOffset);
}
if (is_saved_bounds || has_restore_bounds) {
- gfx::Rect work_area = screen->GetDisplayNearestWindow(target).work_area();
+ gfx::Rect work_area = target->GetDisplayNearestWindow().work_area();
bounds_in_out->AdjustToFit(work_area);
// Use adjusted saved bounds or restore bounds, if there is one.
return;
@@ -320,12 +283,12 @@ void WindowPositioner::GetBoundsAndShowStateForNewWindow(
// static
void WindowPositioner::RearrangeVisibleWindowOnHideOrRemove(
- const aura::Window* removed_window) {
+ const wm::WmWindow* removed_window) {
if (!UseAutoWindowManager(removed_window))
return;
// Find a single open browser window.
bool single_window;
- aura::Window* other_shown_window = GetReferenceWindow(
+ wm::WmWindow* other_shown_window = GetReferenceWindow(
removed_window->GetRootWindow(), removed_window, &single_window);
if (!other_shown_window || !single_window ||
!WindowPositionCanBeManaged(other_shown_window))
@@ -342,20 +305,19 @@ bool WindowPositioner::DisableAutoPositioning(bool ignore) {
// static
void WindowPositioner::RearrangeVisibleWindowOnShow(
- aura::Window* added_window) {
- wm::WindowState* added_window_state = wm::GetWindowState(added_window);
- if (!added_window->TargetVisibility())
+ wm::WmWindow* added_window) {
+ wm::WindowState* added_window_state = added_window->GetWindowState();
+ if (!added_window->GetTargetVisibility())
return;
if (!UseAutoWindowManager(added_window) ||
added_window_state->bounds_changed_by_user()) {
if (added_window_state->minimum_visibility()) {
// Guarantee minimum visibility within the work area.
- gfx::Rect work_area = GetWorkAreaForWindowInParent(added_window);
- gfx::Rect bounds = added_window->bounds();
+ gfx::Rect work_area = GetDisplayWorkAreaBoundsInParent(added_window);
+ gfx::Rect bounds = added_window->GetBounds();
gfx::Rect new_bounds = bounds;
- ash::wm::AdjustBoundsToEnsureMinimumWindowVisibility(work_area,
- &new_bounds);
+ wm::AdjustBoundsToEnsureMinimumWindowVisibility(work_area, &new_bounds);
if (new_bounds != bounds)
added_window->SetBounds(new_bounds);
}
@@ -363,7 +325,7 @@ void WindowPositioner::RearrangeVisibleWindowOnShow(
}
// Find a single open managed window.
bool single_window;
- aura::Window* other_shown_window = GetReferenceWindow(
+ wm::WmWindow* other_shown_window = GetReferenceWindow(
added_window->GetRootWindow(), added_window, &single_window);
if (!other_shown_window) {
@@ -376,8 +338,8 @@ void WindowPositioner::RearrangeVisibleWindowOnShow(
return;
}
- gfx::Rect other_bounds = other_shown_window->bounds();
- gfx::Rect work_area = GetWorkAreaForWindowInParent(added_window);
+ gfx::Rect other_bounds = other_shown_window->GetBounds();
+ gfx::Rect work_area = GetDisplayWorkAreaBoundsInParent(added_window);
bool move_other_right =
other_bounds.CenterPoint().x() > work_area.x() + work_area.width() / 2;
@@ -386,8 +348,7 @@ void WindowPositioner::RearrangeVisibleWindowOnShow(
// When going from one to two windows both windows loose their
// "positioned by user" flags.
added_window_state->set_bounds_changed_by_user(false);
- wm::WindowState* other_window_state =
- wm::GetWindowState(other_shown_window);
+ wm::WindowState* other_window_state = other_shown_window->GetWindowState();
other_window_state->set_bounds_changed_by_user(false);
if (WindowPositionCanBeManaged(other_shown_window)) {
@@ -405,21 +366,21 @@ void WindowPositioner::RearrangeVisibleWindowOnShow(
// Remember the current location of the window if it's new and push
// it also to the opposite location if needed. Since it is just
// being shown, we do not need to animate it.
- gfx::Rect added_bounds = added_window->bounds();
+ gfx::Rect added_bounds = added_window->GetBounds();
if (!added_window_state->pre_auto_manage_window_bounds())
added_window_state->SetPreAutoManageWindowBounds(added_bounds);
if (MoveRectToOneSide(work_area, !move_other_right, &added_bounds))
added_window->SetBounds(added_bounds);
}
-WindowPositioner::WindowPositioner()
- : pop_position_offset_increment_x(0),
+WindowPositioner::WindowPositioner(wm::WmGlobals* globals)
+ : globals_(globals),
+ pop_position_offset_increment_x(0),
pop_position_offset_increment_y(0),
popup_position_offset_from_screen_corner_x(0),
popup_position_offset_from_screen_corner_y(0),
last_popup_position_x_(0),
- last_popup_position_y_(0) {
-}
+ last_popup_position_y_(0) {}
WindowPositioner::~WindowPositioner() {
}
@@ -459,12 +420,10 @@ gfx::Rect WindowPositioner::GetPopupPosition(const gfx::Rect& old_pos) {
pop_position_offset_increment_y = grid;
// We handle the Multi monitor support by retrieving the active window's
// work area.
- aura::Window* window = wm::GetActiveWindow();
+ wm::WmWindow* window = globals_->GetActiveWindow();
const gfx::Rect work_area =
window && window->IsVisible()
- ? gfx::Screen::GetScreen()
- ->GetDisplayNearestWindow(window)
- .work_area()
+ ? window->GetDisplayNearestWindow().work_area()
: gfx::Screen::GetScreen()->GetPrimaryDisplay().work_area();
// Only try to reposition the popup when it is not spanning the entire
// screen.
@@ -521,22 +480,22 @@ gfx::Rect WindowPositioner::SmartPopupPosition(
const gfx::Rect& old_pos,
const gfx::Rect& work_area,
int grid) {
- const std::vector<aura::Window*> windows = ash::Shell::GetInstance()->
- mru_window_tracker()->BuildWindowListIgnoreModal();
+ const std::vector<wm::WmWindow*> windows =
+ globals_->GetMruWindowListIgnoreModals();
std::vector<const gfx::Rect*> regions;
// Process the window list and check if we can bail immediately.
for (size_t i = 0; i < windows.size(); i++) {
// We only include opaque and visible windows.
- if (windows[i] && windows[i]->IsVisible() && windows[i]->layer() &&
- (!windows[i]->transparent() ||
- windows[i]->layer()->GetTargetOpacity() == 1.0)) {
- wm::WindowState* window_state = wm::GetWindowState(windows[i]);
+ if (windows[i] && windows[i]->IsVisible() && windows[i]->GetLayer() &&
+ (windows[i]->GetLayer()->fills_bounds_opaquely() ||
+ windows[i]->GetLayer()->GetTargetOpacity() == 1.0)) {
+ wm::WindowState* window_state = windows[i]->GetWindowState();
// When any window is maximized we cannot find any free space.
if (window_state->IsMaximizedOrFullscreen())
return gfx::Rect(0, 0, 0, 0);
if (window_state->IsNormalOrSnapped())
- regions.push_back(&windows[i]->bounds());
+ regions.push_back(&windows[i]->GetBounds());
}
}
« no previous file with comments | « ash/wm/window_positioner.h ('k') | ash/wm/window_positioner_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698