Bug 89252 – Make serve-video.php serve 206 responses

• Status: RESOLVED
• Resolution: FIXED
• Component: Media

[Ami Fischman]

http/tests/media/resources/serve-video.php has code that seemingly serves 206 in response to Range requests, but is slightly buggy, causing all its responses to always be 200's.  This hid the bug fixed in bug 89185.
Once chromium is updated to use the new option (exposeAllResponseHeaders) and rolls into WebKit, the .php file can be fixed to return real 206's without breaking tests.

[Ami Fischman]

Chromium-side fix is in https://chromiumcodereview.appspot.com/10554014/

[Ami Fischman]

Created attachment 147990 [details]
Patch

[Ami Fischman]

(patch is expected to make cr-linux EWS fail b/c the chromium-side CL hasn't landed yet)

[WebKit Review Bot]

Comment on attachment 147990 [details]
Patch

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

New failing tests:
http/tests/security/video-cross-origin-readback.html

[WebKit Review Bot]

Created attachment 147995 [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: <class 'webkitpy.common.config.ports.ChromiumXVFBPort'>  Platform: Linux-2.6.35-28-virtual-x86_64-with-Ubuntu-10.10-maverick

[WebKit Review Bot]

Comment on attachment 147990 [details]
Patch

Clearing flags on attachment: 147990

Committed r120660: <http://trac.webkit.org/changeset/120660>

[WebKit Review Bot]

All reviewed patches have been landed.  Closing bug.