commit 54b74ebd76fbb1f7166c0bc2770660436e5233fd
author Sam Davis Omekara <samomekarajr@microsoft.com> Thu Nov 30 19:21:45 2023
committer Chromium LUCI CQ <chromium-scoped@luci-project-accounts.iam.gserviceaccount.com> Thu Nov 30 19:21:45 2023
tree 6659441fdf04f263ffc123713af1ae2632ac14ef
parent d81a05f30b9429cfb8e5a7e8dcb33b8c7f676d1c

Reland "[PageColors] Integration of PageColors feature with NativeTheme"

This is a reland of commit da792e88b7ab6c96293af62d7ec62c7f6ecc51cf.
The issue was that we incorrectly assumed that
NativeTheme::ShouldUseDarkColors() would always return false initially.
However, on ChromeOS this function could sometimes return true.
To avoid this inconsistency, we explicitly set the desired value rather
than assuming anything about the native theme.

Original change's description:
> [PageColors] Integration of PageColors feature with NativeTheme
>
> This CL makes relevant state changes to the NativeTheme web instance
> when Page colors changes. This is done in the
> ColorSchemeNativeThemeObserver::OnNativeThemeUpdated method.
>
> In the ColorSchemeNativeThemeObserver::OnNativeThemeUpdated method,
> we preserves the current behavior if Page Colors is 'Off' and forced
> colors is false OR forced colors is true and Page Colors is
> 'High Contrast'. However, if that is not the case then:
>
> 1. Page Colors is 'Off' and forced colors is true, we override the
> state of forced colors for the native theme instance for web since the
> Page colors browser setting takes precedence over OS for web content.
> Here, we also watch for changes to the preferred color scheme and make
> sure we adjust the state appropriately.
> 2. Page Colors is one of the Contrast themes i.e. 'Dusk', 'Desert',
> 'Black', 'White'. Here, we set the forced colors state to true and
> other states appropriately for web content.
>
> This CL adds the color scheme observer as a native theme observer for
> Linux in order to receive the correct state changes from
> NativeThemeAura.
>
> Bug: 1231644
> Change-Id: I6792aadb9fead7a5d686c9cc0c67a29f986030ec
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4959835
> Reviewed-by: Peter Kasting <pkasting@chromium.org>
> Reviewed-by: Alison Maher <almaher@microsoft.com>
> Reviewed-by: Benjamin Beaudry <benjamin.beaudry@microsoft.com>
> Commit-Queue: Sam Davis Omekara <samomekarajr@microsoft.com>
> Cr-Commit-Position: refs/heads/main@{#1230166}

Bug: 1231644
Change-Id: I516065d7a3b33e516257afa25e11fda24f8fccdf
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5074715
Reviewed-by: Alison Maher <almaher@microsoft.com>
Reviewed-by: Peter Kasting <pkasting@chromium.org>
Reviewed-by: Benjamin Beaudry <benjamin.beaudry@microsoft.com>
Commit-Queue: Sam Davis Omekara <samomekarajr@microsoft.com>
Cr-Commit-Position: refs/heads/main@{#1231446}