Move Windows DPI Code from ui/gfx to ui/display

Move Windows DPI Code from ui/gfx to ui/display

BUG=597761
Committed: https://crrev.com/18e220e823d94c65a507bc5edadf5623bf1601d0
Cr-Commit-Position: refs/heads/master@{#388224}

[robliao, 2016-04-16 00:08:44]

The CQ bit was checked by robliao@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-04-16 00:09:22]

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

[commit-bot: I haz the power, 2016-04-16 00:17:30]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-04-16 00:17:31]

Dry run: Try jobs failed on following builders:
  win8_chromium_ng on tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win8_chromium_ng/...)
  win_clang on tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_clang/builds/...)

[robliao, 2016-04-18 19:31:54]

Patchset #1 (id:1) has been deleted

[robliao, 2016-04-18 19:32:02]

Patchset #1 (id:20001) has been deleted

[robliao, 2016-04-18 19:32:08]

The CQ bit was checked by robliao@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-04-18 19:32:49]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1889423002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1889423002/40001

[commit-bot: I haz the power, 2016-04-18 19:42:15]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-04-18 19:42:16]

Dry run: Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[robliao, 2016-04-18 20:10:25]

The CQ bit was checked by robliao@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-04-18 20:11:13]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1889423002/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1889423002/60001

[commit-bot: I haz the power, 2016-04-18 20:21:13]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-04-18 20:21:14]

Dry run: Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[robliao, 2016-04-18 21:43:01]

Patchset #1 (id:40001) has been deleted

[robliao, 2016-04-18 21:43:35]

robliao@chromium.org changed reviewers:
+ oshima@chromium.org

[robliao, 2016-04-18 21:43:36]

oshima: Please review this CL. Thanks!

[oshima, 2016-04-18 23:26:45]

lgtm

https://codereview.chromium.org/1889423002/diff/60001/content/renderer/render...
File content/renderer/renderer_main_platform_delegate_win.cc (right):

https://codereview.chromium.org/1889423002/diff/60001/content/renderer/render...
content/renderer/renderer_main_platform_delegate_win.cc:78:
blink::WebFontRendering::setDeviceScaleFactor(display::win::GetDPIScale());
Can you add TODO to eliminate this? This should be set via WebScreenInfo

[robliao, 2016-04-18 23:40:20]

The CQ bit was checked by robliao@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-04-18 23:40:42]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1889423002/80001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1889423002/80001

[robliao, 2016-04-18 23:41:27]

robliao@chromium.org changed reviewers:
+ piman@chromium.org, sky@chromium.org

[robliao, 2016-04-18 23:41:28]

OWNERs, please review the path patterns under your username. Thanks!

piman:
content/*

sky:
chrome/*
ui/*

https://codereview.chromium.org/1889423002/diff/60001/content/renderer/render...
File content/renderer/renderer_main_platform_delegate_win.cc (right):

https://codereview.chromium.org/1889423002/diff/60001/content/renderer/render...
content/renderer/renderer_main_platform_delegate_win.cc:78:
blink::WebFontRendering::setDeviceScaleFactor(display::win::GetDPIScale());
On 2016/04/18 23:26:44, oshima wrote:
> Can you add TODO to eliminate this? This should be set via WebScreenInfo

Done.

[piman, 2016-04-19 00:33:58]

lgtm

[commit-bot: I haz the power, 2016-04-19 00:49:00]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-04-19 00:49:01]

Dry run: This issue passed the CQ dry run.

[sky, 2016-04-19 13:14:58]

LGTM

[robliao, 2016-04-19 16:40:49]

The CQ bit was checked by robliao@chromium.org

[robliao, 2016-04-19 16:40:50]

The patchset sent to the CQ was uploaded after l-g-t-m from oshima@chromium.org
Link to the patchset: https://codereview.chromium.org/1889423002/#ps80001
(title: "Add a TODO")

[commit-bot: I haz the power, 2016-04-19 16:41:10]

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

[commit-bot: I haz the power, 2016-04-19 16:47:23]

Committed patchset #2 (id:80001)

[commit-bot: I haz the power, 2016-04-22 19:12:53]

Description was changed from

==========
Move Windows DPI Code from ui/gfx to ui/display

BUG=597761
==========

to

==========
Move Windows DPI Code from ui/gfx to ui/display

BUG=597761

Committed: https://crrev.com/18e220e823d94c65a507bc5edadf5623bf1601d0
Cr-Commit-Position: refs/heads/master@{#388224}
==========

[commit-bot: I haz the power, 2016-04-22 19:12:55]

Patchset 2 (id:??) landed as
https://crrev.com/18e220e823d94c65a507bc5edadf5623bf1601d0
Cr-Commit-Position: refs/heads/master@{#388224}