I am trying to add a stylo reftest like so: == text-indent-1a.html text-indent-1a.html where the file is: <!DOCTYPE html> <iframe sandbox style="width: 400px" srcdoc="<canvas style='text-indent: 50vw; display: block'>Text</canvas>"></iframe> This test fails in an opt build (in the sense that the stylo and gecko rendering do not match) but passes in a debug build. The failing case looks like "50vw" computed to "0" when loading with stylo. Loading this testcase outside the reftest harness shows the right thing. So it's possible we're getting the correct viewport size to the subframe in an async fashion or something and don't actually have the right style/layout onload....
2 months ago
Ryan, you've been poking at viewport units. Can you look at this?
Emilio says he still wants this one. ;-)
Seems to also be making layout/reftests/w3c-css/received/css-values-3/vh-support-atviewport.html fail. That test fails even when the @viewport is removed, so it's basically the 100vw / 100vh computing to zero.
I can't repro Comment 3 now (I could last week though), and the test case in comment 1 seems to be working AFAICT (it may be the case that the test-case was modified before landing). Manish, can you confirm that layout/reftests/w3c-css/received/css-values-3/vh-support-atviewport.html shows the right thing without the @vieport rule?
Nope, it shows a small green square on a red background, even when I remove the @viewport. I'm on yesterday's tip.