[webkit-reviews] review granted: [Bug 208168] Remove render update throttling : [Attachment 391679] Patch

[bugzilla-daemon at webkit.org]

zalan <zalan at apple.com> has granted Ben Nham <nham at apple.com>'s request for
review:
Bug 208168: Remove render update throttling
https://bugs.webkit.org/show_bug.cgi?id=208168

Attachment 391679: Patch

https://bugs.webkit.org/attachment.cgi?id=391679&action=review




--- Comment #11 from zalan <zalan at apple.com> ---
Comment on attachment 391679
  --> https://bugs.webkit.org/attachment.cgi?id=391679
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=391679&action=review

>
Source/WebKit/WebProcess/WebPage/RemoteLayerTree/RemoteLayerTreeDrawingArea.mm:
197
> +	   scheduleRenderingUpdate();

shouldn't the initial paint always be immediate
(scheduleImmediateRenderingUpdate())? I wonder if m_updateRenderingTimer is
ever active in this case and if it is whether overriding it with a 0ms timeout
is actually better.