Bug 95476 - [BlackBerry] Remove unneeded force-immediate-repaint from InRegionScroller::setLayerScrollPosition
Summary: [BlackBerry] Remove unneeded force-immediate-repaint from InRegionScroller::s...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit BlackBerry (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Antonio Gomes
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-08-30 10:33 PDT by Antonio Gomes
Modified: 2012-08-30 10:48 PDT (History)
3 users (show)

See Also:


Attachments
patch (3.27 KB, patch)
2012-08-30 10:37 PDT, Antonio Gomes
yong.li.webkit: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Antonio Gomes 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
Comment 1 Antonio Gomes 2012-08-30 10:37:26 PDT
Created attachment 161512 [details]
patch
Comment 2 Yong Li 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"
Comment 3 Antonio Gomes 2012-08-30 10:48:04 PDT
http://trac.webkit.org/changeset/127160