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

Issue 2541163006: Wire up CursorManager for DesktopWindowTreeHostMus (Closed)

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

Description

Wire up CursorManager for DesktopWindowTreeHostMus This change makes DesktopWindowTreeHostMus use almost the same cursor manager as NativeWidgetMus. BUG=660994 TEST=none R=erg@chromium.org Committed: https://crrev.com/f71b727b75294db523d5ed01b4b7cd0c57f4b9c3 Cr-Commit-Position: refs/heads/master@{#435779}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+121 lines, -18 lines) Patch
M ui/aura/mus/window_port_mus.cc View 1 chunk +3 lines, -0 lines 0 comments Download
M ui/views/mus/desktop_window_tree_host_mus.h View 3 chunks +7 lines, -0 lines 0 comments Download
M ui/views/mus/desktop_window_tree_host_mus.cc View 5 chunks +69 lines, -0 lines 0 comments Download
M ui/views/widget/desktop_aura/desktop_native_widget_aura.h View 1 chunk +3 lines, -0 lines 0 comments Download
M ui/views/widget/desktop_aura/desktop_native_widget_aura.cc View 2 chunks +24 lines, -18 lines 0 comments Download
M ui/views/widget/desktop_aura/desktop_window_tree_host.h View 1 chunk +5 lines, -0 lines 0 comments Download
M ui/views/widget/desktop_aura/desktop_window_tree_host_win.h View 1 chunk +1 line, -0 lines 0 comments Download
M ui/views/widget/desktop_aura/desktop_window_tree_host_win.cc View 1 chunk +4 lines, -0 lines 0 comments Download
M ui/views/widget/desktop_aura/desktop_window_tree_host_x11.h View 1 chunk +1 line, -0 lines 0 comments Download
M ui/views/widget/desktop_aura/desktop_window_tree_host_x11.cc View 1 chunk +4 lines, -0 lines 0 comments Download

Messages

Total messages: 11 (6 generated)
sky
4 years ago (2016-12-01 22:07:25 UTC) #1
Elliot Glaysher
lgtm
4 years ago (2016-12-01 22:34:11 UTC) #4
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/2541163006/1
4 years ago (2016-12-01 22:47:55 UTC) #7
commit-bot: I haz the power
Committed patchset #1 (id:1)
4 years ago (2016-12-01 23:59:38 UTC) #9
commit-bot: I haz the power
4 years ago (2016-12-02 00:03:22 UTC) #11
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/f71b727b75294db523d5ed01b4b7cd0c57f4b9c3
Cr-Commit-Position: refs/heads/master@{#435779}

Powered by Google App Engine
This is Rietveld 408576698