The overhead of having a function for StringImpl::find() is significant.
Created attachment 136803 [details] Patch
Comment on attachment 136803 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=136803&action=review r=me > Source/WTF/wtf/text/StringImpl.cpp:-780 > -size_t StringImpl::find(UChar c, unsigned start) It looks like this code sometimes used "index" and sometimes used "start" for this argument name. FWIW, I slightly prefer "start" over "index", since it's more precise. > Source/WTF/wtf/text/StringImpl.h:484 > + inline size_t find(UChar character, unsigned index = 0); No need for "inline" here.
Committed r114071: <http://trac.webkit.org/changeset/114071>