Summary: | Region::contains(IntPoint) is slow | ||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Dana Jansens <danakj> | ||||||||||||
Component: | New Bugs | Assignee: | Dana Jansens <danakj> | ||||||||||||
Status: | RESOLVED FIXED | ||||||||||||||
Severity: | Normal | CC: | andersca, backer, jamesr, piman, webkit.review.bot | ||||||||||||
Priority: | P2 | ||||||||||||||
Version: | 528+ (Nightly build) | ||||||||||||||
Hardware: | Unspecified | ||||||||||||||
OS: | Unspecified | ||||||||||||||
Attachments: |
|
Description
Dana Jansens
2012-03-13 10:08:20 PDT
Created attachment 131653 [details]
Patch
Created attachment 131672 [details]
Patch
Created attachment 131680 [details]
Patch
Removed the early-out on if (isEmpty()). The first for loop will break immediately anyhow, so this is just adding extra comparisons.
Comment on attachment 131680 [details]
Patch
Oops, meant to r- this. Could you add a more descriptive explanation in the ChangeLog? Other than that the change looks fine!
Created attachment 131682 [details]
Patch
Created attachment 131685 [details]
Patch
Comment on attachment 131685 [details] Patch Clearing flags on attachment: 131685 Committed r110624: <http://trac.webkit.org/changeset/110624> All reviewed patches have been landed. Closing bug. |