Bug 63151 - WebGL test failing on SL bots: fast/canvas/webgl/data-view-test.html
Summary: WebGL test failing on SL bots: fast/canvas/webgl/data-view-test.html
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebGL (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Jian Li
URL:
Keywords: MakingBotsRed
Depends on: 63011
Blocks:
  Show dependency treegraph
 
Reported: 2011-06-22 10:10 PDT by Matthew Delaney
Modified: 2011-06-29 14:29 PDT (History)
8 users (show)

See Also:


Attachments
Proposed Patch (133.49 KB, patch)
2011-06-23 17:16 PDT, Jian Li
kbr: review+
jianli: commit-queue-
Details | Formatted Diff | Diff
Archive of layout-test-results from ec2-cr-linux-02 (1.60 MB, application/zip)
2011-06-23 18:37 PDT, WebKit Review Bot
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Matthew Delaney 2011-06-22 10:10:17 PDT
This WebGL test recently started failing on the bots. I looked at it briefly, but realized that the test may need some looking at since the TypeError and SyntaxError messages don't seem to match up. I get the feeling that our expected results file is incorrect and/or the test may need updating to conform with WebIDL.
Comment 1 Matthew Delaney 2011-06-22 10:54:15 PDT
Added a platform/mac expected results file here to keep tree from burning: http://trac.webkit.org/changeset/89450
Comment 2 Kenneth Russell 2011-06-22 11:51:55 PDT
This is likely failing due to http://trac.webkit.org/changeset/89315 . From looking at the Mac specific test expectations, it looks like the test needs a new baseline, but it also looks like there might also be custom bindings that should change from throwing SyntaxError to throwing TypeError.

Jian Li, would you be willing to take this?
Comment 3 Adam Barth 2011-06-22 12:07:31 PDT
Yeah, we change the type of error thrown in this case to match the specs.  Sorry we missed this test result.  Thanks for cleaning up after us.
Comment 4 Matthew Delaney 2011-06-22 12:12:06 PDT
No worries. So, should the platform independent expected result be updated then (and the platform expected results be blown away)? Because mac SL currently fails against the current platform independent result.

I can't wait to have all of these type error issues out of the way.
Comment 5 Kenneth Russell 2011-06-22 12:14:18 PDT
Ideally yes, we should have the same result for all platforms. Hopefully Jian Li can make this work.
Comment 6 Jian Li 2011-06-23 17:16:02 PDT
Created attachment 98438 [details]
Proposed Patch
Comment 7 Kenneth Russell 2011-06-23 17:21:24 PDT
Comment on attachment 98438 [details]
Proposed Patch

It's great that so many duplicated test expectations can be deleted. Did you test on the three affected platforms?
Comment 8 Jian Li 2011-06-23 17:45:16 PDT
Tested on Mac. Will test on Chromium before landing.
Comment 9 WebKit Review Bot 2011-06-23 18:37:49 PDT
Comment on attachment 98438 [details]
Proposed Patch

Attachment 98438 [details] did not pass chromium-ews (chromium-xvfb):
Output: http://queues.webkit.org/results/8933299

New failing tests:
fast/canvas/webgl/data-view-test.html
Comment 10 WebKit Review Bot 2011-06-23 18:37:54 PDT
Created attachment 98450 [details]
Archive of layout-test-results from ec2-cr-linux-02

The attached test failures were seen while running run-webkit-tests on the chromium-ews.
Bot: ec2-cr-linux-02  Port: Chromium  Platform: Linux-2.6.35-28-virtual-x86_64-with-Ubuntu-10.10-maverick
Comment 11 Jian Li 2011-06-29 14:29:58 PDT
Committed as https://trac.webkit.org/changeset/90046.