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

Issue 1918883002: Makes Panel related classes use ash/wm/common types (Closed)

Created:
4 years, 8 months ago by sky
Modified:
4 years, 8 months ago
Reviewers:
James Cook
CC:
chromium-reviews, kalyank, sadrul, dcheng
Base URL:
https://chromium.googlesource.com/chromium/src.git@phantom_window_controller
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Makes Panel related classes use ash/wm/common types Specifically the layoutmanager and windowresizer implementations. And this removes GetAuraTarget() from WmWindowResizer. BUG=603369 TEST=covered by tests R=jamescook@chromium.org Committed: https://crrev.com/ad031502362573a9fe05c53c8f0776e5f7009dba Cr-Commit-Position: refs/heads/master@{#389540}

Patch Set 1 #

Patch Set 2 : merge to trunk #

Patch Set 3 : tweak #

Patch Set 4 : fix windows #

Total comments: 5

Patch Set 5 : feedback #

Unified diffs Side-by-side diffs Delta from patch set Stats (+451 lines, -325 lines) Patch
M ash/accelerators/accelerator_controller_unittest.cc View 1 chunk +2 lines, -3 lines 0 comments Download
M ash/ash.gyp View 1 chunk +2 lines, -0 lines 0 comments Download
M ash/root_window_controller.cc View 1 chunk +3 lines, -2 lines 0 comments Download
M ash/wm/aura/wm_globals_aura.h View 4 chunks +20 lines, -1 line 0 comments Download
M ash/wm/aura/wm_globals_aura.cc View 5 chunks +38 lines, -0 lines 0 comments Download
M ash/wm/aura/wm_window_aura.h View 1 chunk +1 line, -0 lines 0 comments Download
M ash/wm/aura/wm_window_aura.cc View 3 chunks +15 lines, -1 line 0 comments Download
A ash/wm/common/wm_display_observer.h View 1 chunk +27 lines, -0 lines 0 comments Download
M ash/wm/common/wm_globals.h View 2 chunks +8 lines, -0 lines 0 comments Download
A ash/wm/common/wm_overview_mode_observer.h View 1 chunk +26 lines, -0 lines 0 comments Download
M ash/wm/common/wm_window.h View 1 chunk +1 line, -0 lines 0 comments Download
M ash/wm/common/wm_window_property.h View 1 chunk +3 lines, -0 lines 0 comments Download
M ash/wm/dock/docked_window_layout_manager_unittest.cc View 3 chunks +3 lines, -9 lines 0 comments Download
M ash/wm/dock/docked_window_resizer_unittest.cc View 1 chunk +2 lines, -3 lines 0 comments Download
M ash/wm/drag_window_resizer.h View 1 chunk +2 lines, -0 lines 0 comments Download
M ash/wm/drag_window_resizer.cc View 1 chunk +4 lines, -0 lines 0 comments Download
M ash/wm/overview/window_selector.cc View 3 chunks +5 lines, -6 lines 0 comments Download
M ash/wm/overview/window_selector_unittest.cc View 2 chunks +10 lines, -14 lines 0 comments Download
M ash/wm/panels/panel_layout_manager.h View 1 2 3 4 8 chunks +59 lines, -41 lines 0 comments Download
M ash/wm/panels/panel_layout_manager.cc View 1 2 3 4 33 chunks +155 lines, -152 lines 0 comments Download
M ash/wm/panels/panel_layout_manager_unittest.cc View 1 chunk +6 lines, -7 lines 0 comments Download
M ash/wm/panels/panel_window_resizer.h View 1 chunk +2 lines, -2 lines 0 comments Download
M ash/wm/panels/panel_window_resizer.cc View 1 2 7 chunks +42 lines, -59 lines 0 comments Download
M ash/wm/toplevel_window_event_handler.cc View 7 chunks +11 lines, -9 lines 0 comments Download
M ash/wm/window_resizer.h View 2 chunks +0 lines, -10 lines 0 comments Download
M ash/wm/window_resizer.cc View 1 chunk +0 lines, -1 line 0 comments Download
M ash/wm/workspace/multi_window_resize_controller.cc View 2 chunks +2 lines, -2 lines 0 comments Download
M ash/wm/workspace_controller_unittest.cc View 1 chunk +2 lines, -3 lines 0 comments Download

Messages

Total messages: 9 (3 generated)
sky
4 years, 8 months ago (2016-04-25 15:25:24 UTC) #1
James Cook
For 5 years I have been waiting for the day I find a mistake in ...
4 years, 8 months ago (2016-04-25 17:16:05 UTC) #2
sky
https://codereview.chromium.org/1918883002/diff/60001/ash/wm/panels/panel_layout_manager.cc File ash/wm/panels/panel_layout_manager.cc (right): https://codereview.chromium.org/1918883002/diff/60001/ash/wm/panels/panel_layout_manager.cc#newcode188 ash/wm/panels/panel_layout_manager.cc:188: explicit PanelCalloutWidget(wm::WmWindow* container) : background_(NULL) { On 2016/04/25 17:16:05, ...
4 years, 8 months ago (2016-04-25 18:17:58 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1918883002/80001 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1918883002/80001
4 years, 8 months ago (2016-04-25 18:20:09 UTC) #6
commit-bot: I haz the power
Committed patchset #5 (id:80001)
4 years, 8 months ago (2016-04-25 19:40:46 UTC) #7
commit-bot: I haz the power
4 years, 8 months ago (2016-04-25 19:42:06 UTC) #9
Message was sent while issue was closed.
Patchset 5 (id:??) landed as
https://crrev.com/ad031502362573a9fe05c53c8f0776e5f7009dba
Cr-Commit-Position: refs/heads/master@{#389540}

Powered by Google App Engine
This is Rietveld 408576698