Bug 99441

Summary: Layout Test svg/zoom/page/zoom-img-preserveAspectRatio-support-1.html etc. is failing
Product: WebKit Reporter: Shinya Kawanaka <shinyak>
Component: Tools / TestsAssignee: Nobody <webkit-unassigned>
Status: REOPENED ---    
Severity: Normal CC: abarth, danakj, dpranke, eae, leviw
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   

Description Shinya Kawanaka 2012-10-16 02:07:47 PDT
The following layout tests are failing on Win.

svg/zoom/page/zoom-img-preserveAspectRatio-support-1.html
svg/zoom/page/zoom-replaced-intrinsic-ratio-001.htm
svg/zoom/page/zoom-svg-through-object-with-absolute-size-2.xhtml
svg/zoom/page/zoom-svg-through-object-with-absolute-size.xhtml
svg/zoom/page/zoom-svg-through-object-with-percentage-size.xhtml
tables/mozilla/bugs/bug73321.html
Comment 1 Ojan Vafai 2012-10-16 11:45:48 PDT
shinyak: If you can, in the future, add a link to where the tests started failing, that's helpful in getting these resolved.

These are forgotten rebaselines after: http://trac.webkit.org/changeset/131358.
Comment 2 Dana Jansens 2012-10-16 11:49:02 PDT
Yes garden-o-matic refused to rebaseline these for me yesterday. I'll try again today.
Comment 3 Ojan Vafai 2012-10-16 11:51:33 PDT
OIC. This is because the win7 release bot is having trouble running. I wonder if webkit-patch should try to detect this case and fallback to the debug bot? Maybe that's crazy-talk.
Comment 4 Dirk Pranke 2012-10-16 12:01:09 PDT
(In reply to comment #3)
> OIC. This is because the win7 release bot is having trouble running. I wonder if webkit-patch should try to detect this case and fallback to the debug bot? Maybe that's crazy-talk.

Yeah, crazy talk :).
Comment 5 Emil A Eklund 2012-10-16 12:09:23 PDT
What would be _really_ useful is a way of getting results before landing a change.

We have the cbuild bots and ews but yet the only way to get platform specific results is to either manually run the tests and collect the results on all platforms or commit without results and update the expected results after the fact.

Not only would this make it easier to keep the tree green it would also greatly reduce the number of commits and thus improve cycle times for the cbuild. Not to mention allow one to see all test changes during the review process.

It would require a way to side-slip a change onto the bots though and a way to gather the results.
Comment 6 Ojan Vafai 2012-10-16 12:21:40 PDT
Getting results off the EWS bots is only blocked on having someone taking the time to do the legwork. Everyone agrees that would be better.
Comment 7 Emil A Eklund 2012-10-16 12:23:52 PDT
It would also require being able to request try jobs without uploading a patch (either to the ews or cbuild bots) and if we use the ews bots then we'll need to add one for each supported platform.

Not a trivial amount of work sadly.
Comment 8 Ojan Vafai 2012-10-16 13:15:14 PDT
(In reply to comment #7)
> It would also require being able to request try jobs without uploading a patch (either to the ews or cbuild bots) and if we use the ews bots then we'll need to add one for each supported platform.

I think just being able to 1) easily view the new results + diffs for each EWS bot and 2) choose some to get merged into your local patch, or the patch on bugzilla would be sufficient.

I don't think uploading a patch without a bug is a blocker here. If you can only do this for patches uploaded to bugzilla, that will cover 99% of patches. Getting EWS bots running tests for each platform will naturally happen once we have this. It's in each port's interest to setup such a bot to keep their trees green with minimal maintenance work.
Comment 9 Dana Jansens 2012-10-16 17:13:02 PDT
Committed r131531: <http://trac.webkit.org/changeset/131531>
Comment 10 Dana Jansens 2012-10-16 17:17:15 PDT
garden-o-matic finally let met rebaseline these. the CL only has 4 in it for some reason though it claims to have rebaselined 7.

There might be some further rebaselining to do still.