Chromium support for window.webkitRequestAnimationFrame()

webkit/tools/test_shell/test_webview_delegate.cc

Index: webkit/tools/test_shell/test_webview_delegate.cc
diff --git a/webkit/tools/test_shell/test_webview_delegate.cc b/webkit/tools/test_shell/test_webview_delegate.cc
index fccf8e7d78f0bd03f2cd05c8a382449081aef1be..4f391373da8d212a5b4176acc0dbacd8594c22cc 100644
--- a/webkit/tools/test_shell/test_webview_delegate.cc
+++ b/webkit/tools/test_shell/test_webview_delegate.cc
@@ -683,6 +683,11 @@ void TestWebViewDelegate::scheduleComposite() {
     host->ScheduleComposite();
 }
 
+void TestWebViewDelegate::scheduleAnimation() {
+    MessageLoop::current()->PostTask(FROM_HERE, NewRunnableMethod(

[darin (slow to review), 2011/01/11 22:02:30]

nit: indentation

Piggy backing on scheduleComposite here might cause problems in the future if
scheduleComposite gets "fixed" to trigger a call to WebWidget::composite(). 
Maybe we should plumb a scheduleAnimation instead and then just make it an
implementation detail of the WebWidgetHost that scheduleAnimation and
scheduleComposite do the same thing?

[jamesr, 2011/01/12 00:02:43]

It's a pain to do deferred work in WebWidgetHost since it isn't
ThreadSafeRefCounted (so I can't use PostTask()).  I don't think anyone is
planning to support real composition in test_shell any time soon, so I think
(hope) the concern is mostly theoretical.

+                this, &TestWebViewDelegate::scheduleComposite));
+}
+
 void TestWebViewDelegate::didFocus() {
   if (WebWidgetHost* host = GetWidgetHost())
     shell_->SetFocus(host, true);