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

Issue 2924173003: [M60] exo: Send DSF of internal display to ARC (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: Send DSF of internal display to ARC Since the client-side DPI in ARC is configured at build time based on the internal display, the client must scale the cursor by the DSF of the internal (not the primary) display to convert from pixels to DIPs. TBR=reveman@chromium.org NOTRY=true NOPRESUBMIT=true BUG=642894 TEST=caroline: Cursor scale is correct for each display/resolution. Review-Url: https://codereview.chromium.org/2779823002 Cr-Original-Commit-Position: refs/heads/master@{#476423} Review-Url: https://codereview.chromium.org/2924173003 Cr-Commit-Position: refs/branch-heads/3112@{#266} Cr-Branched-From: b6460e24cf59f429d69de255538d0fc7a425ccf9-refs/heads/master@{#474897} Committed: https://chromium.googlesource.com/chromium/src/+/93dbb873288016afde40cd102c58b734071ea8da

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+177 lines, -165 lines) Patch
M components/exo/wayland/server.cc View 2 chunks +6 lines, -1 line 0 comments Download
M third_party/wayland-protocols/include/protocol/remote-shell-unstable-v1-client-protocol.h View 7 chunks +70 lines, -67 lines 0 comments Download
M third_party/wayland-protocols/include/protocol/remote-shell-unstable-v1-server-protocol.h View 3 chunks +95 lines, -93 lines 0 comments Download
M third_party/wayland-protocols/protocol/remote-shell-protocol.c View 2 chunks +5 lines, -4 lines 0 comments Download
M third_party/wayland-protocols/unstable/remote-shell/remote-shell-unstable-v1.xml View 1 chunk +1 line, -0 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/2924173003/1
3 years, 6 months ago (2017-06-08 20:51:39 UTC) #2
commit-bot: I haz the power
3 years, 6 months ago (2017-06-08 20:52:16 UTC) #5
Message was sent while issue was closed.
Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/93dbb873288016afde40cd102c58...

Powered by Google App Engine
This is Rietveld 408576698