Skip to content
  • tonikitoo@webkit.org's avatar
    [BlackBerry] Remove the 'in region scrollable starting node' concept from InRegionScroller · e5b7e6e2
    tonikitoo@webkit.org authored
    https://bugs.webkit.org/show_bug.cgi?id=95020
    
    Reviewed by Rob Buis.
    Patch by Antonio Gomes <agomes@rim.com>
    
    'In-region start scrolling node' is an obsolete concept, and we can remove it
    in favor of using the information from the cached layers.
    
    * Api/InRegionScroller.cpp:
    (WebKit):
    (BlackBerry::WebKit::InRegionScrollerPrivate::reset): Adjusted as it used to clear
    the cached 'node'.
    (BlackBerry::WebKit::InRegionScrollerPrivate::isActive): Added method to be
    checked directly instead of only null-checking the previously cached 'node'.
    (BlackBerry::WebKit::InRegionScrollerPrivate::clearDocumentData): New method to
    clear the cached resources if its document is done.
    (BlackBerry::WebKit::InRegionScrollerPrivate::pushBackInRegionScrollable): Adjusted
    to not care about the cached 'node' anymore.
    * Api/InRegionScroller_p.h:
    (InRegionScrollerPrivate):
    * Api/WebPage.cpp:
    (BlackBerry::WebKit::WebPagePrivate::notifyInRegionScrollStopped): Check ::isActive instead
    of ::hasNode, since the later is gone.
    (BlackBerry::WebKit::WebPagePrivate::enqueueRenderingOfClippedContentOfScrollableAreaAfterInRegionScrolling):
    Changed the method signature, given that we do have a cached 'node' to pass in as parameter anymore.
    (BlackBerry::WebKit::WebPagePrivate::clearDocumentData): Delegate all the related work to InRegionScroller.
    * Api/WebPage_p.h:
    (WebPagePrivate):
    * WebKitSupport/TouchEventHandler.cpp:
    (BlackBerry::WebKit::TouchEventHandler::drawTapHighlight):
    
    git-svn-id: http://svn.webkit.org/repository/webkit/trunk@126768 268f45cc-cd09-0410-ab3c-d52691b4dbfc
    e5b7e6e2