Changeset 92030 in webkit


Ignore:
Timestamp:
Jul 29, 2011 4:11:13 PM (13 years ago)
Author:
enne@google.com
Message:

[chromium] Only force slow scrolling for iframes under --force-compositing-mode
https://bugs.webkit.org/show_bug.cgi?id=65394

Reviewed by James Robinson.

The previous code introduced by r91591 made composited frames as well
as non-composited subframes go down the slow scrolling path. Adjust
the check to use ownerElement to test for subframe and use
hasCompositedContent() which is a more correct test for compositing
than isEnclosedInCompositingLayer.

  • page/FrameView.cpp:

(WebCore::FrameView::useSlowRepaints):

Location:
trunk/Source/WebCore
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • trunk/Source/WebCore/ChangeLog

    r92025 r92030  
     12011-07-29  Adrienne Walker  <enne@google.com>
     2
     3        [chromium] Only force slow scrolling for iframes under --force-compositing-mode
     4        https://bugs.webkit.org/show_bug.cgi?id=65394
     5
     6        Reviewed by James Robinson.
     7
     8        The previous code introduced by r91591 made composited frames as well
     9        as non-composited subframes go down the slow scrolling path.  Adjust
     10        the check to use ownerElement to test for subframe and use
     11        hasCompositedContent() which is a more correct test for compositing
     12        than isEnclosedInCompositingLayer.
     13
     14        * page/FrameView.cpp:
     15        (WebCore::FrameView::useSlowRepaints):
     16
    1172011-07-29  Emil A Eklund  <eae@chromium.org>
    218
  • trunk/Source/WebCore/page/FrameView.cpp

    r92011 r92030  
    11471147    // The chromium compositor does not support scrolling a non-composited frame within a composited page through
    11481148    // the fast scrolling path, so force slow scrolling in that case.
    1149     if (!isEnclosedInCompositingLayer() && m_frame->page() && m_frame->page()->mainFrame()->view()->hasCompositedContent())
     1149    if (m_frame->ownerElement() && !hasCompositedContent() && m_frame->page() && m_frame->page()->mainFrame()->view()->hasCompositedContent())
    11501150        return true;
    11511151#endif
Note: See TracChangeset for help on using the changeset viewer.