Bug 22538

Summary: StringImpl::startsWith uses find which searches the whole string.
Product: WebKit Reporter: David Levin <levin>
Component: TextAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: levin
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Attachments:
Description Flags
Simply changes the find to reverseFind in StringImpl::startswith eric: review+

David Levin
Reported 2008-11-28 00:02:46 PST
It wouldn't if it used StringImpl::reverseFind.
Attachments
Simply changes the find to reverseFind in StringImpl::startswith (1.59 KB, patch)
2008-11-28 08:26 PST, David Levin 		eric: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
David Levin
Comment 1 2008-11-28 08:26:49 PST
Created attachment 25573 [details] Simply changes the find to reverseFind in StringImpl::startswith
Eric Seidel (no email)
Comment 2 2008-12-01 13:01:03 PST
Comment on attachment 25573 [details] Simply changes the find to reverseFind in StringImpl::startswith Looks fine to me.
Eric Seidel (no email)
Comment 3 2008-12-02 12:27:45 PST
Committing to http://svn.webkit.org/repository/webkit/trunk ... M WebCore/ChangeLog M WebCore/platform/text/StringImpl.h Committed r38910
Note You need to log in before you can comment on or make changes to this bug.