New issue
Advanced search Search tips

Issue 687657 link

Starred by 1 user

Issue metadata

Status: Archived
Owner:
Closed: May 2017
Cc:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug

Blocking:
issue 671246



Sign in to add a comment

Remove WmLayoutManager

Project Member Reported by sky@chromium.org, Feb 1 2017

Issue description

And use aura::LayoutManager directly
 

Comment 1 by sky@chromium.org, Feb 1 2017

Blocking: 671246

Comment 2 by yiyix@chromium.org, Mar 7 2017

Owner: yiyix@chromium.org
Status: Assigned (was: Untriaged)

Comment 3 by yiyix@chromium.org, Mar 8 2017

Status: Started (was: Assigned)
Project Member

Comment 6 by bugdroid1@chromium.org, May 8 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/3919a2d21a0c3df90ab7077aa6ebee97262dfd29

commit 3919a2d21a0c3df90ab7077aa6ebee97262dfd29
Author: sky <sky@chromium.org>
Date: Mon May 08 15:52:27 2017

chromeos: Converts WorkspaceLayoutManager to aura::LayoutManager

BUG= 687657 
TEST=covered by tests
R=msw@chromium.org

Review-Url: https://codereview.chromium.org/2861243003
Cr-Commit-Position: refs/heads/master@{#469993}

[modify] https://crrev.com/3919a2d21a0c3df90ab7077aa6ebee97262dfd29/ash/root_window_controller.cc
[modify] https://crrev.com/3919a2d21a0c3df90ab7077aa6ebee97262dfd29/ash/shelf/shelf_layout_manager.cc
[modify] https://crrev.com/3919a2d21a0c3df90ab7077aa6ebee97262dfd29/ash/wm/always_on_top_controller.cc
[modify] https://crrev.com/3919a2d21a0c3df90ab7077aa6ebee97262dfd29/ash/wm/always_on_top_controller_unittest.cc
[modify] https://crrev.com/3919a2d21a0c3df90ab7077aa6ebee97262dfd29/ash/wm/fullscreen_window_finder.cc
[modify] https://crrev.com/3919a2d21a0c3df90ab7077aa6ebee97262dfd29/ash/wm/fullscreen_window_finder.h
[modify] https://crrev.com/3919a2d21a0c3df90ab7077aa6ebee97262dfd29/ash/wm/mru_window_tracker.cc
[modify] https://crrev.com/3919a2d21a0c3df90ab7077aa6ebee97262dfd29/ash/wm/switchable_windows.cc
[modify] https://crrev.com/3919a2d21a0c3df90ab7077aa6ebee97262dfd29/ash/wm/switchable_windows.h
[modify] https://crrev.com/3919a2d21a0c3df90ab7077aa6ebee97262dfd29/ash/wm/workspace/workspace_layout_manager.cc
[modify] https://crrev.com/3919a2d21a0c3df90ab7077aa6ebee97262dfd29/ash/wm/workspace/workspace_layout_manager.h
[modify] https://crrev.com/3919a2d21a0c3df90ab7077aa6ebee97262dfd29/ash/wm/workspace/workspace_layout_manager_keyboard_unittest.cc
[modify] https://crrev.com/3919a2d21a0c3df90ab7077aa6ebee97262dfd29/ash/wm/workspace/workspace_layout_manager_unittest.cc
[modify] https://crrev.com/3919a2d21a0c3df90ab7077aa6ebee97262dfd29/ash/wm/workspace_controller.cc
[modify] https://crrev.com/3919a2d21a0c3df90ab7077aa6ebee97262dfd29/ash/wm/workspace_controller.h

Project Member

Comment 7 by bugdroid1@chromium.org, May 9 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/97f9a74388f0a4ef84ac44a9e811dbb106c35275

commit 97f9a74388f0a4ef84ac44a9e811dbb106c35275
Author: sky <sky@chromium.org>
Date: Tue May 09 05:14:31 2017

Converts remaining usage of WmLayoutManager to aura::LayoutManager

Yay!

BUG= 687657 
TEST=covered by tests
R=msw@chromium.org

Review-Url: https://codereview.chromium.org/2871813002
Cr-Commit-Position: refs/heads/master@{#470233}

[modify] https://crrev.com/97f9a74388f0a4ef84ac44a9e811dbb106c35275/ash/BUILD.gn
[delete] https://crrev.com/6b2241e2042fe5365c84d84828c8db3a3625c18f/ash/aura/aura_layout_manager_adapter.cc
[delete] https://crrev.com/6b2241e2042fe5365c84d84828c8db3a3625c18f/ash/aura/aura_layout_manager_adapter.h
[modify] https://crrev.com/97f9a74388f0a4ef84ac44a9e811dbb106c35275/ash/keyboard/keyboard_observer_register.cc
[modify] https://crrev.com/97f9a74388f0a4ef84ac44a9e811dbb106c35275/ash/keyboard/keyboard_observer_register.h
[modify] https://crrev.com/97f9a74388f0a4ef84ac44a9e811dbb106c35275/ash/root_window_controller.cc
[modify] https://crrev.com/97f9a74388f0a4ef84ac44a9e811dbb106c35275/ash/root_window_controller.h
[modify] https://crrev.com/97f9a74388f0a4ef84ac44a9e811dbb106c35275/ash/root_window_controller_unittest.cc
[modify] https://crrev.com/97f9a74388f0a4ef84ac44a9e811dbb106c35275/ash/shelf/shelf_layout_manager.cc
[modify] https://crrev.com/97f9a74388f0a4ef84ac44a9e811dbb106c35275/ash/shelf/shelf_layout_manager.h
[modify] https://crrev.com/97f9a74388f0a4ef84ac44a9e811dbb106c35275/ash/shelf/shelf_widget.cc
[modify] https://crrev.com/97f9a74388f0a4ef84ac44a9e811dbb106c35275/ash/shell.cc
[modify] https://crrev.com/97f9a74388f0a4ef84ac44a9e811dbb106c35275/ash/shell.h
[modify] https://crrev.com/97f9a74388f0a4ef84ac44a9e811dbb106c35275/ash/shell_observer.h
[modify] https://crrev.com/97f9a74388f0a4ef84ac44a9e811dbb106c35275/ash/shell_port.cc
[modify] https://crrev.com/97f9a74388f0a4ef84ac44a9e811dbb106c35275/ash/shell_port.h
[modify] https://crrev.com/97f9a74388f0a4ef84ac44a9e811dbb106c35275/ash/system/status_area_layout_manager.cc
[modify] https://crrev.com/97f9a74388f0a4ef84ac44a9e811dbb106c35275/ash/system/status_area_layout_manager.h
[modify] https://crrev.com/97f9a74388f0a4ef84ac44a9e811dbb106c35275/ash/wm/lock_layout_manager.cc
[modify] https://crrev.com/97f9a74388f0a4ef84ac44a9e811dbb106c35275/ash/wm/lock_layout_manager.h
[modify] https://crrev.com/97f9a74388f0a4ef84ac44a9e811dbb106c35275/ash/wm/panels/panel_layout_manager.cc
[modify] https://crrev.com/97f9a74388f0a4ef84ac44a9e811dbb106c35275/ash/wm/panels/panel_layout_manager.h
[modify] https://crrev.com/97f9a74388f0a4ef84ac44a9e811dbb106c35275/ash/wm/root_window_layout_manager.cc
[modify] https://crrev.com/97f9a74388f0a4ef84ac44a9e811dbb106c35275/ash/wm/root_window_layout_manager.h
[modify] https://crrev.com/97f9a74388f0a4ef84ac44a9e811dbb106c35275/ash/wm/system_modal_container_layout_manager.cc
[modify] https://crrev.com/97f9a74388f0a4ef84ac44a9e811dbb106c35275/ash/wm/system_modal_container_layout_manager.h
[modify] https://crrev.com/97f9a74388f0a4ef84ac44a9e811dbb106c35275/ash/wm/system_modal_container_layout_manager_unittest.cc
[modify] https://crrev.com/97f9a74388f0a4ef84ac44a9e811dbb106c35275/ash/wm/window_positioning_utils.cc
[modify] https://crrev.com/97f9a74388f0a4ef84ac44a9e811dbb106c35275/ash/wm/wm_snap_to_pixel_layout_manager.cc
[modify] https://crrev.com/97f9a74388f0a4ef84ac44a9e811dbb106c35275/ash/wm/wm_snap_to_pixel_layout_manager.h
[modify] https://crrev.com/97f9a74388f0a4ef84ac44a9e811dbb106c35275/ash/wm/workspace/workspace_layout_manager.cc
[modify] https://crrev.com/97f9a74388f0a4ef84ac44a9e811dbb106c35275/ash/wm/workspace/workspace_layout_manager.h
[delete] https://crrev.com/6b2241e2042fe5365c84d84828c8db3a3625c18f/ash/wm_layout_manager.h
[modify] https://crrev.com/97f9a74388f0a4ef84ac44a9e811dbb106c35275/ash/wm_window.cc
[modify] https://crrev.com/97f9a74388f0a4ef84ac44a9e811dbb106c35275/ash/wm_window.h
[modify] https://crrev.com/97f9a74388f0a4ef84ac44a9e811dbb106c35275/chrome/browser/chromeos/login/ui/webui_login_view.cc
[modify] https://crrev.com/97f9a74388f0a4ef84ac44a9e811dbb106c35275/chrome/browser/chromeos/login/ui/webui_login_view.h

Comment 8 by sky@chromium.org, May 9 2017

Owner: sky@chromium.org
Status: Fixed (was: Started)

Comment 9 by dchan@chromium.org, Aug 1 2017

Labels: VerifyIn-61

Comment 10 by dchan@chromium.org, Jan 22 2018

Status: Archived (was: Fixed)

Sign in to add a comment