Skip to content

Remove incorrect getComputedStyle() test for ::first-line#60419

Merged
chromium-wpt-export-bot merged 1 commit into
masterfrom
chromium-export-3f1e0af710
Jun 5, 2026
Merged

Remove incorrect getComputedStyle() test for ::first-line#60419
chromium-wpt-export-bot merged 1 commit into
masterfrom
chromium-export-3f1e0af710

Conversation

@chromium-wpt-export-bot
Copy link
Copy Markdown
Collaborator

Only a few properties apply to ::first-line, but that does not mean the
computed value is affected. The test incorrectly assumed the computed
value of vertical-align for ::first-line is reset to its initial value.

Notified the author here:

WebKit/WebKit#48454 (comment)

Bug: 503879985
Change-Id: I1f37b490a0d4eef452c89242d0e962a7b834e6a8
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7900679
Commit-Queue: Rune Lillesveen <futhark@chromium.org>
Reviewed-by: Steinar H Gunderson <sesse@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1642181}

Only a few properties apply to ::first-line, but that does not mean the
computed value is affected. The test incorrectly assumed the computed
value of vertical-align for ::first-line is reset to its initial value.

Notified the author here:

WebKit/WebKit#48454 (comment)

Bug: 503879985
Change-Id: I1f37b490a0d4eef452c89242d0e962a7b834e6a8
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7900679
Commit-Queue: Rune Lillesveen <futhark@chromium.org>
Reviewed-by: Steinar H Gunderson <sesse@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1642181}
Copy link
Copy Markdown
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.

The review process for this patch is being conducted in the Chromium project.

@chromium-wpt-export-bot chromium-wpt-export-bot merged commit 9de8a7a into master Jun 5, 2026
9 checks passed
@chromium-wpt-export-bot chromium-wpt-export-bot deleted the chromium-export-3f1e0af710 branch June 5, 2026 09:23
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.

3 participants