Chromium support for window.webkitRequestAnimationFrame()

webkit/tools/test_shell/test_webview_delegate.cc

 // Copyright (c) 2010 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 // This file contains the implementation of TestWebViewDelegate, which serves
 // as the WebViewDelegate for the TestShellWebHost.  The host is expected to
 // have initialized a MessageLoop before these methods are called.
 
 #include "webkit/tools/test_shell/test_webview_delegate.h"
 
@@ skipping 665 matching lines @@
                                         const WebRect& clip_rect) {
   if (WebWidgetHost* host = GetWidgetHost())
     host->DidScrollRect(dx, dy, clip_rect);
 }
 
 void TestWebViewDelegate::scheduleComposite() {
   if (WebWidgetHost* host = GetWidgetHost())
     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);
 }
 
 void TestWebViewDelegate::didBlur() {
   if (WebWidgetHost* host = GetWidgetHost())
     shell_->SetFocus(host, false);
 }
 
@@ skipping 625 matching lines @@
 }
 
 void TestWebViewDelegate::set_fake_window_rect(const WebRect& rect) {
   fake_rect_ = rect;
   using_fake_rect_ = true;
 }
 
 WebRect TestWebViewDelegate::fake_window_rect() {
   return fake_rect_;
 }