Bug 157561 - Optimize DataDetection's searchForLinkRemovingExistingDDLinks()
Summary: Optimize DataDetection's searchForLinkRemovingExistingDDLinks()
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebCore Misc. (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Chris Dumez
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-05-10 22:14 PDT by Chris Dumez
Modified: 2016-05-12 21:50 PDT (History)
3 users (show)

See Also:


Attachments
Patch (6.73 KB, patch)
2016-05-10 22:26 PDT, Chris Dumez
rniwa: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Chris Dumez 2016-05-10 22:14:28 PDT
Optimize DataDetection's searchForLinkRemovingExistingDDLinks()
Comment 1 Chris Dumez 2016-05-10 22:26:31 PDT
Created attachment 278587 [details]
Patch
Comment 2 Ryosuke Niwa 2016-05-10 23:41:14 PDT
Comment on attachment 278587 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=278587&action=review

> Source/WebCore/ChangeLog:17
> +       2. The second loop was using again Node::childNodes() and kept calling

It looks like 2 here is one space outdented.
Comment 3 Chris Dumez 2016-05-12 21:50:50 PDT
Landed in r200690.