[caret-color] Styling caret-color with optional second color value. [chromium/src : main]

1 view
Skip to first unread message

Ziran Sun (Gerrit)

unread,
May 5, 2026, 8:17:23 AM (10 days ago) May 5
to Kent Tamura, Manuel Rego Casasnovas, Chromium LUCI CQ, Menard, Alexis, chromium...@chromium.org, Olga Gerchikov, apavlo...@chromium.org, blink-revie...@chromium.org, blink-re...@chromium.org, blink-rev...@chromium.org, blink-revie...@chromium.org, blink-...@chromium.org, jmedle...@chromium.org, kinuko...@chromium.org
Attention needed from Kent Tamura and Manuel Rego Casasnovas

Ziran Sun voted Commit-Queue+1

Commit-Queue+1
Open in Gerrit

Related details

Attention is currently required from:
  • Kent Tamura
  • Manuel Rego Casasnovas
Submit Requirements:
  • requirement satisfiedCode-Coverage
  • requirement is not satisfiedCode-Owners
  • requirement is not satisfiedCode-Review
  • requirement is not satisfiedReview-Enforcement
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
Gerrit-MessageType: comment
Gerrit-Project: chromium/src
Gerrit-Branch: main
Gerrit-Change-Id: I8c2422ea0b59e5be18a3c728830c842aaa7ae7b0
Gerrit-Change-Number: 7816635
Gerrit-PatchSet: 1
Gerrit-Owner: Ziran Sun <zs...@igalia.com>
Gerrit-Reviewer: Kent Tamura <tk...@chromium.org>
Gerrit-Reviewer: Manuel Rego Casasnovas <re...@igalia.com>
Gerrit-Reviewer: Ziran Sun <zs...@igalia.com>
Gerrit-CC: Menard, Alexis <alexis...@intel.com>
Gerrit-CC: Olga Gerchikov <gerc...@microsoft.com>
Gerrit-Attention: Manuel Rego Casasnovas <re...@igalia.com>
Gerrit-Attention: Kent Tamura <tk...@chromium.org>
Gerrit-Comment-Date: Tue, 05 May 2026 12:17:06 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
satisfied_requirement
unsatisfied_requirement
open
diffy

Kent Tamura (Gerrit)

unread,
May 6, 2026, 7:08:50 PM (8 days ago) May 6
to Ziran Sun, Kent Tamura, Manuel Rego Casasnovas, Chromium LUCI CQ, Menard, Alexis, chromium...@chromium.org, Olga Gerchikov, apavlo...@chromium.org, blink-revie...@chromium.org, blink-re...@chromium.org, blink-rev...@chromium.org, blink-revie...@chromium.org, blink-...@chromium.org, jmedle...@chromium.org, kinuko...@chromium.org
Attention needed from Manuel Rego Casasnovas and Ziran Sun

Kent Tamura voted Code-Review+1

Code-Review+1
Open in Gerrit

Related details

Attention is currently required from:
  • Manuel Rego Casasnovas
  • Ziran Sun
Submit Requirements:
  • requirement satisfiedCode-Coverage
  • requirement satisfiedCode-Owners
  • requirement satisfiedCode-Review
  • requirement satisfiedReview-Enforcement
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
Gerrit-MessageType: comment
Gerrit-Project: chromium/src
Gerrit-Branch: main
Gerrit-Change-Id: I8c2422ea0b59e5be18a3c728830c842aaa7ae7b0
Gerrit-Change-Number: 7816635
Gerrit-PatchSet: 3
Gerrit-Owner: Ziran Sun <zs...@igalia.com>
Gerrit-Reviewer: Kent Tamura <tk...@chromium.org>
Gerrit-Reviewer: Manuel Rego Casasnovas <re...@igalia.com>
Gerrit-Reviewer: Ziran Sun <zs...@igalia.com>
Gerrit-CC: Menard, Alexis <alexis...@intel.com>
Gerrit-CC: Olga Gerchikov <gerc...@microsoft.com>
Gerrit-Attention: Manuel Rego Casasnovas <re...@igalia.com>
Gerrit-Attention: Ziran Sun <zs...@igalia.com>
Gerrit-Comment-Date: Wed, 06 May 2026 23:08:15 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
satisfied_requirement
open
diffy

