Changeset 180846 in webkit


Ignore:
Timestamp:
Feb 28, 2015 5:04:05 PM (9 years ago)
Author:
Simon Fraser
Message:

FrameView::layoutTimerFired() should update style if needed before doing layout
https://bugs.webkit.org/show_bug.cgi?id=141688

Reviewed by Andreas Kling.

If the style recalc timer has been scheduled to fire after the layout timer,
when the layout timer fires, we might as well just do the style recalc
too. The call to updateStyleIfNeeded() will cancel the pending style
recalc timer.

This doesn't have much impact on the number of layouts (measured via PLT)
but seems like a reasonable thing to do.

  • page/FrameView.cpp:

(WebCore::FrameView::layoutTimerFired):

Location:
trunk/Source/WebCore
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • trunk/Source/WebCore/ChangeLog

    r180839 r180846  
     12015-02-28  Simon Fraser  <simon.fraser@apple.com>
     2
     3        FrameView::layoutTimerFired() should update style if needed before doing layout
     4        https://bugs.webkit.org/show_bug.cgi?id=141688
     5
     6        Reviewed by Andreas Kling.
     7
     8        If the style recalc timer has been scheduled to fire after the layout timer,
     9        when the layout timer fires, we might as well just do the style recalc
     10        too. The call to updateStyleIfNeeded() will cancel the pending style
     11        recalc timer.
     12       
     13        This doesn't have much impact on the number of layouts (measured via PLT)
     14        but seems like a reasonable thing to do.
     15
     16        * page/FrameView.cpp:
     17        (WebCore::FrameView::layoutTimerFired):
     18
    1192015-02-28  Simon Fraser  <simon.fraser@apple.com>
    220
  • trunk/Source/WebCore/page/FrameView.cpp

    r180615 r180846  
    24402440        printf("Layout timer fired at %lld\n", frame().document()->elapsedTime().count());
    24412441#endif
     2442    frame().document()->updateStyleIfNeeded();
    24422443    layout();
    24432444}
Note: See TracChangeset for help on using the changeset viewer.