RESOLVED FIXED 70160
[chromium] Route requestAnimationFrame through CCProxy in threaded mode
https://bugs.webkit.org/show_bug.cgi?id=70160
Summary [chromium] Route requestAnimationFrame through CCProxy in threaded mode
Nat Duca
Reported 2011-10-14 17:15:19 PDT
[chromium] Route requestAnimationFrame through CCProxy in threaded mode
Attachments
Patch (11.58 KB, patch)
2011-10-14 17:17 PDT, Nat Duca
no flags
Patch for landing (11.63 KB, patch)
2011-10-19 15:18 PDT, Nat Duca
no flags
Nat Duca
Comment 1 2011-10-14 17:17:21 PDT
James Robinson
Comment 2 2011-10-17 15:24:47 PDT
Comment on attachment 111112 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=111112&action=review R=me - have some nits to address while landing. > Source/WebCore/platform/graphics/chromium/cc/CCSingleThreadProxy.cpp:144 > + ASSERT(0); we have ASSERT_NOT_REACHED() for this > Source/WebKit/chromium/src/ChromeClientImpl.cpp:542 > +#if USE(ACCELERATED_COMPOSITING) > + if (!m_webView->isAcceleratedCompositingActive()) > +#endif > + m_webView->client()->scheduleAnimation(); > +#if USE(ACCELERATED_COMPOSITING) > + else > + m_webView->scheduleAnimation(); > +#endif i think things would be slightly cleaner if ChromeClientImpl::scheduleAnimation() simply delegated to m_webView->scheduleAnimation() and all of the #if USE(ACCELE...) etc crap was inside WebViewImpl.cpp. WebViewImpl::scheduleAnimation() call call out to its client if it needs to > Source/WebKit/chromium/src/WebViewImpl.h:-384 > - void doUpdateAndComposite(); guessing this function wasn't actually defined or referenced before?
Nat Duca
Comment 3 2011-10-17 18:10:19 PDT
All great points. Thanks... (In reply to comment #2)
Nat Duca
Comment 4 2011-10-19 15:18:37 PDT
Created attachment 111681 [details] Patch for landing
WebKit Review Bot
Comment 5 2011-10-19 19:22:41 PDT
Comment on attachment 111681 [details] Patch for landing Clearing flags on attachment: 111681 Committed r97922: <http://trac.webkit.org/changeset/97922>
WebKit Review Bot
Comment 6 2011-10-19 19:22:46 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.