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

Issue 2336883002: mash: Port context menu code to WmShell and WmRootWindowController. (Closed)

Created:
4 years, 3 months ago by msw
Modified:
4 years, 3 months ago
Reviewers:
sky
CC:
chromium-reviews, kalyank, sadrul
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

mash: Port context menu code to WmShell and WmRootWindowController. BUG=640693 TEST=Context menus show up for the wallpaper and shelf. R=sky@chromium.org Committed: https://crrev.com/20734f8bc75635b1cf93c7cd9a4333e207041623 Cr-Commit-Position: refs/heads/master@{#418263}

Patch Set 1 #

Patch Set 2 : Cleanup. #

Patch Set 3 : Sync and rebase. #

Patch Set 4 : Fix error. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+78 lines, -95 lines) Patch
M ash/aura/wm_shell_aura.h View 1 chunk +0 lines, -2 lines 0 comments Download
M ash/aura/wm_shell_aura.cc View 1 1 chunk +0 lines, -5 lines 0 comments Download
M ash/common/wm_root_window_controller.h View 1 2 4 chunks +22 lines, -0 lines 0 comments Download
M ash/common/wm_root_window_controller.cc View 1 2 3 4 chunks +39 lines, -0 lines 0 comments Download
M ash/common/wm_shell.h View 1 2 1 chunk +3 lines, -4 lines 0 comments Download
M ash/common/wm_shell.cc View 1 2 2 chunks +14 lines, -0 lines 0 comments Download
M ash/mus/bridge/wm_shell_mus.h View 1 chunk +0 lines, -2 lines 0 comments Download
M ash/mus/bridge/wm_shell_mus.cc View 1 chunk +0 lines, -6 lines 0 comments Download
M ash/root_window_controller.h View 1 2 5 chunks +0 lines, -17 lines 0 comments Download
M ash/root_window_controller.cc View 1 2 3 chunks +0 lines, -38 lines 0 comments Download
M ash/shell.h View 1 2 3 chunks +0 lines, -6 lines 0 comments Download
M ash/shell.cc View 1 2 1 chunk +0 lines, -15 lines 0 comments Download

Messages

Total messages: 22 (15 generated)
msw
Hey Scott, please take a look; thanks!
4 years, 3 months ago (2016-09-13 01:13:09 UTC) #3
sky
I realize you're just moving code, but ShowContextMenu on RootWindowController and Shell is a very ...
4 years, 3 months ago (2016-09-13 02:12:53 UTC) #12
msw
On 2016/09/13 02:12:53, sky wrote: > I realize you're just moving code, but ShowContextMenu on ...
4 years, 3 months ago (2016-09-13 06:03:27 UTC) #15
sky
ShowContextMenu it is then. LGTM
4 years, 3 months ago (2016-09-13 13:13:00 UTC) #16
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2336883002/60001
4 years, 3 months ago (2016-09-13 16:04:19 UTC) #18
commit-bot: I haz the power
Committed patchset #4 (id:60001)
4 years, 3 months ago (2016-09-13 16:09:15 UTC) #20
commit-bot: I haz the power
4 years, 3 months ago (2016-09-13 16:10:52 UTC) #22
Message was sent while issue was closed.
Patchset 4 (id:??) landed as
https://crrev.com/20734f8bc75635b1cf93c7cd9a4333e207041623
Cr-Commit-Position: refs/heads/master@{#418263}

Powered by Google App Engine
This is Rietveld 408576698