[webkit-reviews] review granted: [Bug 92323] Plumb vsync-enabled flag up to compositor thread and support disable-vsync : [Attachment 155093] Patch

[bugzilla-daemon at webkit.org]

James Robinson <jamesr at chromium.org> has granted John Bates
<jbates at google.com>'s request for review:
Bug 92323: Plumb vsync-enabled flag up to compositor thread and support
disable-vsync
https://bugs.webkit.org/show_bug.cgi?id=92323

Attachment 155093: Patch
https://bugs.webkit.org/attachment.cgi?id=155093&action=review

------- Additional Comments from James Robinson <jamesr at chromium.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=155093&action=review


R=me, just one question for you before setting cq?

> Source/WebCore/platform/graphics/chromium/cc/CCFrameRateController.cpp:91
> +	   postManualTick();

do we need to post a tick if we aren't active?