Paint scrollbars at device scale resolution.

Paint scrollbars at device scale resolution.

BUG=531257
Committed: https://crrev.com/201cd54baaf4f94a76e7d254d70034c946168986
Cr-Commit-Position: refs/heads/master@{#358464}

[chrishtr, 2015-11-06 23:10:29]

Description was changed from

==========
none

BUG=
==========

to

==========
Paint scrollbars at device scale resolution.

BUG=531257
==========

[chrishtr, 2015-11-06 23:11:09]

chrishtr@chromium.org changed reviewers:
+ ccameron@chromium.org

[ccameron, 2015-11-06 23:15:20]

lgtm

https://codereview.chromium.org/1430503007/diff/1/third_party/WebKit/Source/p...
File third_party/WebKit/Source/platform/exported/WebScrollbarThemePainter.cpp
(right):

https://codereview.chromium.org/1430503007/diff/1/third_party/WebKit/Source/p...
third_party/WebKit/Source/platform/exported/WebScrollbarThemePainter.cpp:59:
pictureBuilder.context().setDeviceScaleFactor(m_deviceScaleFactor);
I have a vague preference for rolling this into SkPictureBuilder ctor, but that
involves lots of churn (probably not worth the bother).

[chrishtr, 2015-11-06 23:22:06]

On 2015/11/06 at 23:15:20, ccameron wrote:
> lgtm
> 
>
https://codereview.chromium.org/1430503007/diff/1/third_party/WebKit/Source/p...
> File third_party/WebKit/Source/platform/exported/WebScrollbarThemePainter.cpp
(right):
> 
>
https://codereview.chromium.org/1430503007/diff/1/third_party/WebKit/Source/p...
> third_party/WebKit/Source/platform/exported/WebScrollbarThemePainter.cpp:59:
pictureBuilder.context().setDeviceScaleFactor(m_deviceScaleFactor);
> I have a vague preference for rolling this into SkPictureBuilder ctor, but
that involves lots of churn (probably not worth the bother).

Considered that, but didn't want to add a constructor for a one-off case.

[chrishtr, 2015-11-06 23:22:10]

The CQ bit was checked by chrishtr@chromium.org

[commit-bot: I haz the power, 2015-11-06 23:22:22]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1430503007/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1430503007/1

[commit-bot: I haz the power, 2015-11-07 00:16:46]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-11-07 00:17:51]

Patchset 1 (id:??) landed as
https://crrev.com/201cd54baaf4f94a76e7d254d70034c946168986
Cr-Commit-Position: refs/heads/master@{#358464}