interop icon indicating copy to clipboard operation
interop copied to clipboard

Consider removing css/css-view-transitions/no-painting-while-render-blocked.html

Open vmpstr opened this issue 8 months ago • 3 comments

Test List

https://wpt.fyi/results/css/css-view-transitions/no-painting-while-render-blocked.html

Rationale

This test uses a takeScreenshot within a callback while the page is not producing frames. I don't believe this is a valid way to test what is currently on the screen, since takeScreenshot AFAIK removes the reftest-wait and relies on a frame being produced (which is the very thing being tested not happening).

From the looks of it, all browsers are failing this test.

I propose removing it.

/cc @mattwoodrow @nt1m

vmpstr avatar Mar 20 '25 16:03 vmpstr

Can the test be modified to match the actual behavior, or is there really nothing useful that can be tested while the page is not producing frames?

foolip avatar Mar 20 '25 16:03 foolip

I think there is a fundamental limitation to what we can test here. In Chromium we do have tests that verify that content isn't presented/updated while we're render blocked, but they are not WPTs or web tests, but rather a variant of unittest/browsertest.

I don't know of a way to query "what is currently on screen" without actually producing a frame in a WPT environment.

vmpstr avatar Mar 20 '25 16:03 vmpstr

I have the same initial reaction as @foolip, but I'd be fine dropping if there's no way to fix these tests.

nt1m avatar Mar 24 '25 15:03 nt1m

CC @BorisChiou

jgraham avatar Apr 03 '25 16:04 jgraham

I'm OK to drop this if nothing we can do to fix this test, given that the current test doesn't use a valid way to verify this feature.

(Note that Gecko is still working on rendering suppression, but yes I agree with this so it's fine to me to drop it.)

BorisChiou avatar Apr 03 '25 16:04 BorisChiou