Bug 50334 - xhr.responseType = 'arraybuffer' works on Mac but not on Windows
Summary: xhr.responseType = 'arraybuffer' works on Mac but not on Windows
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC Windows XP
: P2 Normal
Assignee: Chris Marrin
URL:
Keywords: InRadar, LayoutTestFailure, PlatformOnly
Depends on:
Blocks:
 
Reported: 2010-12-01 10:36 PST by Adam Roben (:aroben)
Modified: 2011-04-11 16:33 PDT (History)
4 users (show)

See Also:


Attachments
Patch (8.73 KB, patch)
2011-04-11 16:08 PDT, Chris Marrin
mitz: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Adam Roben (:aroben) 2010-12-01 10:36:39 PST
As part of bug 32933, blob support got turned on for the Mac. As time has gone on, more and more code has come to depend on ENABLE(BLOB). This means that Mac and Windows are diverging more and more in terms of web APIs. For example, the XHR responseType 'arraybuffer' is dependent on ENABLE(BLOB), meaning that it works on Mac but not on Windows. This is causing multiple regression tests to fail on Windows currently.

We should make Mac and Windows behave the same in this regard. We should either turn off BLOB on Mac or turn it on on Windows.

(You can see the regression test failures here: <http://build.webkit.org/results/Windows%207%20Release%20(Tests)/r73026%20(6983)/results.html>. Ignore the non-XHR ones.)
Comment 1 Adam Roben (:aroben) 2010-12-01 10:37:14 PST
<rdar://problem/8716459>
Comment 2 Brian Weinstein 2010-12-01 11:36:49 PST
Landed failing results for Windows for:

fast/xmlhttprequest/xmlhttprequest-responsetype-abort.html
fast/xmlhttprequest/xmlhttprequest-responsetype-arraybuffer.html
http/tests/xmlhttprequest/send-array-buffer.html

In r73048.
Comment 3 Chris Marrin 2011-04-11 16:08:54 PDT
Created attachment 89114 [details]
Patch
Comment 4 mitz 2011-04-11 16:10:59 PDT
Comment on attachment 89114 [details]
Patch

rs=me
Comment 5 Chris Marrin 2011-04-11 16:33:18 PDT
Fixed in http://trac.webkit.org/changeset/83532