Bug 122850 - Avoid iterating over all the isolate characters
Summary: Avoid iterating over all the isolate characters
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: Layout and Rendering (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Myles C. Maxfield
URL:
Keywords: BlinkMergeCandidate
Depends on:
Blocks:
 
Reported: 2013-10-15 12:20 PDT by Ryosuke Niwa
Modified: 2013-10-15 13:00 PDT (History)
4 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Ryosuke Niwa 2013-10-15 12:20:06 PDT 
Consider merging https://chromium.googlesource.com/chromium/blink/+/496b731fbfccba1a4ac26e44a34e0e8ea70b0a80

Isolate elements are treated as neutral in UBA however Blink UBA implementation iterates over all the isolate characters.
The purpose of this patch is reduce the algorithm complexity skipping isolate elements when found in the UBA. 
Additionaly, rename the pastEnd variable to lastParagraphEnded.