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

Issue 2657673005: [M57] exo: Fix cursor size for multi-display ARC (Closed)

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

Description

[M57] exo: Fix cursor size for multi-display ARC When a low-DPI monitor is connected to a high-DPI device, the cursor in ARC windows appears twice as large if: 1) The primary display is the external display, unless the internal display uses native resolution and the window is on the internal display. This includes docked mode. 2) The primary display is the internal display, the internal display uses native resolution, and the window is on the external display. TBR=reveman@chromium.org NOTRY=true NOPRESUBMIT=true BUG=684672 BUG=642894 TEST=samus: Cursor size is correct in the above cases. Review-Url: https://codereview.chromium.org/2645273005 Cr-Commit-Position: refs/heads/master@{#446192} (cherry picked from commit f79ede40658ecab4a6c72b6e51c456d68dab2ad0) Review-Url: https://codereview.chromium.org/2657673005 Cr-Commit-Position: refs/branch-heads/2987@{#110} Cr-Branched-From: ad51088c0e8776e8dcd963dbe752c4035ba6dab6-refs/heads/master@{#444943} Committed: https://chromium.googlesource.com/chromium/src/+/7c15664f05e4ad7bd99a426fa54eee096e6e777b

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+24 lines, -6 lines) Patch
M components/exo/pointer.cc View 1 chunk +24 lines, -6 lines 0 comments Download

Messages

Total messages: 15 (9 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/2657673005/1
3 years, 11 months ago (2017-01-26 17:45:42 UTC) #2
commit-bot: I haz the power
No L-G-T-M from a valid reviewer yet. CQ run can only be started by full ...
3 years, 11 months ago (2017-01-26 17:45:44 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/2657673005/1
3 years, 11 months ago (2017-01-26 18:02:46 UTC) #7
commit-bot: I haz the power
CLs for remote refs other than refs/heads/master must contain NOTRY=true and NOPRESUBMIT=true in order for ...
3 years, 11 months ago (2017-01-26 18:02:47 UTC) #9
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/2657673005/1
3 years, 11 months ago (2017-01-26 18:04:39 UTC) #12
commit-bot: I haz the power
3 years, 11 months ago (2017-01-26 18:06:39 UTC) #15
Message was sent while issue was closed.
Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/7c15664f05e4ad7bd99a426fa54e...

Powered by Google App Engine
This is Rietveld 408576698