Bug 99649

Summary: [BlackBerry] Cache TouchDown HitTestResult to avoid calling RenderLayer::hitTest unnecessarily
Product: WebKit Reporter: Mike Lattanzio <mlattanzio>
Component: WebKit BlackBerryAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: allan.jensen, mifenton, rwlbuis, tonikitoo, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Other   
OS: Unspecified   
Attachments:
Description Flags
Patch none

Mike Lattanzio
Reported 2012-10-17 16:01:50 PDT
RenderLayer::hitTest() can be expensive. We can merge together some of the calls by caching the result. This speeds up the hot path in our touch scrolling code.
Attachments
Patch (12.04 KB, patch)
2012-10-23 15:10 PDT, Mike Lattanzio 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Mike Lattanzio
Comment 1 2012-10-23 15:10:48 PDT
Created attachment 170244 [details] Patch
WebKit Review Bot
Comment 2 2012-10-23 16:11:01 PDT
Comment on attachment 170244 [details] Patch Clearing flags on attachment: 170244 Committed r132279: <http://trac.webkit.org/changeset/132279>
WebKit Review Bot
Comment 3 2012-10-23 16:11:04 PDT
All reviewed patches have been landed. Closing bug.
Antonio Gomes
Comment 4 2012-10-24 19:51:16 PDT
Comment on attachment 170244 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=170244&action=review Good stuff Lattanzio. > Source/WebKit/blackberry/Api/WebPage.cpp:6338 > +bool WebPagePrivate::postponeDocumentStyleRecalc() Why is it returning bool?
Note You need to log in before you can comment on or make changes to this bug.