Make elevation icon size correct in different DPI settings

Make elevation icon size correct in different DPI settings

BUG=449731
Committed: https://crrev.com/5a36bfe017b8c4408b8816246e07abcec96e0781
Cr-Commit-Position: refs/heads/master@{#312458}

[xiaoling, 2015-01-21 19:56:19]

xiaolingbao@chromium.org changed reviewers:
+ pkasting@chromium.org

[Peter Kasting, 2015-01-21 20:17:46]

LGTM

[Peter Kasting, 2015-01-21 20:18:37]

https://codereview.chromium.org/849543003/diff/20001/chrome/browser/ui/views/...
File chrome/browser/ui/views/elevation_icon_setter.cc (right):

https://codereview.chromium.org/849543003/diff/20001/chrome/browser/ui/views/...
chrome/browser/ui/views/elevation_icon_setter.cc:77: device_scale_factor =
gfx::GetDPIScale();
Actually you might want a comment here like:

Windows gives us back a correctly-scaled image for the current DPI, so mark this
image as having been scaled for the current DPI already.

[xiaoling, 2015-01-21 20:23:35]

ty

https://codereview.chromium.org/849543003/diff/20001/chrome/browser/ui/views/...
File chrome/browser/ui/views/elevation_icon_setter.cc (right):

https://codereview.chromium.org/849543003/diff/20001/chrome/browser/ui/views/...
chrome/browser/ui/views/elevation_icon_setter.cc:77: device_scale_factor =
gfx::GetDPIScale();
On 2015/01/21 20:18:37, Peter Kasting wrote:
> Actually you might want a comment here like:
> 
> Windows gives us back a correctly-scaled image for the current DPI, so mark
this
> image as having been scaled for the current DPI already.

Done.

[xiaoling, 2015-01-21 20:23:44]

The CQ bit was checked by xiaolingbao@chromium.org

[commit-bot: I haz the power, 2015-01-21 20:25:51]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/849543003/40001

[commit-bot: I haz the power, 2015-01-21 21:29:38]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-01-21 21:30:28]

Patchset 3 (id:??) landed as
https://crrev.com/5a36bfe017b8c4408b8816246e07abcec96e0781
Cr-Commit-Position: refs/heads/master@{#312458}