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

Unified Diff: ash/wm/workspace/workspace_layout_manager.cc

Issue 2895713002: [mus+ash] Removes WmWindow from ash/wm/mru_window_tracker and overview mode (Closed)
Patch Set: Address nits, unit_tests target compiles Created 3 years, 7 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
Index: ash/wm/workspace/workspace_layout_manager.cc
diff --git a/ash/wm/workspace/workspace_layout_manager.cc b/ash/wm/workspace/workspace_layout_manager.cc
index b861681442c95591faaf5725add6456bc699297d..b8e06c2d26e78aaa86088e01a810dd3ec3227b69 100644
--- a/ash/wm/workspace/workspace_layout_manager.cc
+++ b/ash/wm/workspace/workspace_layout_manager.cc
@@ -87,7 +87,7 @@ void WorkspaceLayoutManager::OnWindowAddedToLayout(aura::Window* child) {
UpdateFullscreenState();
backdrop_controller_->OnWindowAddedToLayout(child);
- WindowPositioner::RearrangeVisibleWindowOnShow(WmWindow::Get(child));
+ WindowPositioner::RearrangeVisibleWindowOnShow(child);
if (Shell::Get()->screen_pinning_controller()->IsPinned())
wm::GetWindowState(child)->DisableAlwaysOnTop(nullptr);
}
@@ -98,8 +98,7 @@ void WorkspaceLayoutManager::OnWillRemoveWindowFromLayout(aura::Window* child) {
wm::GetWindowState(child)->RemoveObserver(this);
if (child->layer()->GetTargetVisibility())
- WindowPositioner::RearrangeVisibleWindowOnHideOrRemove(
- WmWindow::Get(child));
+ WindowPositioner::RearrangeVisibleWindowOnHideOrRemove(child);
}
void WorkspaceLayoutManager::OnWindowRemovedFromLayout(aura::Window* child) {
@@ -116,10 +115,9 @@ void WorkspaceLayoutManager::OnChildWindowVisibilityChanged(aura::Window* child,
window_state->Unminimize();
if (child->layer()->GetTargetVisibility())
- WindowPositioner::RearrangeVisibleWindowOnShow(WmWindow::Get(child));
+ WindowPositioner::RearrangeVisibleWindowOnShow(child);
else
- WindowPositioner::RearrangeVisibleWindowOnHideOrRemove(
- WmWindow::Get(child));
+ WindowPositioner::RearrangeVisibleWindowOnHideOrRemove(child);
UpdateFullscreenState();
UpdateShelfVisibility();
backdrop_controller_->OnChildWindowVisibilityChanged(child, visible);

Powered by Google App Engine
This is Rietveld 408576698