[Webkit-unassigned] [Bug 95480] Repaints should not be deferred after initial page load is complete

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=95480


Simon Fraser (smfr) <simon.fraser at apple.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #161727|review?                     |review+
               Flag|                            |




--- Comment #3 from Simon Fraser (smfr) <simon.fraser at apple.com>  2012-08-31 10:41:30 PST ---
(From update of attachment 161727)
View in context: https://bugs.webkit.org/attachment.cgi?id=161727&action=review

> Source/WebCore/ChangeLog:21
> +        (WebCore):

You can remove this line.

> Source/WebCore/loader/FrameLoader.cpp:758
> +        m_frame->view()->checkStopDelayingDeferredRepaints();

I have a hard time parsing this function name. Maybe we can rename it?

> Source/WebCore/page/FrameView.cpp:104
> +double FrameView::s_normalDeferredRepaintDelay = 0.025;

Should we make this 16ms to avoid crapping on animations?

> Source/WebCore/page/FrameView.cpp:2059
> +    double increamentedRepaintDelay = m_deferredRepaintDelay + s_deferredRepaintDelayIncrementDuringLoading;

increamentedRepaintDelay -> incrementedRepaintDelay

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.