Enable Per-Monitor DPI Unless Explicitly Disabled

Enable Per-Monitor DPI Unless Explicitly Disabled

                   Per Monitor DPI Settings
Flags              Before             After
No Flags           Disabled           Enabled
Enable             Enabled            Enabled
Enable + Disabled  Disabled           Disabled
Disabled           Disalbed           Disabled

BUG=426656
Committed: https://crrev.com/23364eecc5d1bb1770f8c22a7c0a320154e3eac7
Cr-Commit-Position: refs/heads/master@{#407508}

[robliao, 2016-07-22 21:48:20]

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

[robliao, 2016-07-22 21:48:49]

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

[robliao, 2016-07-22 21:48:50]

Preparing to throw the switch on Monday.

[commit-bot: I haz the power, 2016-07-22 21:49:01]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Lei Zhang, 2016-07-22 22:21:42]

lgtm

[commit-bot: I haz the power, 2016-07-22 22:34:55]

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

[commit-bot: I haz the power, 2016-07-22 22:34:55]

Dry run: This issue passed the CQ dry run.

[robliao, 2016-07-25 17:07:44]

The CQ bit was checked by robliao@chromium.org

[commit-bot: I haz the power, 2016-07-25 17:08:07]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-25 17:51:22]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-07-25 17:56:11]

Description was changed from

==========
Enable Per-Monitor DPI Unless Explicitly Disabled

                   Per Monitor DPI Settings
Flags              Before             After
No Flags           Disabled           Enabled
Enable             Enabled            Enabled
Enable + Disabled  Disabled           Disabled
Disabled           Disalbed           Disabled

BUG=426656
==========

to

==========
Enable Per-Monitor DPI Unless Explicitly Disabled

                   Per Monitor DPI Settings
Flags              Before             After
No Flags           Disabled           Enabled
Enable             Enabled            Enabled
Enable + Disabled  Disabled           Disabled
Disabled           Disalbed           Disabled

BUG=426656

Committed: https://crrev.com/23364eecc5d1bb1770f8c22a7c0a320154e3eac7
Cr-Commit-Position: refs/heads/master@{#407508}
==========

[commit-bot: I haz the power, 2016-07-25 17:56:12]

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

[Peter Kasting, 2016-07-27 01:58:21]

This broke --force-device-scale-factor=, which is now ignored.

Is that an expected result, and if so, what is the intended workaround? 
Otherwise, seems like specifying this flag should override per-monitor DPI just
like it used to override global DPI.

[Peter Kasting, 2016-07-27 02:00:13]

On 2016/07/27 01:58:21, Peter Kasting wrote:
> Is that an expected result, and if so, what is the intended workaround? 

(By this I don't mean "pass --disable-per-monitor-dpi" as clearly that will go
away eventually.)

[robliao, 2016-07-27 02:12:08]

On 2016/07/27 02:00:13, Peter Kasting wrote:
> On 2016/07/27 01:58:21, Peter Kasting wrote:
> > Is that an expected result, and if so, what is the intended workaround? 
> 
> (By this I don't mean "pass --disable-per-monitor-dpi" as clearly that will go
> away eventually.)

Yup. And that's fixed as of https://codereview.chromium.org/2176183005/