Fix black video frames when seeking (which also fixes flashing poster issue).

webkit/glue/media/video_renderer_impl.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 "webkit/glue/media/video_renderer_impl.h"
 
 #include "media/base/video_frame.h"
 #include "media/base/yuv_convert.h"
 #include "webkit/glue/webmediaplayer_impl.h"
 
@@ skipping 31 matching lines @@
 
 void VideoRendererImpl::SetWebMediaPlayerImplProxy(
     WebMediaPlayerImpl::Proxy* proxy) {
   proxy_ = proxy;
 }
 
 void VideoRendererImpl::SetRect(const gfx::Rect& rect) {
 }
 
 // This method is always called on the renderer's thread.
+bool VideoRendererImpl::GetCurrentBitmap(SkBitmap* bitmap) {
+  scoped_refptr<media::VideoFrame> video_frame;
+  GetCurrentFrame(&video_frame);
+
+  if (!video_frame) {
+    return false;
+  } else {

[scherkus (not reviewing), 2010/12/17 18:50:40]

nit: de-else/indent this block due to early return

+    if (bitmap->empty() ||
+        bitmap->config() != SkBitmap::kARGB_8888_Config ||
+        bitmap->width() != width() || bitmap->height() != height()) {
+      bitmap->reset();
+      bitmap->setConfig(SkBitmap::kARGB_8888_Config, width(), height());
+      bitmap->allocPixels(NULL, NULL);
+      bitmap->eraseRGB(0xff, 0x00, 0x00);
+    }
+
+    SlowPaintToBitmap(video_frame, bitmap);
+  }
+
+  PutCurrentFrame(video_frame);
+
+  return true;
+}
+
+// This method is always called on the renderer's thread.
 void VideoRendererImpl::Paint(skia::PlatformCanvas* canvas,
                               const gfx::Rect& dest_rect) {
   scoped_refptr<media::VideoFrame> video_frame;
   GetCurrentFrame(&video_frame);
+
   if (!video_frame) {
     SkPaint paint;
     paint.setColor(SK_ColorBLACK);
     canvas->drawRectCoords(
         static_cast<float>(dest_rect.x()),
         static_cast<float>(dest_rect.y()),
         static_cast<float>(dest_rect.right()),
         static_cast<float>(dest_rect.bottom()),
         paint);
   } else {
     if (CanFastPaint(canvas, dest_rect)) {
       FastPaint(video_frame, canvas, dest_rect);
     } else {
-      SlowPaint(video_frame, canvas, dest_rect);
+      SlowPaintToBitmap(video_frame, &bitmap_);
+      CopyBitmapToCanvas(bitmap_, canvas, dest_rect);
     }
 
     // Presentation timestamp logging is primarily used to measure performance
     // on low-end devices.  When profiled on an Intel Atom N280 @ 1.66GHz this
     // code had a ~63 microsecond perf hit when logging to a file (not stdout),
     // which is neglible enough for measuring playback performance.
     if (pts_logging_)
       VLOG(1) << "pts=" << video_frame->GetTimestamp().InMicroseconds();
   }
 
@@ skipping 64 matching lines @@
 
     if (config == SkBitmap::kARGB_8888_Config && device->isOpaque() &&
         device_rect.contains(total_clip.getBounds())) {
       return true;
     }
   }
 
   return false;
 }
 
