Webcam ppapi flash fix.

Return buffers that are in flight during shutdown and make sure to issue 
defered stop.

BUG=106369
Review URL: http://codereview.chromium.org/8819010

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@113467 0039d316-1c4b-4281-b951-d872f2087c98

content/browser/renderer_host/media/video_capture_controller.cc

@@ -222,6 +222,14 @@ void VideoCaptureController::ReturnBuffer(
     client->event_handler->OnReadyToDelete(client->controller_id);
     delete client;
     controller_clients_.remove(client);
+
+    if (controller_clients_.empty()) {
+       // No more clients. Stop device.
+       video_capture_manager_->Stop(current_params_.session_id,
+          base::Bind(&VideoCaptureController::OnDeviceStopped, this));
+      frame_info_available_ = false;
+      state_ = video_capture::kStopping;
+    }
   }
   {
     base::AutoLock lock(lock_);

webkit/plugins/ppapi/ppb_video_capture_impl.cc

@@ -209,24 +209,25 @@ void PPB_VideoCapture_Impl::OnRemoved(media::VideoCapture* capture) {
 void PPB_VideoCapture_Impl::OnBufferReady(
     media::VideoCapture* capture,
     scoped_refptr<media::VideoCapture::VideoFrameBuffer> buffer) {
-  if (is_dead_)
-    return;
-
-  DCHECK(buffer.get());
-  for (uint32_t i = 0; i < buffers_.size(); ++i) {
-    if (!buffers_[i].in_use) {
-      // TODO(piman): it looks like stride isn't actually used/filled.
-      DCHECK(buffer->stride == 0);
-      size_t size = std::min(static_cast<size_t>(buffers_[i].buffer->size()),
-          buffer->buffer_size);
-      memcpy(buffers_[i].data, buffer->memory_pointer, size);
-      buffers_[i].in_use = true;
-      platform_video_capture_->FeedBuffer(buffer);
-      ppp_videocapture_->OnBufferReady(pp_instance(), pp_resource(), i);
-      return;
+  if (!is_dead_) {
+    DCHECK(buffer.get());
+    for (uint32_t i = 0; i < buffers_.size(); ++i) {
+      if (!buffers_[i].in_use) {
+        // TODO(ihf): Switch to a size calculation based on stride.
+        // Stride is filled out now but not more meaningful than size
+        // until wjia unifies VideoFrameBuffer and media::VideoFrame.
+        size_t size = std::min(static_cast<size_t>(buffers_[i].buffer->size()),
+            buffer->buffer_size);
+        memcpy(buffers_[i].data, buffer->memory_pointer, size);
+        buffers_[i].in_use = true;
+        platform_video_capture_->FeedBuffer(buffer);
+        ppp_videocapture_->OnBufferReady(pp_instance(), pp_resource(), i);
+        return;
+      }
     }
   }
-  // TODO(piman): signal dropped buffers ?
+  // Even after we have stopped and are dead we have to return buffers that
+  // are in flight to us. Otherwise VideoCaptureController will not tear down.
   platform_video_capture_->FeedBuffer(buffer);
 }