RESOLVED FIXED 142080
Implement ES6 StringIterator 
https://bugs.webkit.org/show_bug.cgi?id=142080
Summary Implement ES6 StringIterator
Yusuke Suzuki
Reported 2015-02-27 02:25:16 PST
Implement ES6 StringIterator
Attachments
rev1 prototype (33.50 KB, patch)
2015-02-27 03:40 PST, Yusuke Suzuki 		no flags
DetailsFormatted DiffDiff
prototype rev2 (46.49 KB, patch)
2015-02-27 09:14 PST, Yusuke Suzuki 		no flags
DetailsFormatted DiffDiff
Patch (46.45 KB, patch)
2015-03-03 10:15 PST, Yusuke Suzuki 		no flags
DetailsFormatted DiffDiff
Patch (46.57 KB, patch)
2015-03-05 08:39 PST, Yusuke Suzuki 		no flags
DetailsFormatted DiffDiff
Show Obsolete (3) View All Add attachment proposed patch, testcase, etc.
Yusuke Suzuki
Comment 1 2015-02-27 03:40:12 PST
Created attachment 247507 [details] rev1 prototype prototype. not including build configuration for OSX and Windows yet. And this patch requires https://bugs.webkit.org/show_bug.cgi?id=141351 change.
Yusuke Suzuki
Comment 2 2015-02-27 09:14:27 PST
Created attachment 247516 [details] prototype rev2 Update for Windows / OSX build
WebKit Commit Bot
Comment 3 2015-03-02 04:07:08 PST
Attachment 247516 [details] did not pass style-queue: ERROR: Source/JavaScriptCore/runtime/StringIteratorPrototype.cpp:42: Alphabetical sorting problem. [build/include_order] [4] Total errors found: 1 in 18 files If any of these errors are false positives, please file a bug against check-webkit-style.
Yusuke Suzuki
Comment 4 2015-03-03 10:15:01 PST
Created attachment 247767 [details] Patch
Filip Pizlo
Comment 5 2015-03-04 20:10:56 PST
Comment on attachment 247767 [details] Patch LGTM. I can r+ once you're ready.
Yusuke Suzuki
Comment 6 2015-03-05 08:39:01 PST
Created attachment 247952 [details] Patch
WebKit Commit Bot
Comment 7 2015-03-05 08:41:02 PST
Attachment 247952 [details] did not pass style-queue: ERROR: Source/JavaScriptCore/runtime/StringIteratorPrototype.cpp:42: Alphabetical sorting problem. [build/include_order] [4] Total errors found: 1 in 18 files If any of these errors are false positives, please file a bug against check-webkit-style.
Yusuke Suzuki
Comment 8 2015-03-05 08:42:40 PST
(In reply to comment #5) > Comment on attachment 247767 [details] > Patch > > LGTM. I can r+ once you're ready. Great! I've landed https://trac.webkit.org/changeset/181077, rebased my patch and marked r? :D
Yusuke Suzuki
Comment 9 2015-03-05 09:41:49 PST
Comment on attachment 247952 [details] Patch Thank you!
WebKit Commit Bot
Comment 10 2015-03-05 10:27:52 PST
Comment on attachment 247952 [details] Patch Clearing flags on attachment: 247952 Committed r181084: <http://trac.webkit.org/changeset/181084>
WebKit Commit Bot
Comment 11 2015-03-05 10:27:57 PST
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.