[webkit-reviews] review granted: [Bug 76063] Suspend/Resume API for pausing timers and animations : [Attachment 123294] WebKit2 side of patch

[bugzilla-daemon at webkit.org]

Kenneth Rohde Christiansen <kenneth at webkit.org> has granted Allan Sandfeld
<sandfeld at kde.org>'s request for review:
Bug 76063: Suspend/Resume API for pausing timers and animations
https://bugs.webkit.org/show_bug.cgi?id=76063

Attachment 123294: WebKit2 side of patch
https://bugs.webkit.org/attachment.cgi?id=123294&action=review

------- Additional Comments from Kenneth Rohde Christiansen
<kenneth at webkit.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=123294&action=review


This patch (WebKit2 side) looks good to me

> Source/WebKit2/WebProcess/WebPage/WebPage.cpp:1623
> +	   // Suspend CSS animations.
> +	   frame->animation()->suspendAnimations();

I dont think this comment gives any value, it is pretty obvious already