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

Issue 2931963002: [M60] exo: Fix cursor scale when crossing displays (Closed)

Created:
3 years, 6 months ago by Dominik Laskowski
Modified:
3 years, 6 months ago
Reviewers:
reveman
CC:
chromium-reviews
Target Ref:
refs/branch-heads/3112
Project:
chromium
Visibility:
Public.

Description

[M60] exo: Fix cursor scale when crossing displays While dragging, the cursor was locked, so CursorClient::SetCursor calls were collapsed into a single call at the end of the drag. When crossing displays with different densities, the cursor scale was incorrect until the window was dropped. This CL removes cursor locking to prevent the OnCursorDisplayChanged update from being postponed. TBR=reveman@chromium.org NOTRY=true NOPRESUBMIT=true BUG=631136 BUG=642894 TEST=caroline: Cursor scale is correct while dragging across displays. Review-Url: https://codereview.chromium.org/2812663002 Cr-Original-Commit-Position: refs/heads/master@{#476375} Review-Url: https://codereview.chromium.org/2931963002 Cr-Commit-Position: refs/branch-heads/3112@{#265} Cr-Branched-From: b6460e24cf59f429d69de255538d0fc7a425ccf9-refs/heads/master@{#474897} Committed: https://chromium.googlesource.com/chromium/src/+/f9f0ccb6c9db27cea6caca0c6e7e323e0994e710

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -5 lines) Patch
M components/exo/shell_surface.cc View 2 chunks +1 line, -5 lines 0 comments Download

Messages

Total messages: 5 (3 generated)
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/2931963002/1
3 years, 6 months ago (2017-06-08 20:46:20 UTC) #2
commit-bot: I haz the power
3 years, 6 months ago (2017-06-08 20:47:00 UTC) #5
Message was sent while issue was closed.
Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/f9f0ccb6c9db27cea6caca0c6e7e...

Powered by Google App Engine
This is Rietveld 408576698