Blink W3C Test Autoroller (Gerrit)

unread,
May 6, 2026, 7:14:27 PM (8 days ago) May 6
to Ziran Sun, Kent Tamura, Manuel Rego Casasnovas, Chromium LUCI CQ, Menard, Alexis, chromium...@chromium.org, Olga Gerchikov, apavlo...@chromium.org, blink-revie...@chromium.org, blink-re...@chromium.org, blink-rev...@chromium.org, blink-revie...@chromium.org, blink-...@chromium.org, jmedle...@chromium.org, kinuko...@chromium.org
Attention needed from Manuel Rego Casasnovas and Ziran Sun

Message from Blink W3C Test Autoroller

Exportable changes to web-platform-tests were detected in this CL and a pull request in the upstream repo has been made: https://github.com/web-platform-tests/wpt/pull/59715.

When this CL lands, the bot will automatically merge the PR on GitHub if the required GitHub checks pass; otherwise, ecosystem-infra@ team will triage the failures and may contact you.

WPT Export docs:
https://chromium.googlesource.com/chromium/src/+/main/docs/testing/web_platform_tests.md#Automatic-export-process

Open in Gerrit

Related details

Attention is currently required from:
  • Manuel Rego Casasnovas
  • Ziran Sun
Submit Requirements:
  • requirement satisfiedCode-Coverage
  • requirement satisfiedCode-Owners
  • requirement satisfiedCode-Review
  • requirement satisfiedReview-Enforcement
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
Gerrit-MessageType: comment
Gerrit-Project: chromium/src
Gerrit-Branch: main
Gerrit-Change-Id: I8c2422ea0b59e5be18a3c728830c842aaa7ae7b0
Gerrit-Change-Number: 7816635
Gerrit-PatchSet: 3
Gerrit-Owner: Ziran Sun <zs...@igalia.com>
Gerrit-Reviewer: Kent Tamura <tk...@chromium.org>
Gerrit-Reviewer: Manuel Rego Casasnovas <re...@igalia.com>
Gerrit-Reviewer: Ziran Sun <zs...@igalia.com>
Gerrit-CC: Blink W3C Test Autoroller <blink-w3c-te...@chromium.org>
Gerrit-CC: Menard, Alexis <alexis...@intel.com>
Gerrit-CC: Olga Gerchikov <gerc...@microsoft.com>
Gerrit-Attention: Manuel Rego Casasnovas <re...@igalia.com>
Gerrit-Attention: Ziran Sun <zs...@igalia.com>
Gerrit-Comment-Date: Wed, 06 May 2026 23:14:14 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
satisfied_requirement
open
diffy

Ziran Sun (Gerrit)

unread,
May 7, 2026, 4:00:28 AM (8 days ago) May 7
to Blink W3C Test Autoroller, Kent Tamura, Manuel Rego Casasnovas, Chromium LUCI CQ, Menard, Alexis, chromium...@chromium.org, Olga Gerchikov, apavlo...@chromium.org, blink-revie...@chromium.org, blink-re...@chromium.org, blink-rev...@chromium.org, blink-revie...@chromium.org, blink-...@chromium.org, jmedle...@chromium.org, kinuko...@chromium.org
Attention needed from Manuel Rego Casasnovas

Ziran Sun voted Commit-Queue+2

Commit-Queue+2
Open in Gerrit

Related details

Attention is currently required from:
  • Manuel Rego Casasnovas
Submit Requirements:
  • requirement satisfiedCode-Coverage
  • requirement satisfiedCode-Owners
  • requirement satisfiedCode-Review
  • requirement satisfiedReview-Enforcement
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
Gerrit-MessageType: comment
Gerrit-Project: chromium/src
Gerrit-Branch: main
Gerrit-Change-Id: I8c2422ea0b59e5be18a3c728830c842aaa7ae7b0
Gerrit-Change-Number: 7816635
Gerrit-PatchSet: 3
Gerrit-Owner: Ziran Sun <zs...@igalia.com>
Gerrit-Reviewer: Kent Tamura <tk...@chromium.org>
Gerrit-Reviewer: Manuel Rego Casasnovas <re...@igalia.com>
Gerrit-Reviewer: Ziran Sun <zs...@igalia.com>
Gerrit-CC: Blink W3C Test Autoroller <blink-w3c-te...@chromium.org>
Gerrit-CC: Menard, Alexis <alexis...@intel.com>
Gerrit-CC: Olga Gerchikov <gerc...@microsoft.com>
Gerrit-Attention: Manuel Rego Casasnovas <re...@igalia.com>
Gerrit-Comment-Date: Thu, 07 May 2026 08:00:11 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
satisfied_requirement
open
diffy

