RESOLVED FIXED 128457
Stop using getCharactersWithUpconvert in JavaScriptCore 
https://bugs.webkit.org/show_bug.cgi?id=128457
Summary Stop using getCharactersWithUpconvert in JavaScriptCore
Anders Carlsson
Reported 2014-02-08 10:17:49 PST
Stop using getCharactersWithUpconvert in JavaScriptCore
Attachments
Patch (6.49 KB, patch)
2014-02-08 10:19 PST, Anders Carlsson 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Anders Carlsson
Comment 1 2014-02-08 10:19:47 PST
Created attachment 223570 [details] Patch
Andreas Kling
Comment 2 2014-02-08 12:43:21 PST
Comment on attachment 223570 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=223570&action=review Nice! r=me > Source/WTF/wtf/text/StringView.h:141 > + if (is8Bit()) > + return WTF::find(characters8(), length(), character, start); I wonder if it would be useful to have an early return here if 'character' is outside the LChar range. NABD.
Geoffrey Garen
Comment 3 2014-02-08 13:11:51 PST
Comment on attachment 223570 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=223570&action=review > Source/JavaScriptCore/ChangeLog:9 > + if the source or replacement strings area 16-bit. "are"
WebKit Commit Bot
Comment 4 2014-02-08 15:26:39 PST
Comment on attachment 223570 [details] Patch Clearing flags on attachment: 223570 Committed r163727: <http://trac.webkit.org/changeset/163727>
WebKit Commit Bot
Comment 5 2014-02-08 15:26:41 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.