Bug 95476

Summary: [BlackBerry] Remove unneeded force-immediate-repaint from InRegionScroller::setLayerScrollPosition
Product: WebKit Reporter: Antonio Gomes <tonikitoo>
Component: WebKit BlackBerryAssignee: Antonio Gomes <tonikitoo>
Status: RESOLVED FIXED    
Severity: Normal CC: mifenton, tonikitoo, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
patch yong.li.webkit: review+

Antonio Gomes
Reported 2012-08-30 10:33:17 PDT
This code is not needed at this point for neither the fast nor slow scroll code paths at this point, and it a huge performance beast as it forces all containers to get full-repainted by scroll call (in webkit thread). PR #200704
Attachments
patch (3.27 KB, patch)
2012-08-30 10:37 PDT, Antonio Gomes 		yong.li.webkit: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Antonio Gomes
Comment 1 2012-08-30 10:37:26 PDT
Created attachment 161512 [details] patch
Yong Li
Comment 2 2012-08-30 10:43:13 PDT
Comment on attachment 161512 [details] patch View in context: https://bugs.webkit.org/attachment.cgi?id=161512&action=review > Source/WebKit/blackberry/ChangeLog:11 > + scroll codepaths, and it a huge performance beast as it forces all missing "is"
Antonio Gomes
Comment 3 2012-08-30 10:48:04 PDT
http://trac.webkit.org/changeset/127160
Note You need to log in before you can comment on or make changes to this bug.