-void VideoRendererImpl::SlowPaint(media::VideoFrame* video_frame,
-                                  skia::PlatformCanvas* canvas,
-                                  const gfx::Rect& dest_rect) {
+void VideoRendererImpl::SlowPaintToBitmap(media::VideoFrame* video_frame,
+                                          SkBitmap* bitmap) {
   // 1. Convert YUV frame to RGB.

[scherkus (not reviewing), 2010/12/17 18:50:40]

nit: remove the 1. (or the comment itself, your call)

   base::TimeDelta timestamp = video_frame->GetTimestamp();
   if (video_frame != last_converted_frame_ ||
       timestamp != last_converted_timestamp_) {
     last_converted_frame_ = video_frame;
     last_converted_timestamp_ = timestamp;
     DCHECK(video_frame->format() == media::VideoFrame::YV12 ||
            video_frame->format() == media::VideoFrame::YV16);
     DCHECK(video_frame->stride(media::VideoFrame::kUPlane) ==
            video_frame->stride(media::VideoFrame::kVPlane));
     DCHECK(video_frame->planes() == media::VideoFrame::kNumYUVPlanes);
-    bitmap_.lockPixels();
+    bitmap->lockPixels();
     media::YUVType yuv_type =
         (video_frame->format() == media::VideoFrame::YV12) ?
         media::YV12 : media::YV16;
     media::ConvertYUVToRGB32(video_frame->data(media::VideoFrame::kYPlane),
                              video_frame->data(media::VideoFrame::kUPlane),
                              video_frame->data(media::VideoFrame::kVPlane),
-                             static_cast<uint8*>(bitmap_.getPixels()),
+                             static_cast<uint8*>(bitmap->getPixels()),
                              video_frame->width(),
                              video_frame->height(),
                              video_frame->stride(media::VideoFrame::kYPlane),
                              video_frame->stride(media::VideoFrame::kUPlane),
-                             bitmap_.rowBytes(),
+                             bitmap->rowBytes(),
                              yuv_type);
-    bitmap_.unlockPixels();
+    bitmap->unlockPixels();
   }
+}
 
+void VideoRendererImpl::CopyBitmapToCanvas(const SkBitmap& bitmap,
+                                           skia::PlatformCanvas* canvas,
+                                           const gfx::Rect& dest_rect) {
   // 2. Paint the bitmap to canvas.

[scherkus (not reviewing), 2010/12/17 18:50:40]

ditto

   SkMatrix matrix;
   matrix.setTranslate(static_cast<SkScalar>(dest_rect.x()),
                       static_cast<SkScalar>(dest_rect.y()));
-  if (dest_rect.width()  != video_size_.width() ||
-      dest_rect.height() != video_size_.height()) {
+  if (dest_rect.width()  != bitmap_.width() ||
+      dest_rect.height() != bitmap_.height()) {
     matrix.preScale(SkIntToScalar(dest_rect.width()) /
-                    SkIntToScalar(video_size_.width()),
+                    SkIntToScalar(bitmap_.width()),
                     SkIntToScalar(dest_rect.height()) /
-                    SkIntToScalar(video_size_.height()));
+                    SkIntToScalar(bitmap_.height()));
   }
   SkPaint paint;
   paint.setFlags(SkPaint::kFilterBitmap_Flag);
-  canvas->drawBitmapMatrix(bitmap_, matrix, &paint);
+  canvas->drawBitmapMatrix(bitmap, matrix, &paint);
 }
 
 void VideoRendererImpl::FastPaint(media::VideoFrame* video_frame,

[scherkus (not reviewing), 2010/12/17 18:50:40]

nit: maybe rename this to FastPaintToCanvas ?

                                   skia::PlatformCanvas* canvas,
                                   const gfx::Rect& dest_rect) {
   DCHECK(video_frame->format() == media::VideoFrame::YV12 ||
          video_frame->format() == media::VideoFrame::YV16);
   DCHECK(video_frame->stride(media::VideoFrame::kUPlane) ==
          video_frame->stride(media::VideoFrame::kVPlane));
   DCHECK(video_frame->planes() == media::VideoFrame::kNumYUVPlanes);
   const SkBitmap& bitmap = canvas->getDevice()->accessBitmap(true);
   media::YUVType yuv_type = (video_frame->format() == media::VideoFrame::YV12) ?
                             media::YV12 : media::YV16;
@@ skipping 95 matching lines @@
     // Transform destination rect to local coordinates.
     SkRect transformed_rect;
     SkRect skia_dest_rect;
     skia_dest_rect.iset(src_rect.x(), src_rect.y(),
                         src_rect.right(), src_rect.bottom());
     matrix.mapRect(&transformed_rect, skia_dest_rect);
     transformed_rect.round(dest_rect);
 }
 
 }  // namespace webkit_glue