Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implement forced-color-adjust CSS property #17188

Merged
merged 1 commit into from Jun 11, 2019

Conversation

chromium-wpt-export-bot
Copy link
Collaborator

@chromium-wpt-export-bot chromium-wpt-export-bot commented Jun 5, 2019

This change adds support for the forced-color-adjust CSS property and
updates the update_use_counter_css.py script to adjust enums.xml based
on UseCounterHelper::MapCSSPropertyIdToCSSSampleIdForHistogram rather
than the old location in UseCounter.

Spec:
https://www.w3.org/TR/css-color-adjust-1/#forced-color-adjust-prop

Bug: 970285
Change-Id: I74759517a4d8be092b3b75301baca8a707c22ce7
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1644498
Commit-Queue: Alison Maher <almaher@microsoft.com>
Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
Reviewed-by: Steven Holte <holte@chromium.org>
Reviewed-by: Anders Hartvoll Ruud <andruud@chromium.org>
Reviewed-by: Kevin Babbitt <kbabbitt@microsoft.com>
Cr-Commit-Position: refs/heads/master@{#667852}

Copy link
Collaborator

@wpt-pr-bot wpt-pr-bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Already reviewed downstream.

@chromium-wpt-export-bot chromium-wpt-export-bot force-pushed the chromium-export-cl-1644498 branch 4 times, most recently from f5f0259 to 6fba5a6 Compare June 11, 2019 00:54
This change adds support for the forced-color-adjust CSS property and
updates the update_use_counter_css.py script to adjust enums.xml based
on UseCounterHelper::MapCSSPropertyIdToCSSSampleIdForHistogram rather
than the old location in UseCounter.

Spec:
https://www.w3.org/TR/css-color-adjust-1/#forced-color-adjust-prop

Bug: 970285
Change-Id: I74759517a4d8be092b3b75301baca8a707c22ce7
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1644498
Commit-Queue: Alison Maher <almaher@microsoft.com>
Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
Reviewed-by: Steven Holte <holte@chromium.org>
Reviewed-by: Anders Hartvoll Ruud <andruud@chromium.org>
Reviewed-by: Kevin Babbitt <kbabbitt@microsoft.com>
Cr-Commit-Position: refs/heads/master@{#667852}
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants