[Webkit-unassigned] [Bug 90401] Compositing layer sync should cause deferred repaints to be fired immediately

[bugzilla-daemon at webkit.org]

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


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

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #150475|review?                     |review+, commit-queue-
               Flag|                            |




--- Comment #3 from Simon Fraser (smfr) <simon.fraser at apple.com>  2012-07-02 14:51:42 PST ---
(From update of attachment 150475)
View in context: https://bugs.webkit.org/attachment.cgi?id=150475&action=review

> Source/WebCore/page/FrameView.cpp:752
>      root->compositor()->flushPendingLayerChanges(rootFrameForSync == m_frame);
>  
> +    // If we sync compositing layers and allow the repaint to be deferred, there is time for a
> +    // visible flash to occur. Instead, stop the deferred repaint timer and repaint immediately.
> +    stopDelayingDeferredRepaints();

I think we should swap these around.

If deferred repaints at some point affects compositing layers (it does not yet), then we'd want those repaints to get flushed.

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