Bug 139958 - fast/canvas/webgl/angle-instanced-arrays.html is very flaky
Summary: fast/canvas/webgl/angle-instanced-arrays.html is very flaky
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Alexey Proskuryakov
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-12-26 13:40 PST by Alexey Proskuryakov
Modified: 2014-12-27 15:22 PST (History)
11 users (show)

See Also:


Attachments
proposed fix (134.89 KB, patch)
2014-12-26 16:12 PST, Alexey Proskuryakov
buildbot: commit-queue-
Details | Formatted Diff | Diff
Archive of layout-test-results from ews106 for mac-mountainlion-wk2 (555.06 KB, application/zip)
2014-12-26 17:01 PST, Build Bot
no flags Details
proposed fix (140.39 KB, patch)
2014-12-26 18:32 PST, Alexey Proskuryakov
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Alexey Proskuryakov 2014-12-26 13:40:23 PST
fast/canvas/webgl/angle-instanced-arrays.html frequently fails, dumping TEST COMPLETE twice:

@@ -81,4 +81,7 @@
 PASS successfullyParsed is true
 
 TEST COMPLETE
+PASS successfullyParsed is true
 
+TEST COMPLETE
+

This is a problem with both the test and test framework. They way js-test-post.js is inserted is incorrect, because it is affected by how quickly test document loads.
Comment 1 Alexey Proskuryakov 2014-12-26 16:12:36 PST
Created attachment 243766 [details]
proposed fix

This cleans up test harness usage in all fast/canvas/webgl.

We also have many of the same tests as part of WebGL 1.0.x test suites, and looks like they often have the same issues. I'm not sure if there is a script that imports these, if so, it should be probably updated to use these techniques. Otherwise, this cleanup could serve as an example for how these other copies should be updated.
Comment 2 Build Bot 2014-12-26 17:01:06 PST
Comment on attachment 243766 [details]
proposed fix

Attachment 243766 [details] did not pass mac-wk2-ews (mac-wk2):
Output: http://webkit-queues.appspot.com/results/5868542205362176

New failing tests:
svg/custom/html_document_set_title.html
fast/dom/HTMLImageElement/sizes/image-sizes-js-innerhtml.html
fast/dom/HTMLImageElement/sizes/image-sizes-2x.html
fast/dom/HTMLImageElement/sizes/image-sizes-1x.html
fast/hidpi/image-srcset-src-selection-1x-both.html
fast/hidpi/image-srcset-invalid-descriptor.html
Comment 3 Build Bot 2014-12-26 17:01:10 PST
Created attachment 243768 [details]
Archive of layout-test-results from ews106 for mac-mountainlion-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews106  Port: mac-mountainlion-wk2  Platform: Mac OS X 10.8.5
Comment 4 Alexey Proskuryakov 2014-12-26 18:32:34 PST
Created attachment 243769 [details]
proposed fix

Nice, so several other tests that use js-test.js have nicer output now.
Comment 5 WebKit Commit Bot 2014-12-27 14:35:00 PST
Comment on attachment 243769 [details]
proposed fix

Rejecting attachment 243769 [details] from commit-queue.

Failed to run "['/Volumes/Data/EWS/WebKit/Tools/Scripts/webkit-patch', '--status-host=webkit-queues.appspot.com', '--bot-id=webkit-cq-03', 'land-attachment', '--force-clean', '--non-interactive', '--parent-command=commit-queue', 243769, '--port=mac']" exit_code: 2 cwd: /Volumes/Data/EWS/WebKit

Last 500 characters of output:
nch master is up to date.
Auto packing the repository for optimum performance. You may also
run "git gc" manually. See "git help gc" for more information.
ERROR: Not all changes have been committed into SVN, however the committed
ones (if any) seem to be successfully integrated into the working tree.
Please see the above messages for details.


Failed to run "['git', 'svn', 'dcommit', '--rmdir']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit
Updating OpenSource
Current branch master is up to date.

Full output: http://webkit-queues.appspot.com/results/6122017216528384
Comment 6 WebKit Commit Bot 2014-12-27 15:22:07 PST
Comment on attachment 243769 [details]
proposed fix

Clearing flags on attachment: 243769

Committed r177772: <http://trac.webkit.org/changeset/177772>
Comment 7 WebKit Commit Bot 2014-12-27 15:22:12 PST
All reviewed patches have been landed.  Closing bug.