Improve the first-letter-background LCP test. [chromium/src : main]

0 views
Skip to first unread message

Johannes Henkel (Gerrit)

unread,
Jun 17, 2025, 7:01:58 PMJun 17
to Scott Haseley, Michal Mocny, AyeAye, speed-metrics...@chromium.org, blink-...@chromium.org, blink-revie...@chromium.org, lighthouse-eng-extern...@google.com
Attention needed from Michal Mocny and Scott Haseley

Johannes Henkel added 1 comment

File third_party/blink/web_tests/external/wpt/largest-contentful-paint/first-letter-background.html
Line 38, Patchset 1 (Parent): } else {
const url = window.location.origin + '/images/black-rectangle.png';
checkImage(entry, url, 'target', 0, beforeLoad, ['sizeLowerBound']);
}
Johannes Henkel . resolved

In the old version of this test, there's no guarantee that the part of the test that checks for the image is ever executed - it can pass without that. Which is a bit sad since this is also the title and purpose of the test.

Open in Gerrit

Related details

Attention is currently required from:
  • Michal Mocny
  • Scott Haseley
Submit Requirements:
  • requirement satisfiedCode-Coverage
  • requirement is not satisfiedCode-Review
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: Id5ee1e1fd2ef386178d4c287b56464f8ac4e242f
Gerrit-Change-Number: 6651302
Gerrit-PatchSet: 1
Gerrit-Owner: Johannes Henkel <joha...@chromium.org>
Gerrit-Reviewer: Michal Mocny <mmo...@chromium.org>
Gerrit-Reviewer: Scott Haseley <shas...@chromium.org>
Gerrit-Attention: Michal Mocny <mmo...@chromium.org>
Gerrit-Attention: Scott Haseley <shas...@chromium.org>
Gerrit-Comment-Date: Tue, 17 Jun 2025 23:01:47 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
satisfied_requirement
unsatisfied_requirement
open
diffy

Michal Mocny (Gerrit)

unread,
Jun 18, 2025, 7:43:30 AMJun 18
to Johannes Henkel, Chromium LUCI CQ, Scott Haseley, AyeAye, speed-metrics...@chromium.org, blink-...@chromium.org, blink-revie...@chromium.org, lighthouse-eng-extern...@google.com
Attention needed from Johannes Henkel and Scott Haseley

Michal Mocny voted and added 2 comments

Votes added by Michal Mocny

Code-Review+1

2 comments

Patchset-level comments
File-level comment, Patchset 1 (Latest):
Michal Mocny . resolved

Somewhat of a stamp-- glad this is being improved, but the test likely could be remade better as well.

File third_party/blink/web_tests/external/wpt/largest-contentful-paint/first-letter-background.html
Line 57, Patchset 1 (Latest): assert_greater_than_equal(entry.renderTime, beforeLoad2, "blaaa");
Michal Mocny . resolved

"blaaa" "bleee" lol

Open in Gerrit

Related details

Attention is currently required from:
  • Johannes Henkel
  • Scott Haseley
Submit Requirements:
  • requirement satisfiedCode-Coverage
  • requirement satisfiedCode-Review
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: Id5ee1e1fd2ef386178d4c287b56464f8ac4e242f
Gerrit-Change-Number: 6651302
Gerrit-PatchSet: 1
Gerrit-Owner: Johannes Henkel <joha...@chromium.org>
Gerrit-Reviewer: Johannes Henkel <joha...@chromium.org>
Gerrit-Reviewer: Michal Mocny <mmo...@chromium.org>
Gerrit-Reviewer: Scott Haseley <shas...@chromium.org>
Gerrit-Attention: Scott Haseley <shas...@chromium.org>
Gerrit-Attention: Johannes Henkel <joha...@chromium.org>
Gerrit-Comment-Date: Wed, 18 Jun 2025 11:43:16 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
satisfied_requirement
open
diffy

Scott Haseley (Gerrit)

unread,
Jun 20, 2025, 12:07:51 PMJun 20
to Johannes Henkel, Michal Mocny, Chromium LUCI CQ, AyeAye, speed-metrics...@chromium.org, blink-...@chromium.org, blink-revie...@chromium.org, lighthouse-eng-extern...@google.com
Attention needed from Johannes Henkel

Scott Haseley added 1 comment

Patchset-level comments
Scott Haseley . resolved

Looks like this is timing out -- will wait for bots to go green to review

Open in Gerrit

Related details

Attention is currently required from:
  • Johannes Henkel
Submit Requirements:
  • requirement satisfiedCode-Coverage
  • requirement satisfiedCode-Review
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: Id5ee1e1fd2ef386178d4c287b56464f8ac4e242f
Gerrit-Change-Number: 6651302
Gerrit-PatchSet: 1
Gerrit-Owner: Johannes Henkel <joha...@chromium.org>
Gerrit-Reviewer: Johannes Henkel <joha...@chromium.org>
Gerrit-Reviewer: Michal Mocny <mmo...@chromium.org>
Gerrit-Reviewer: Scott Haseley <shas...@chromium.org>
Gerrit-Attention: Johannes Henkel <joha...@chromium.org>
Gerrit-Comment-Date: Fri, 20 Jun 2025 16:07:39 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
satisfied_requirement
open
diffy

Blink W3C Test Autoroller (Gerrit)

unread,
Jun 25, 2025, 4:14:47 PMJun 25
to Johannes Henkel, Michal Mocny, Chromium LUCI CQ, Scott Haseley, AyeAye, speed-metrics...@chromium.org, blink-...@chromium.org, blink-revie...@chromium.org, lighthouse-eng-extern...@google.com
Attention needed from Johannes Henkel

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/53401.

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:
  • Johannes Henkel
Submit Requirements:
  • requirement satisfiedCode-Coverage
  • requirement satisfiedCode-Owners
  • requirement satisfiedCode-Review
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: Id5ee1e1fd2ef386178d4c287b56464f8ac4e242f
Gerrit-Change-Number: 6651302
Gerrit-PatchSet: 2
Gerrit-Owner: Johannes Henkel <joha...@chromium.org>
Gerrit-Reviewer: Johannes Henkel <joha...@chromium.org>
Gerrit-Reviewer: Michal Mocny <mmo...@chromium.org>
Gerrit-Reviewer: Scott Haseley <shas...@chromium.org>
Gerrit-CC: Blink W3C Test Autoroller <blink-w3c-te...@chromium.org>
Gerrit-Attention: Johannes Henkel <joha...@chromium.org>
Gerrit-Comment-Date: Wed, 25 Jun 2025 20:14:40 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
satisfied_requirement
open
diffy
Reply all
Reply to author
Forward
0 new messages