In webkit.org/b/156013, I refactored stringProtoFunc into stringProtoFuncSplit. In that patch, I left an unnecessary block of code in its original block (with FIXMEs) to keep the diff for that patch minimal. Now that the patch for webkit.org/b/156013 has landed, I will unindent that block and remove the FIXMEs.
(In reply to comment #0) > In webkit.org/b/156013, I refactored stringProtoFunc into > stringProtoFuncSplit. In that patch, I left an unnecessary block of code in > its original block (with FIXMEs) to keep the diff for that patch minimal. > Now that the patch for webkit.org/b/156013 has landed, I will unindent that > block and remove the FIXMEs. typo: /stringProtoFunc/stringProtoFuncSplit/ and /stringProtoFuncSplit/stringProtoFuncSplitFast/.
Created attachment 276838 [details] proposed patch.
Landed in r199780: <http://trac.webkit.org/r199780>.
Comment on attachment 276838 [details] proposed patch. View in context: https://bugs.webkit.org/attachment.cgi?id=276838&action=review > Source/JavaScriptCore/runtime/StringPrototype.cpp:1191 > + String separator = separatorValue.toString(exec)->value(exec); Seems we should always use toWTFString instead of writing it out like this.