[webkit-reviews] review denied: [Bug 87301] [chromium] Make single-thread proxy tick animations and use the main thread tree host like multi-thread proxy does : [Attachment 143631] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed May 23 13:34:55 PDT 2012


James Robinson <jamesr at chromium.org> has denied Dana Jansens
<danakj at chromium.org>'s request for review:
Bug 87301: [chromium] Make single-thread proxy tick animations and use the main
thread tree host like multi-thread proxy does
https://bugs.webkit.org/show_bug.cgi?id=87301

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

------- Additional Comments from James Robinson <jamesr at chromium.org>
I don't understand this at all.  content::RenderWidget is calling
willBeginFrame()/updateAnimations()/layout() - you want to call them again? 
This patch feels incomplete, at a minimum.


More information about the webkit-reviews mailing list