Changeset 230480 in webkit


Ignore:
Timestamp:
Apr 10, 2018 7:56:21 AM (6 years ago)
Author:
Alan Bujtas
Message:

Do not layout images when we only need the overflow information.
https://bugs.webkit.org/show_bug.cgi?id=175331

Reviewed by Simon Fraser.

This improves MotionMark's Simple Leaves by ~10%.

Covered by existing tests.

  • rendering/RenderImage.cpp:

(WebCore::RenderImage::layout):

Location:
trunk/Source/WebCore
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • trunk/Source/WebCore/ChangeLog

    r230479 r230480  
     12018-04-10  Zalan Bujtas  <zalan@apple.com>
     2
     3        Do not layout images when we only need the overflow information.
     4        https://bugs.webkit.org/show_bug.cgi?id=175331
     5
     6        Reviewed by Simon Fraser.
     7
     8        This improves MotionMark's Simple Leaves by ~10%.
     9
     10        Covered by existing tests.
     11
     12        * rendering/RenderImage.cpp:
     13        (WebCore::RenderImage::layout):
     14
    1152018-04-10  Zan Dobersek  <zdobersek@igalia.com>
    216
  • trunk/Source/WebCore/rendering/RenderImage.cpp

    r229174 r230480  
    700700void RenderImage::layout()
    701701{
     702    // Recomputing overflow is required only when child content is present.
     703    if (needsSimplifiedNormalFlowLayout() && !m_hasShadowControls) {
     704        clearNeedsLayout();
     705        return;
     706    }
     707
    702708    StackStats::LayoutCheckPoint layoutCheckPoint;
    703709
Note: See TracChangeset for help on using the changeset viewer.