Changeset 206066 in webkit


Ignore:
Timestamp:
Sep 17, 2016 1:32:15 AM (8 years ago)
Author:
Carlos Garcia Campos
Message:

[ThreadedCompositor] Scrolling artifacts on accelerated subframes
https://bugs.webkit.org/show_bug.cgi?id=149060

Reviewed by Michael Catanzaro.

The problem is that we are scrolling the main frame even when scrolling is not delegated.

  • WebProcess/WebPage/CoordinatedGraphics/ThreadedCoordinatedLayerTreeHost.cpp:

(WebKit::ThreadedCoordinatedLayerTreeHost::scrollNonCompositedContents): Return early if main frame view doesn't
delegate scrolling.

Location:
trunk/Source/WebKit2
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • trunk/Source/WebKit2/ChangeLog

    r206054 r206066  
     12016-09-17  Carlos Garcia Campos  <cgarcia@igalia.com>
     2
     3        [ThreadedCompositor] Scrolling artifacts on accelerated subframes
     4        https://bugs.webkit.org/show_bug.cgi?id=149060
     5
     6        Reviewed by Michael Catanzaro.
     7
     8        The problem is that we are scrolling the main frame even when scrolling is not delegated.
     9
     10        * WebProcess/WebPage/CoordinatedGraphics/ThreadedCoordinatedLayerTreeHost.cpp:
     11        (WebKit::ThreadedCoordinatedLayerTreeHost::scrollNonCompositedContents): Return early if main frame view doesn't
     12        delegate scrolling.
     13
    1142016-09-16  Simon Fraser  <simon.fraser@apple.com>
    215
  • trunk/Source/WebKit2/WebProcess/WebPage/CoordinatedGraphics/ThreadedCoordinatedLayerTreeHost.cpp

    r205431 r206066  
    8080void ThreadedCoordinatedLayerTreeHost::scrollNonCompositedContents(const IntRect& rect)
    8181{
     82    FrameView* frameView = m_webPage.mainFrameView();
     83    if (!frameView || !frameView->delegatesScrolling())
     84        return;
     85
    8286    m_viewportController.didScroll(rect.location());
    8387    didChangeViewport();
Note: See TracChangeset for help on using the changeset viewer.