Bug 96110 – [chromium] gradient-with-scaled-ancestor.html rebaseline

• Status: RESOLVED
• Resolution: FIXED
• Component: Tools / Tests

Fixed https://bugs.webkit.org/show_bug.cgi?id=95847 to properly set the device scale factor.

Need to rebaseline.

Created attachment 162774 [details]
Patch

[Adrienne Walker]

Why are there cross-platform results here? Is it just the scrollbars? If so, can you just add a body { overflow: hidden } property on the test so you just need one result?

(In reply to comment #2)
> Why are there cross-platform results here? Is it just the scrollbars? If so, can you just add a body { overflow: hidden } property on the test so you just need one result?

Looking at garden-o-matic, Windows version has a difference from mac in the gradient itself, but Linux and Mac only differ by scroll bars.

Hidden overflow sounds like a reasonable change, but it's not my test, so we'll probably need to run it by the author since it'll break the mac baseline as well. I can do it if you think it's worth pursuing.

[Adrienne Walker]

Comment on attachment 162774 [details]
Patch

(In reply to comment #3)
> (In reply to comment #2)
> > Why are there cross-platform results here? Is it just the scrollbars? If so, can you just add a body { overflow: hidden } property on the test so you just need one result?
> 
> Looking at garden-o-matic, Windows version has a difference from mac in the gradient itself, but Linux and Mac only differ by scroll bars.
> 
> Hidden overflow sounds like a reasonable change, but it's not my test, so we'll probably need to run it by the author since it'll break the mac baseline as well. I can do it if you think it's worth pursuing.

Yuck.  I'm confused why Windows has a difference in the gradient, since they should both be using Skia.

At any rate, I'll R+ this patch so it can land.  No need to hold up getting a test rebaselined.

[WebKit Review Bot]

Comment on attachment 162774 [details]
Patch

Clearing flags on attachment: 162774

Committed r128090: <http://trac.webkit.org/changeset/128090>

[WebKit Review Bot]

All reviewed patches have been landed.  Closing bug.

[Adam Klein]

We're seeing a weird release/debug difference on Win: http://test-results.appspot.com/dashboards/flakiness_dashboard.html#tests=fast%2Fhidpi%2Fgradient-with-scaled-ancestor.html

The difference seems to be entirely in the scrollbars: in debug, the background is green, while it's black in release. Is this some known issue? As it is right now, there's no expected result that'll satisfy both the debug and release builds (I caused the release to pass and debug to fail in http://trac.webkit.org/changeset/128216)

(In reply to comment #7)
> We're seeing a weird release/debug difference on Win: http://test-results.appspot.com/dashboards/flakiness_dashboard.html#tests=fast%2Fhidpi%2Fgradient-with-scaled-ancestor.html
> 
> The difference seems to be entirely in the scrollbars: in debug, the background is green, while it's black in release. Is this some known issue? As it is right now, there's no expected result that'll satisfy both the debug and release builds (I caused the release to pass and debug to fail in http://trac.webkit.org/changeset/128216)

It's not a known issue as far as I am aware. My rebaseline only dealt with the size of the image, not the contents.