Monitor profile tool, now with Windows support, too!

Monitor profile tool, now with Windows support, too!

Original Mac support by Matt.

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2095093002

Committed: https://skia.googlesource.com/skia/+/6373c955541f6ebbb075ee6b2a58daa22ea9cecf

[Brian Osman, 2016-07-12 18:50:00]

brianosman@google.com changed reviewers:
+ msarett@google.com, reed@google.com

[Brian Osman, 2016-07-12 18:50:06]

The CQ bit was checked by brianosman@google.com to run a CQ dry run

[commit-bot: I haz the power, 2016-07-12 18:50:23]

Dry run: 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-12 18:53:05]

The CQ bit was unchecked by commit-bot@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-07-12 18:53:06]

Dry run: Try jobs failed on following builders:
  Build-Ubuntu-GCC-x86_64-Release-Trybot on master.client.skia.compile
(JOB_FAILED,
http://build.chromium.org/p/client.skia.compile/builders/Build-Ubuntu-GCC-x86...)

[msarett, 2016-07-12 18:56:50]

https://codereview.chromium.org/2141573004/diff/1/tools/get_current_monitor_p...
File tools/get_current_monitor_profile.cpp (right):

https://codereview.chromium.org/2141573004/diff/1/tools/get_current_monitor_p...
tools/get_current_monitor_profile.cpp:23: SkFILEWStream file("monitor.icc");
Maybe make this monitor_0.icc for consistency?

https://codereview.chromium.org/2141573004/diff/1/tools/get_current_monitor_p...
tools/get_current_monitor_profile.cpp:33: for (int i = 0;
EnumDisplayDevices(NULL, i, &dd, 0); ++i) {
I think it's worth adding a comment that this is "better" than what chrome
currently does, since chrome only detects the primary monitor.

Maybe we'll want to eventually move this code into chrome.

[msarett, 2016-07-12 18:58:36]

FWIW, both Mac profiles I've looked at so far have been sRGB gamma, arbitrary
gamut.

The Win/Linux profiles I've looked at have been 2.2 gamma, arbitrary gamut
(unless we explicitly ask for sRGB).

[Brian Osman, 2016-07-12 19:03:49]

The CQ bit was checked by brianosman@google.com to run a CQ dry run

[commit-bot: I haz the power, 2016-07-12 19:03:57]

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

[Brian Osman, 2016-07-12 19:04:19]

Fixed compile error in unsupported platform code, addressed Matt's feedback.

[msarett, 2016-07-12 19:05:33]

lgtm

[commit-bot: I haz the power, 2016-07-12 19:17:51]

The CQ bit was unchecked by commit-bot@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-07-12 19:17:52]

Dry run: This issue passed the CQ dry run.

[Brian Osman, 2016-07-12 19:27:47]

The CQ bit was checked by brianosman@google.com

[commit-bot: I haz the power, 2016-07-12 19:27:53]

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-12 22:06:28]

Description was changed from

==========
Monitor profile tool, now with Windows support, too!

Original Mac support by Matt.

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2095093002
==========

to

==========
Monitor profile tool, now with Windows support, too!

Original Mac support by Matt.

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2095093002

Committed:
https://skia.googlesource.com/skia/+/6373c955541f6ebbb075ee6b2a58daa22ea9cecf
==========

[commit-bot: I haz the power, 2016-07-12 22:06:30]

Committed patchset #2 (id:20001) as
https://skia.googlesource.com/skia/+/6373c955541f6ebbb075ee6b2a58daa22ea9cecf

[commit-bot: I haz the power, 2016-07-12 22:06:32]

CQ bit was unchecked.