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

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: Ash unittests compile (cleanup #include wm_window.h) 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 0c3dc981a81349903f7b4feb8736d61fbf6f7ca8..93c23cccb225ea63b52d8aa6de226ea0cb1da735 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