color: Add NSScreenColorSpaceDidChangeNotification observer

color: Add NSScreenColorSpaceDidChangeNotification observer

Change RenderWidgetHostViewMac to not query the display color profile
directly, rather, use the display::Display screen information.

This exposed a bug whereby we weren't updating the display::Display
color space when NSScreens changed color spaces.

Clean up RenderWidgetHostViewMac to make UpdateBackingStoreProperties
call UpdateScreenInfo and update color space information, and change
calls to UpdateScreenInfo to go through UpdateBackingStoreProperties
appropriately.

Also make BrowserCompositorMac query the current display scale factor
and color space from the display::Display when activating.

BUG=735613
Review-Url: https://codereview.chromium.org/2959873002
Cr-Commit-Position: refs/heads/master@{#482518}
Committed: https://chromium.googlesource.com/chromium/src/+/942e74b0d721b0090cd4b30f8a31f04adf005fff

[ccameron, 2017-06-26 22:43:31]

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

[commit-bot: I haz the power, 2017-06-26 22:43:54]

Dry run: CQ is trying da patch.

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

[ccameron, 2017-06-26 22:45:01]

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

[ccameron, 2017-06-26 22:45:02]

ptal -- more cleanup leaning more and more heavily on display::Display

https://codereview.chromium.org/2959873002/diff/1/content/browser/renderer_ho...
File content/browser/renderer_host/browser_compositor_view_mac.mm (left):

https://codereview.chromium.org/2959873002/diff/1/content/browser/renderer_ho...
content/browser/renderer_host/browser_compositor_view_mac.mm:359: float
scale_factor = ui::GetScaleFactorForNativeView(ns_view);
At the moment, ui::GetScaleFactorForNativeView wraps
display::Screen::GetScreen()->GetDisplayNearestView on all platforms but Mac. I
plan to make this call uniform on all platforms.

[hubbe, 2017-06-26 22:51:42]

This is a little outside my expertise, but I don't so anything actually wrong
with it, so: LGTM

[commit-bot: I haz the power, 2017-06-27 00:57:17]

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

[commit-bot: I haz the power, 2017-06-27 00:57:18]

Dry run: This issue passed the CQ dry run.

[ccameron, 2017-06-27 02:23:46]

The CQ bit was checked by ccameron@chromium.org

[commit-bot: I haz the power, 2017-06-27 02:23:57]

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, 2017-06-27 02:27:18]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1498530226482930, "parent_rev":
"74dcbb2b8a9caffd350a44bf8c8fc51887792ce4", "commit_rev":
"942e74b0d721b0090cd4b30f8a31f04adf005fff"}

[commit-bot: I haz the power, 2017-06-27 02:27:30]

Description was changed from

==========
color: Add NSScreenColorSpaceDidChangeNotification observer

Change RenderWidgetHostViewMac to not query the display color profile
directly, rather, use the display::Display screen information.

This exposed a bug whereby we weren't updating the display::Display
color space when NSScreens changed color spaces.

Clean up RenderWidgetHostViewMac to make UpdateBackingStoreProperties
call UpdateScreenInfo and update color space information, and change
calls to UpdateScreenInfo to go through UpdateBackingStoreProperties
appropriately.

Also make BrowserCompositorMac query the current display scale factor
and color space from the display::Display when activating.

BUG=735613
==========

to

==========
color: Add NSScreenColorSpaceDidChangeNotification observer

Change RenderWidgetHostViewMac to not query the display color profile
directly, rather, use the display::Display screen information.

This exposed a bug whereby we weren't updating the display::Display
color space when NSScreens changed color spaces.

Clean up RenderWidgetHostViewMac to make UpdateBackingStoreProperties
call UpdateScreenInfo and update color space information, and change
calls to UpdateScreenInfo to go through UpdateBackingStoreProperties
appropriately.

Also make BrowserCompositorMac query the current display scale factor
and color space from the display::Display when activating.

BUG=735613

Review-Url: https://codereview.chromium.org/2959873002
Cr-Commit-Position: refs/heads/master@{#482518}
Committed:
https://chromium.googlesource.com/chromium/src/+/942e74b0d721b0090cd4b30f8a31...
==========

[commit-bot: I haz the power, 2017-06-27 02:27:31]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/942e74b0d721b0090cd4b30f8a31...