Bug 157561

Summary: Optimize DataDetection's searchForLinkRemovingExistingDDLinks()
Product: WebKit Reporter: Chris Dumez <cdumez>
Component: WebCore Misc.Assignee: Chris Dumez <cdumez>
Status: RESOLVED FIXED    
Severity: Normal CC: darin, enrica, rniwa
Priority: P2    
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch rniwa: review+

Chris Dumez
Reported 2016-05-10 22:14:28 PDT
Optimize DataDetection's searchForLinkRemovingExistingDDLinks()
Attachments
Patch (6.73 KB, patch)
2016-05-10 22:26 PDT, Chris Dumez 		rniwa: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Chris Dumez
Comment 1 2016-05-10 22:26:31 PDT
Created attachment 278587 [details] Patch
Ryosuke Niwa
Comment 2 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.
Chris Dumez
Comment 3 2016-05-12 21:50:50 PDT
Landed in r200690.
Note You need to log in before you can comment on or make changes to this bug.