Chromium LUCI CQ (Gerrit)

unread,
May 7, 2026, 4:20:51 AM (8 days ago) May 7
to Ziran Sun, Blink W3C Test Autoroller, Kent Tamura, Manuel Rego Casasnovas, Menard, Alexis, chromium...@chromium.org, Olga Gerchikov, apavlo...@chromium.org, blink-revie...@chromium.org, blink-re...@chromium.org, blink-rev...@chromium.org, blink-revie...@chromium.org, blink-...@chromium.org, jmedle...@chromium.org, kinuko...@chromium.org

Chromium LUCI CQ submitted the change

Change information

Commit message:
[caret-color] Styling caret-color with optional second color value.

A recent spec change at [1] that introduces a second color to
caret-color in order to allow to set the text color when caret-shape
is block. This CL adds the the styling support.

[1] https://drafts.csswg.org/css-ui/#caret-color/
Bug: 498556169
Change-Id: I8c2422ea0b59e5be18a3c728830c842aaa7ae7b0
Reviewed-by: Kent Tamura <tk...@chromium.org>
Commit-Queue: Ziran Sun <zs...@igalia.com>
Cr-Commit-Position: refs/heads/main@{#1626780}
Files:
  • M third_party/blink/renderer/build/scripts/core/style/make_computed_style_base.py
  • M third_party/blink/renderer/core/animation/color_property_functions.cc
  • M third_party/blink/renderer/core/css/build.gni
  • M third_party/blink/renderer/core/css/css_properties.json5
  • M third_party/blink/renderer/core/css/properties/longhands/longhands_custom.cc
  • M third_party/blink/renderer/core/css/resolver/style_builder_converter.cc
  • M third_party/blink/renderer/core/css/resolver/style_builder_converter.h
  • A third_party/blink/renderer/core/css/style_caret_color.h
  • M third_party/blink/renderer/core/style/computed_style.cc
  • M third_party/blink/renderer/core/style/computed_style.h
  • M third_party/blink/renderer/platform/runtime_enabled_features.json5
  • M third_party/blink/web_tests/external/wpt/css/css-ui/parsing/caret-color-computed.html
  • M third_party/blink/web_tests/external/wpt/css/css-ui/parsing/caret-color-invalid.html
  • M third_party/blink/web_tests/external/wpt/css/css-ui/parsing/caret-color-valid.html
Change size: L
Delta: 14 files changed, 230 insertions(+), 30 deletions(-)
Branch: refs/heads/main
Submit Requirements:
  • requirement satisfiedCode-Review: +1 by Kent Tamura
Open in Gerrit
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
Gerrit-MessageType: merged
Gerrit-Project: chromium/src
Gerrit-Branch: main
Gerrit-Change-Id: I8c2422ea0b59e5be18a3c728830c842aaa7ae7b0
Gerrit-Change-Number: 7816635
Gerrit-PatchSet: 4
Gerrit-Owner: Ziran Sun <zs...@igalia.com>
Gerrit-Reviewer: Chromium LUCI CQ <chromiu...@luci-project-accounts.iam.gserviceaccount.com>
Gerrit-Reviewer: Kent Tamura <tk...@chromium.org>
Gerrit-Reviewer: Manuel Rego Casasnovas <re...@igalia.com>
Gerrit-Reviewer: Ziran Sun <zs...@igalia.com>
Gerrit-CC: Blink W3C Test Autoroller <blink-w3c-te...@chromium.org>
open
diffy
satisfied_requirement

Blink W3C Test Autoroller (Gerrit)

unread,
May 7, 2026, 5:19:59 AM (8 days ago) May 7
to Chromium LUCI CQ, Ziran Sun, Kent Tamura, Manuel Rego Casasnovas, Menard, Alexis, chromium...@chromium.org, Olga Gerchikov, apavlo...@chromium.org, blink-revie...@chromium.org, blink-re...@chromium.org, blink-rev...@chromium.org, blink-revie...@chromium.org, blink-...@chromium.org, jmedle...@chromium.org, kinuko...@chromium.org

Message from Blink W3C Test Autoroller

The WPT PR for this CL has been merged upstream! https://github.com/web-platform-tests/wpt/pull/59715

Open in Gerrit

Related details

Attention set is empty
Submit Requirements:
  • requirement satisfiedCode-Coverage
  • requirement satisfiedCode-Owners
  • requirement satisfiedCode-Review
  • requirement satisfiedReview-Enforcement
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
Gerrit-MessageType: comment
Gerrit-Project: chromium/src
Gerrit-Branch: main
Gerrit-Change-Id: I8c2422ea0b59e5be18a3c728830c842aaa7ae7b0
Gerrit-Change-Number: 7816635
Gerrit-PatchSet: 4
Gerrit-Owner: Ziran Sun <zs...@igalia.com>
Gerrit-Reviewer: Chromium LUCI CQ <chromiu...@luci-project-accounts.iam.gserviceaccount.com>
Gerrit-Reviewer: Kent Tamura <tk...@chromium.org>
Gerrit-Reviewer: Manuel Rego Casasnovas <re...@igalia.com>
Gerrit-Reviewer: Ziran Sun <zs...@igalia.com>
Gerrit-CC: Blink W3C Test Autoroller <blink-w3c-te...@chromium.org>
Gerrit-CC: Menard, Alexis <alexis...@intel.com>
Gerrit-CC: Olga Gerchikov <gerc...@microsoft.com>
Gerrit-Comment-Date: Thu, 07 May 2026 09:19:54 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
satisfied_requirement
open
diffy

luci-bisection@appspot.gserviceaccount.com (Gerrit)

unread,
May 7, 2026, 12:46:07 PM (8 days ago) May 7
to Chromium LUCI CQ, Ziran Sun, Blink W3C Test Autoroller, Kent Tamura, Manuel Rego Casasnovas, Menard, Alexis, chromium...@chromium.org, Olga Gerchikov, apavlo...@chromium.org, blink-revie...@chromium.org, blink-re...@chromium.org, blink-rev...@chromium.org, blink-revie...@chromium.org, blink-...@chromium.org, jmedle...@chromium.org, kinuko...@chromium.org

Message from luci-bi...@appspot.gserviceaccount.com

LUCI Bisection has identified this change as the cause of a test failure. See the analysis: https://ci.chromium.org/ui/p/chromium/bisection/test-analysis/b/5412722112987136

Sample build with failed test: https://ci.chromium.org/b/8682473116001001697
Affected test(s):
[://\:chrome_wpt_tests!webtest::external/wpt/css/css-ui/parsing#caret-color-computed.html](https://ci.chromium.org/ui/test/chromium/:%2F%2F%5C:chrome_wpt_tests%21webtest::external%2Fwpt%2Fcss%2Fcss-ui%2Fparsing%23caret-color-computed.html?q=VHash%3A6129bda16b001a29)
A revert for this change was not created because the builder that this CL broke is not watched by gardeners, therefore less important. You can consider revert this CL, fix forward or let builder owners resolve it themselves.

If this is a false positive, please report it at http://b.corp.google.com/createIssue?component=1199205&description=Analysis%3A+https%3A%2F%2Fci.chromium.org%2Fui%2Fp%2Fchromium%2Fbisection%2Ftest-analysis%2Fb%2F5412722112987136&format=PLAIN&priority=P3&title=Wrongly+blamed+https%3A%2F%2Fchromium-review.googlesource.com%2Fc%2Fchromium%2Fsrc%2F%2B%2F7816635&type=BUG

Gerrit-Comment-Date: Thu, 07 May 2026 16:45:40 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
satisfied_requirement
open
diffy
Reply all
Reply to author
Forward
0 new messages