Bug 128903

Summary: [XHR] Cache response JS object in case of arraybuffer and blob response types
Product: WebKit Reporter: youenn fablet <youennf>
Component: WebCore Misc.Assignee: youenn fablet <youennf>
Status: RESOLVED FIXED    
Severity: Normal CC: buildbot, cdumez, commit-queue, rniwa
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
WIP
none
WIP2
none
Patch
none
Archive of layout-test-results from ews103 for mac-yosemite
none
Archive of layout-test-results from ews106 for mac-yosemite-wk2
none
Archive of layout-test-results from ews125 for ios-simulator-wk2
none
Archive of layout-test-results from ews116 for mac-yosemite
none
Fixing responseText
none
Patch for landing none

Description youenn fablet 2014-02-17 05:36:39 PST
XHR currently cache response object when responseType is json.
This can also be done for other types as well (arraybuffer and blob notably) as a small optimization step.
Comment 1 youenn fablet 2014-02-17 05:56:12 PST
Created attachment 224361 [details]
Patch
Comment 2 youenn fablet 2016-07-19 02:33:19 PDT
Created attachment 283987 [details]
WIP
Comment 3 youenn fablet 2016-07-19 02:53:06 PDT
Created attachment 283988 [details]
WIP2
Comment 4 youenn fablet 2016-07-19 05:13:47 PDT
Created attachment 283992 [details]
Patch
Comment 5 Build Bot 2016-07-19 06:09:14 PDT
Comment on attachment 283992 [details]
Patch

Attachment 283992 [details] did not pass mac-ews (mac):
Output: http://webkit-queues.webkit.org/results/1706634

New failing tests:
fast/xmlhttprequest/xmlhttprequest-responsetype-text.html
Comment 6 Build Bot 2016-07-19 06:09:16 PDT
Created attachment 283995 [details]
Archive of layout-test-results from ews103 for mac-yosemite

The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: ews103  Port: mac-yosemite  Platform: Mac OS X 10.10.5
Comment 7 Build Bot 2016-07-19 06:13:03 PDT
Comment on attachment 283992 [details]
Patch

Attachment 283992 [details] did not pass mac-wk2-ews (mac-wk2):
Output: http://webkit-queues.webkit.org/results/1706641

New failing tests:
fast/xmlhttprequest/xmlhttprequest-responsetype-text.html
Comment 8 Build Bot 2016-07-19 06:13:05 PDT
Created attachment 283996 [details]
Archive of layout-test-results from ews106 for mac-yosemite-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews106  Port: mac-yosemite-wk2  Platform: Mac OS X 10.10.5
Comment 9 Build Bot 2016-07-19 06:17:59 PDT
Comment on attachment 283992 [details]
Patch

Attachment 283992 [details] did not pass ios-sim-ews (ios-simulator-wk2):
Output: http://webkit-queues.webkit.org/results/1706637

New failing tests:
fast/xmlhttprequest/xmlhttprequest-responsetype-text.html
Comment 10 Build Bot 2016-07-19 06:18:02 PDT
Created attachment 283997 [details]
Archive of layout-test-results from ews125 for ios-simulator-wk2

The attached test failures were seen while running run-webkit-tests on the ios-sim-ews.
Bot: ews125  Port: ios-simulator-wk2  Platform: Mac OS X 10.11.5
Comment 11 Build Bot 2016-07-19 06:18:34 PDT
Comment on attachment 283992 [details]
Patch

Attachment 283992 [details] did not pass mac-debug-ews (mac):
Output: http://webkit-queues.webkit.org/results/1706640

New failing tests:
fast/xmlhttprequest/xmlhttprequest-responsetype-text.html
Comment 12 Build Bot 2016-07-19 06:18:37 PDT
Created attachment 283998 [details]
Archive of layout-test-results from ews116 for mac-yosemite

The attached test failures were seen while running run-webkit-tests on the mac-debug-ews.
Bot: ews116  Port: mac-yosemite  Platform: Mac OS X 10.10.5
Comment 13 youenn fablet 2016-07-19 06:47:28 PDT
Created attachment 284002 [details]
Fixing responseText
Comment 14 WebKit Commit Bot 2016-07-20 22:51:38 PDT
Comment on attachment 284002 [details]
Fixing responseText

Rejecting attachment 284002 [details] from commit-queue.

Failed to run "['/Volumes/Data/EWS/WebKit/Tools/Scripts/webkit-patch', '--status-host=webkit-queues.webkit.org', '--bot-id=webkit-cq-03', 'apply-attachment', '--no-update', '--non-interactive', 284002, '--port=mac']" exit_code: 2 cwd: /Volumes/Data/EWS/WebKit

Last 500 characters of output:
dl
Hunk #2 succeeded at 80 (offset 2 lines).
patching file Source/WebCore/xml/XMLHttpRequest.js
patching file LayoutTests/ChangeLog
Hunk #1 succeeded at 1 with fuzz 3.
patching file LayoutTests/http/tests/xmlhttprequest/onabort-response-getters-expected.txt
patching file LayoutTests/http/tests/xmlhttprequest/onabort-response-getters.html

Failed to run "[u'/Volumes/Data/EWS/WebKit/Tools/Scripts/svn-apply', '--force', '--reviewer', u'Alex Christensen']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit

Full output: http://webkit-queues.webkit.org/results/1721214
Comment 15 youenn fablet 2016-07-20 23:21:10 PDT
Created attachment 284193 [details]
Patch for landing
Comment 16 WebKit Commit Bot 2016-07-20 23:52:03 PDT
Comment on attachment 284193 [details]
Patch for landing

Clearing flags on attachment: 284193

Committed r203494: <http://trac.webkit.org/changeset/203494>
Comment 17 WebKit Commit Bot 2016-07-20 23:52:08 PDT
All reviewed patches have been landed.  Closing bug.