RESOLVED FIXED 68015
[v8] Improve performance of typed array copy constructor taking Array 
https://bugs.webkit.org/show_bug.cgi?id=68015
Summary [v8] Improve performance of typed array copy constructor taking Array
Ulan Degenbaev
Reported 2011-09-13 11:48:06 PDT
This issue is related to https://bugs.webkit.org/show_bug.cgi?id=63644 . The copy constructor copies elements of the source array one by one using the V8 API functions, which produces many context switches the WebKit and the V8. We can speed it up by invoking the 'set' method of the constructed array in JavaScript.
Attachments
Invoke the 'set' method of the constructed array (5.52 KB, patch)
2011-09-13 12:17 PDT, Ulan Degenbaev 		no flags
DetailsFormatted DiffDiff
Fix style error (include primary header) and include ExceptionCode.h in V8ArrayBufferViewCustom.h (5.71 KB, patch)
2011-09-13 12:47 PDT, Ulan Degenbaev 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Ulan Degenbaev
Comment 1 2011-09-13 12:17:20 PDT
Created attachment 107209 [details] Invoke the 'set' method of the constructed array
WebKit Review Bot
Comment 2 2011-09-13 12:20:51 PDT
Attachment 107209 [details] did not pass style-queue: Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'ChangeLog', u'Source/WebCore/WebCore.gypi'..." exit_code: 1 Source/WebCore/bindings/v8/custom/V8ArrayBufferViewCustom.cpp:28: Found other header before a header this file implements. Should be: config.h, primary header, blank line, and then alphabetically sorted. [build/include_order] [4] Total errors found: 1 in 4 files If any of these errors are false positives, please file a bug against check-webkit-style.
Ulan Degenbaev
Comment 3 2011-09-13 12:47:14 PDT
Created attachment 107212 [details] Fix style error (include primary header) and include ExceptionCode.h in V8ArrayBufferViewCustom.h
Kenneth Russell
Comment 4 2011-09-14 11:48:28 PDT
Comment on attachment 107212 [details] Fix style error (include primary header) and include ExceptionCode.h in V8ArrayBufferViewCustom.h Thanks, looks good. r=me
WebKit Review Bot
Comment 5 2011-09-14 20:27:49 PDT
Comment on attachment 107212 [details] Fix style error (include primary header) and include ExceptionCode.h in V8ArrayBufferViewCustom.h Clearing flags on attachment: 107212 Committed r95157: <http://trac.webkit.org/changeset/95157>
WebKit Review Bot
Comment 6 2011-09-14 20:27:53 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.