Bug 75206

Summary: Yarr: Avoid copying vectors in CharacterClassConstructor.
Product: WebKit Reporter: Andreas Kling <akling@apple.com>
Component: JavaScriptCoreAssignee: Andreas Kling <akling@apple.com>
Status: RESOLVED FIXED    
Severity: Normal CC: dglazkov@chromium.org, webkit.review.bot@gmail.com
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Proposed patch darin: review+, webkit.review.bot: commit‑queue-

Description From 2011-12-25 12:00:39 PST
CharacterClassConstructor::charClass() gets a little hot (1.2%) when loading twitter feeds, and we can avoid the copy altogether by using Vector::swap().
------- Comment #1 From 2011-12-25 12:01:42 PST -------
Created an attachment (id=120512) [details]
Proposed patch
------- Comment #2 From 2011-12-25 12:54:05 PST -------
(From update of attachment 120512 [details])
Attachment 120512 [details] did not pass chromium-ews (chromium-xvfb):
Output: http://queues.webkit.org/results/11027710

New failing tests:
http/tests/inspector/resource-tree/resource-tree-document-url.html
------- Comment #3 From 2011-12-25 13:21:20 PST -------
Committed r103672: <http://trac.webkit.org/changeset/103672>