Chromium support for webkitAnimationTime property

chrome/renderer/render_widget.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.
 
 #include "chrome/renderer/render_widget.h"
 
 #include "app/surface/transport_dib.h"
 #include "base/command_line.h"
 #include "base/logging.h"
 #include "base/message_loop.h"
@@ skipping 548 matching lines @@
 
     for (size_t i = 0; i < copy_rects.size(); ++i)
       PaintRect(copy_rects[i], bounds.origin(), canvas.get());
 
     dib_id = current_paint_buf_->id();
   } else {  // Accelerated compositing path
     // Begin painting.
     webwidget_->composite(false);
   }
 
+  webwidget_->clearCurrentAnimationTime();

[darin (slow to review), 2011/01/06 05:52:29]

what about other places where we call WebWidget::paint(), like
PaintViewIntoCanvas() in render_view.cc?

it is unfortunate that we need to expose such a low-level detail to the embedder
here.  <half-baked> it would be neat if we had beginPaint() and endPaint()
functions wrapping the set of paint() calls.  then we could hide clearing of the
current animation time inside endPaint(), and maybe layout() and animate() steps
could be hidden in beginPaint(). </half-baked>

+
   // sending an ack to browser process that the paint is complete...
   ViewHostMsg_UpdateRect_Params params;
   params.bitmap = dib_id;
   params.bitmap_rect = bounds;
   params.dx = update.scroll_delta.x();
   params.dy = update.scroll_delta.y();
   if (is_accelerated_compositing_active_) {
     // If painting is done via the gpu process then we clear out all damage
     // rects to save the browser process from doing unecessary work.
     params.scroll_rect = gfx::Rect();
@@ skipping 442 matching lines @@
 
 void RenderWidget::CleanupWindowInPluginMoves(gfx::PluginWindowHandle window) {
   for (WebPluginGeometryVector::iterator i = plugin_window_moves_.begin();
        i != plugin_window_moves_.end(); ++i) {
     if (i->window == window) {
       plugin_window_moves_.erase(i);
       break;
     }
   }
 }