} else {
const url = window.location.origin + '/images/black-rectangle.png';
checkImage(entry, url, 'target', 0, beforeLoad, ['sizeLowerBound']);
}
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.
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Code-Review | +1 |
Somewhat of a stamp-- glad this is being improved, but the test likely could be remade better as well.
assert_greater_than_equal(entry.renderTime, beforeLoad2, "blaaa");
"blaaa" "bleee" lol
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Looks like this is timing out -- will wait for bots to go green to review
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
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
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |