Bug 83053

Summary: Fix MediaSource LayoutTests to work with new XHR behavior
Product: WebKit Reporter: Aaron Colwell <acolwell>
Component: MediaAssignee: Aaron Colwell <acolwell>
Status: RESOLVED FIXED    
Severity: Normal CC: dglazkov, tkent, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Archive of layout-test-results from ec2-cr-linux-03
none
Removed clusters to make test duration shorter. none

Description Aaron Colwell 2012-04-03 11:27:30 PDT
At some point XHR behavior changed so that the 'arraybuffer' responseType could not be used with synchronous XHR requests. The LayoutTests for the MediaSource API rely on being able to make synchronous XHR requests that returned ArrayBuffer responses. 

This bug is for patches to fix the LayoutTests so they work with the new XHR behavior. These LayoutTests will also be enabled for Chromium by default so that such breakages can be detected earlier.
Comment 1 Aaron Colwell 2012-04-03 11:46:37 PDT
Created attachment 135378 [details]
Patch
Comment 2 WebKit Review Bot 2012-04-03 13:25:56 PDT
Comment on attachment 135378 [details]
Patch

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

New failing tests:
http/tests/media/media-source/webm/video-media-source-play.html
http/tests/media/media-source/webm/video-media-source-seek.html
Comment 3 WebKit Review Bot 2012-04-03 13:26:02 PDT
Created attachment 135408 [details]
Archive of layout-test-results from ec2-cr-linux-03

The attached test failures were seen while running run-webkit-tests on the chromium-ews.
Bot: ec2-cr-linux-03  Port: <class 'webkitpy.common.config.ports.ChromiumXVFBPort'>  Platform: Linux-2.6.35-28-virtual-x86_64-with-Ubuntu-10.10-maverick
Comment 4 Aaron Colwell 2012-04-03 14:00:51 PDT
Created attachment 135420 [details]
Removed clusters to make test duration shorter.
Comment 5 Aaron Colwell 2012-04-06 13:09:22 PDT
ping. Will someone review this please?
Comment 6 WebKit Review Bot 2012-04-06 14:19:48 PDT
Comment on attachment 135420 [details]
Removed clusters to make test duration shorter.

Clearing flags on attachment: 135420

Committed r113495: <http://trac.webkit.org/changeset/113495>
Comment 7 WebKit Review Bot 2012-04-06 14:20:01 PDT
All reviewed patches have been landed.  Closing bug.