[wpt-sync] Sync PR 36426 - Reland "Report the time of a video's first displayed frame for LCP"
Categories
(Testing :: web-platform-tests, task, P4)
Tracking
(firefox108 fixed)
| Tracking | Status | |
|---|---|---|
| firefox108 | --- | fixed |
People
(Reporter: wpt-sync, Unassigned)
References
(Blocks 1 open bug, )
Details
(Whiteboard: [wptsync downstream])
Sync web-platform-tests PR 36426 into mozilla-central (this bug is closed when the sync is complete).
PR: https://github.com/web-platform-tests/wpt/pull/36426
Details from upstream follow.
Ian Clelland <iclelland@chromium.org> wrote:
Reland "Report the time of a video's first displayed frame for LCP"
This reverts commit d2c573466851694a43bc0ce067b4f4ad51aa1408.
Reason for revert: Relanding with the flaky part of the test disabled
Original change's description:
Revert "Report the time of a video's first displayed frame for LCP"
This reverts commit ba1da51a29d331b40e90cbed08c2a379e96dacc1.
Reason for revert: Test is failing on a number of builders.
Bug: 1373981
Owners-Override: Dana Fried \<dfried@google.com>Original change's description:
Report the time of a video's first displayed frame for LCP
This CL plumbs the information needed to capture LCP from videos
into Video timing information for LCP into the HTMLVideoElement, from
where it is sent to the PaintTimingDetector when the initial frame is
displayed.Video byte size is taken from the WebMediaPlayer's PipelineStatistics,
while the result of the Timing-Allow-Origin check goes through URLIndex,
in the same manner as CORS checks.Bug: 1289664
Change-Id: I9e8ed63a2913d5dc70b0f1b2bf9f928db4f1b908
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3814106
Commit-Queue: Ian Clelland \<iclelland@chromium.org>
Reviewed-by: Yoav Weiss \<yoavweiss@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1058088}Bug: 1289664
Change-Id: I1be173c71d5e39c333ca61a9c8c9b73f28b598af
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3949734
Commit-Queue: Dana Fried \<dfried@google.com>
Bot-Commit: Rubber Stamper \<rubber-stamper@appspot.gserviceaccount.com>
Owners-Override: Dana Fried \<dfried@google.com>
Reviewed-by: Ian Clelland \<iclelland@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1058179}Bug: 1373981
Bug: 1289664
Change-Id: Id67dbf2aa99a45b7ec008b273c3a40335f0f1e92Reviewed-on: https://chromium-review.googlesource.com/3950373
WPT-Export-Revision: 41f0db28ff1273d95879e6073b06bc645324f8b9
|
Assignee | |
Comment 1•2 years ago
|
||
|
|
Assignee | |
Comment 2•2 years ago
|
||
|
|
Assignee | |
Comment 3•2 years ago
|
||
CI Results
Ran 10 Firefox configurations based on mozilla-central, and Firefox, and Chrome on GitHub CI
Total 33 tests and 1 subtests
Status Summary
Firefox
OK : 29[Gecko-android-em-7.0-x86_64-qr-debug-isolated-process-geckoview] 31[Gecko-android-em-7.0-x86_64-lite-qr-opt-geckoview, Gecko-android-em-7.0-x86_64-qr-debug-geckoview, Gecko-android-em-7.0-x86_64-qr-opt-geckoview, Gecko-linux1804-64-qr-debug, Gecko-linux1804-64-qr-opt, Gecko-windows10-32-2004-qr-debug, Gecko-windows10-32-2004-qr-opt, Gecko-windows10-64-2004-qr-debug, Gecko-windows10-64-2004-qr-opt, GitHub]
CRASH : 1
FAIL : 30[Gecko-android-em-7.0-x86_64-qr-debug-isolated-process-geckoview] 31[Gecko-android-em-7.0-x86_64-lite-qr-opt-geckoview, Gecko-android-em-7.0-x86_64-qr-debug-geckoview, Gecko-android-em-7.0-x86_64-qr-opt-geckoview, Gecko-linux1804-64-qr-debug, Gecko-linux1804-64-qr-opt, Gecko-windows10-32-2004-qr-debug, Gecko-windows10-32-2004-qr-opt, Gecko-windows10-64-2004-qr-debug, Gecko-windows10-64-2004-qr-opt, GitHub]
TIMEOUT: 1
ERROR : 2
Chrome
OK : 32
PASS : 31
FAIL : 10
TIMEOUT: 2
Links
Gecko CI (Treeherder)
GitHub PR Head
GitHub PR Base
Details
Crashes
- /largest-contentful-paint/video-poster.html [wpt.fyi]:
CRASH[Gecko-android-em-7.0-x86_64-qr-debug-isolated-process-geckoview],OK[Gecko-android-em-7.0-x86_64-lite-qr-opt-geckoview,Gecko-android-em-7.0-x86_64-qr-debug-geckoview,Gecko-android-em-7.0-x86_64-qr-opt-geckoview,Gecko-linux1804-64-qr-debug,Gecko-linux1804-64-qr-opt,Gecko-windows10-32-2004-qr-debug,Gecko-windows10-32-2004-qr-opt,Gecko-windows10-64-2004-qr-debug,Gecko-windows10-64-2004-qr-opt,GitHub]
New Tests That Don't Pass
- /largest-contentful-paint/animated/observe-animated-image-gif.tentative.html [wpt.fyi]
- Same origin animated image is observable and has a first frame.:
FAIL(Chrome:FAIL)
- Same origin animated image is observable and has a first frame.:
- /largest-contentful-paint/animated/observe-animated-image-webp.tentative.html [wpt.fyi]
- Same origin animated image is observable and has a first frame.:
FAIL(Chrome:FAIL)
- Same origin animated image is observable and has a first frame.:
- /largest-contentful-paint/animated/observe-animated-image.tentative.html [wpt.fyi]
- Same origin animated image is observable and has a first frame.:
FAIL(Chrome:FAIL)
- Same origin animated image is observable and has a first frame.:
- /largest-contentful-paint/animated/observe-cross-origin-animated-image.tentative.html [wpt.fyi]
- Same origin animated image is observable and has a first frame.:
FAIL(Chrome:FAIL)
- Same origin animated image is observable and has a first frame.:
- /largest-contentful-paint/animated/observe-cross-origin-tao-animated-image.tentative.html [wpt.fyi]
- Same origin animated image is observable and has a first frame.:
FAIL(Chrome:FAIL)
- Same origin animated image is observable and has a first frame.:
- /largest-contentful-paint/animated/observe-non-animated-image.tentative.html [wpt.fyi]
- Same origin animated image is observable and has a first frame.:
FAIL(Chrome:FAIL)
- Same origin animated image is observable and has a first frame.:
- /largest-contentful-paint/animated/observe-video.tentative.html [wpt.fyi]
- Same origin animated image is observable and has a first frame.:
FAIL(Chrome:TIMEOUT)
- Same origin animated image is observable and has a first frame.:
- /largest-contentful-paint/contracted-image.html [wpt.fyi]
- Largest Contentful Paint: |size| attribute is bounded by display size.:
FAIL(Chrome:PASS)
- Largest Contentful Paint: |size| attribute is bounded by display size.:
- /largest-contentful-paint/cross-origin-image.sub.html [wpt.fyi]
- Cross-origin image is observable, with renderTime equal to 0.:
FAIL(Chrome:PASS)
- Cross-origin image is observable, with renderTime equal to 0.:
- /largest-contentful-paint/expanded-image.html [wpt.fyi]
- Largest Contentful Paint: |size| attribute is bounded by intrinsic size.:
FAIL(Chrome:PASS)
- Largest Contentful Paint: |size| attribute is bounded by intrinsic size.:
- /largest-contentful-paint/first-letter-background.html [wpt.fyi]
- Largest Contentful Paint: first-letter is observable.:
FAIL(Chrome:PASS)
- Largest Contentful Paint: first-letter is observable.:
- /largest-contentful-paint/image-TAO.sub.html [wpt.fyi]
- Cross-origin elements with valid TAO have correct renderTime, with invalid TAO have renderTime set to 0.:
FAIL(Chrome:PASS)
- Cross-origin elements with valid TAO have correct renderTime, with invalid TAO have renderTime set to 0.:
- /largest-contentful-paint/image-full-viewport.html [wpt.fyi]
- The intersectionRect of an img element overflowing is computed correctly:
FAIL(Chrome:PASS)
- The intersectionRect of an img element overflowing is computed correctly:
- /largest-contentful-paint/image-inside-svg.html [wpt.fyi]
- Image inside SVG is observable.:
FAIL(Chrome:PASS)
- Image inside SVG is observable.:
- /largest-contentful-paint/image-not-fully-visible.html [wpt.fyi]
- The intersectionRect of an img element overflowing is computed correctly:
FAIL(Chrome:PASS)
- The intersectionRect of an img element overflowing is computed correctly:
- /largest-contentful-paint/image-removed-before-load.html [wpt.fyi]
- Largest Contentful Paint: image removed before loaded does not produce entry.:
FAIL(Chrome:PASS)
- Largest Contentful Paint: image removed before loaded does not produce entry.:
- /largest-contentful-paint/image-src-change.html [wpt.fyi]
- Largest Contentful Paint: changing src causes a new entry to be dispatched.:
FAIL(Chrome:PASS)
- Largest Contentful Paint: changing src causes a new entry to be dispatched.:
- /largest-contentful-paint/image-sw-same-origin.https.html [wpt.fyi]:
ERROR(Chrome:OK) - /largest-contentful-paint/image-upscaling.html [wpt.fyi]:
ERROR(Chrome:OK) - /largest-contentful-paint/larger-image.html [wpt.fyi]
- Largest Contentful Paint: largest image is reported.:
FAIL(Chrome:PASS)
- Largest Contentful Paint: largest image is reported.:
- /largest-contentful-paint/loadTime-after-appendChild.html [wpt.fyi]
- Image loadTime occurs after appendChild is called.:
FAIL(Chrome:PASS)
- Image loadTime occurs after appendChild is called.:
- /largest-contentful-paint/multiple-redirects-TAO.html [wpt.fyi]
- Cross-origin images with passing/failing TAO should/shouldn't have its renderTime set.:
FAIL(Chrome:PASS)
- Cross-origin images with passing/failing TAO should/shouldn't have its renderTime set.:
- /largest-contentful-paint/observe-after-untrusted-scroll.html [wpt.fyi]
- Same-origin image after a JS initiated scroll event is observable.:
FAIL(Chrome:PASS)
- Same-origin image after a JS initiated scroll event is observable.:
- /largest-contentful-paint/observe-image.html [wpt.fyi]
- Same-origin image is observable.:
FAIL(Chrome:PASS)
- Same-origin image is observable.:
- /largest-contentful-paint/observe-svg-background-image.html [wpt.fyi]
- Same-origin SVG background image is observable.:
FAIL(Chrome:PASS)
- Same-origin SVG background image is observable.:
- /largest-contentful-paint/observe-svg-data-uri-background-image.html [wpt.fyi]
- Data-URI background SVG image is observable.:
FAIL(Chrome:PASS)
- Data-URI background SVG image is observable.:
- /largest-contentful-paint/observe-svg-data-uri-image.html [wpt.fyi]
- Same-origin image is observable.:
FAIL(Chrome:PASS)
- Same-origin image is observable.:
- /largest-contentful-paint/observe-svg-image.html [wpt.fyi]
- Same-origin image is observable.:
FAIL(Chrome:PASS)
- Same-origin image is observable.:
- /largest-contentful-paint/placeholder-image.html [wpt.fyi]:
OK[Gecko-android-em-7.0-x86_64-lite-qr-opt-geckoview,Gecko-android-em-7.0-x86_64-qr-debug-geckoview,Gecko-android-em-7.0-x86_64-qr-opt-geckoview,Gecko-linux1804-64-qr-debug,Gecko-linux1804-64-qr-opt,Gecko-windows10-32-2004-qr-debug,Gecko-windows10-32-2004-qr-opt,Gecko-windows10-64-2004-qr-debug,Gecko-windows10-64-2004-qr-opt,GitHub],TIMEOUT[Gecko-android-em-7.0-x86_64-qr-debug-isolated-process-geckoview] (Chrome:OK)- Largest Contentful Paint: changing src causes a new entry to be dispatched.:
FAIL(Chrome:PASS)
- Largest Contentful Paint: changing src causes a new entry to be dispatched.:
- /largest-contentful-paint/redirects-tao-star.html [wpt.fyi]
- Cross-origin image without TAO should not have its renderTime set, with full TAO it should.:
FAIL(Chrome:PASS)
- Cross-origin image without TAO should not have its renderTime set, with full TAO it should.:
- /largest-contentful-paint/repeated-image.html [wpt.fyi]
- Repeated image produces different timestamps.:
FAIL(Chrome:PASS)
- Repeated image produces different timestamps.:
- /largest-contentful-paint/same-origin-redirects.html [wpt.fyi]
- Same-origin image redirect without TAO should have its renderTime set.:
FAIL(Chrome:PASS)
- Same-origin image redirect without TAO should have its renderTime set.:
- /largest-contentful-paint/video-poster.html [wpt.fyi]:
CRASH[Gecko-android-em-7.0-x86_64-qr-debug-isolated-process-geckoview],OK[Gecko-android-em-7.0-x86_64-lite-qr-opt-geckoview,Gecko-android-em-7.0-x86_64-qr-debug-geckoview,Gecko-android-em-7.0-x86_64-qr-opt-geckoview,Gecko-linux1804-64-qr-debug,Gecko-linux1804-64-qr-opt,Gecko-windows10-32-2004-qr-debug,Gecko-windows10-32-2004-qr-opt,Gecko-windows10-64-2004-qr-debug,Gecko-windows10-64-2004-qr-opt,GitHub] (Chrome:OK)- Able to observe a video's poster image.:
FAIL(Chrome:PASS)
- Able to observe a video's poster image.:
|
||
Comment 5•2 years ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/6c58f092954a
https://hg.mozilla.org/mozilla-central/rev/bb562de6abab
Description
•