VideoCapturerDelegate: Retrieve supported/in-use format(s) for constraint negotiation

content/renderer/media/media_stream_video_capturer_source.cc

 // Copyright 2014 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 "content/renderer/media/media_stream_video_capturer_source.h"
 
 #include "base/bind.h"
 #include "base/location.h"
 #include "content/renderer/media/video_capture_impl_manager.h"
 #include "content/renderer/render_thread_impl.h"
@@ skipping 23 matching lines @@
 namespace content {
 
 VideoCapturerDelegate::VideoCapturerDelegate(
     const StreamDeviceInfo& device_info)
     : session_id_(device_info.session_id),
       capture_engine_(
           RenderThreadImpl::current()->video_capture_impl_manager()
               ->UseDevice(device_info.session_id)),
       is_screen_cast_(device_info.device.type == MEDIA_TAB_VIDEO_CAPTURE ||
                       device_info.device.type == MEDIA_DESKTOP_VIDEO_CAPTURE),
-      got_first_frame_(false) {
+      got_first_frame_(false),
+      source_formats_callback_() {

[perkj_chrome, 2014/03/12 19:55:48]

no need to initialize this here.

[mcasas, 2014/03/13 08:04:56]

Done.

   DVLOG(3) << "VideoCapturerDelegate::ctor";
   DCHECK(capture_engine_);
 }
 
 VideoCapturerDelegate::~VideoCapturerDelegate() {
   DVLOG(3) << "VideoCapturerDelegate::dtor";
   DCHECK(new_frame_callback_.is_null());
 }
 
 void VideoCapturerDelegate::GetCurrentSupportedFormats(
@@ skipping 12 matching lines @@
         max_requested_height : MediaStreamVideoSource::kDefaultHeight;
     formats.push_back(
           media::VideoCaptureFormat(
               gfx::Size(width, height),
               MediaStreamVideoSource::kDefaultFrameRate,
               media::PIXEL_FORMAT_I420));
     callback.Run(formats);
     return;
   }
 
-  // This delegate implementation doesn't support capability enumeration.
-  // We need to guess what it supports.
-  media::VideoCaptureFormats formats;
-  for (size_t i = 0; i < arraysize(kVideoFormats); ++i) {
-    formats.push_back(
-        media::VideoCaptureFormat(
-            gfx::Size(kVideoFormats[i].width,
-                      kVideoFormats[i].height),
-            kVideoFormats[i].frame_rate,
-            media::PIXEL_FORMAT_I420));
-  }
-  callback.Run(formats);
+  DCHECK(source_formats_callback_.is_null());
+  source_formats_callback_ = callback;
+  RenderThreadImpl::current()->video_capture_impl_manager()

[perkj_chrome, 2014/03/12 19:55:48]

No... Do
capture_engine_->GetDeviceFormatInUse(base::Bind(&VideoCapturerDelegate::OnDeviceFormatsInUseReceived,
this);

Ie, do not use unretained, unretained is a promise that VideoCapturerDelegate
will not be deleted while waiting for the callback. If you only use this,
base::Bind will add a ref to VideoCaptureDelegate and thus ensure that the
object will not be deleted.

Since we already have capture_engine_, use it.

[mcasas, 2014/03/13 08:04:56]

Done.

+      ->UseDevice(session_id_)->GetDeviceFormatsInUse(
+          base::Bind(&VideoCapturerDelegate::OnDeviceFormatsInUseReceived,
+                     base::Unretained(this)));
 }
 
 void VideoCapturerDelegate::StartDeliver(
     const media::VideoCaptureParams& params,
     const NewFrameCallback& new_frame_callback,
     const StartedCallback& started_callback) {
   DCHECK(params.requested_format.IsValid());
   message_loop_proxy_ = base::MessageLoopProxy::current();
   new_frame_callback_ = new_frame_callback;
   started_callback_ = started_callback;
   got_first_frame_ = false;
 
   // Increase the reference count to ensure the object is not deleted until
   // it is unregistered in VideoCapturerDelegate::OnRemoved.
   AddRef();
   capture_engine_->StartCapture(this, params);
 }
 
 void VideoCapturerDelegate::StopDeliver() {
   // Immediately make sure we don't provide more frames.
   DVLOG(3) << "VideoCapturerDelegate::StopCapture()";
   DCHECK(message_loop_proxy_ == base::MessageLoopProxy::current());
   capture_engine_->StopCapture(this);
   new_frame_callback_.Reset();
   started_callback_.Reset();

[perkj_chrome, 2014/03/12 19:55:48]

Do source_formats_callback_.Reset() here to make sure source_formats_callback_
is not triggered after StopDeliver is called. Please see the comment in
MediaStreamVideoSource.h on StopSourceImpl.

[mcasas, 2014/03/13 08:04:56]

Done.

 }
 
+void VideoCapturerDelegate::OnDeviceFormatsInUseReceived(
+    const media::VideoCaptureFormats& formats_in_use) {
+  DVLOG(3) << "OnDeviceFormatsInUseReceived: " << formats_in_use.size();
+  if(!formats_in_use.empty()) {
+    source_formats_callback_.Run(formats_in_use);

[perkj_chrome, 2014/03/12 19:55:48]

check for isnull.

[mcasas, 2014/03/13 08:04:56]

Done.

+  }else {
+    // If there are no formats in use, try to retrieve the whole list of them.
+    RenderThreadImpl::current()->video_capture_impl_manager()

[perkj_chrome, 2014/03/12 19:55:48]

Same as above.

[mcasas, 2014/03/13 08:04:56]

Done.

+        ->UseDevice(session_id_)->GetDeviceSupportedFormats(base::Bind(
+            &VideoCapturerDelegate::OnDeviceSupportedFormatsEnumerated,
+            base::Unretained(this)));
+  }
+}
+
+void VideoCapturerDelegate::OnDeviceSupportedFormatsEnumerated(
+    const media::VideoCaptureFormats& formats) {
+  DVLOG(3) << "OnDeviceSupportedFormatsEnumerated: " << formats.size()
+           << " received";
+  if(formats.size()) {
+    source_formats_callback_.Run(formats);

[perkj_chrome, 2014/03/12 19:55:48]

check for isnull.

[mcasas, 2014/03/13 08:04:56]

Done.

+  } else {
+    // The capture device doesn't seem to support capability enumeration,
+    // compose a fallback list of capabilities.
+    media::VideoCaptureFormats default_formats;
+    for (size_t i = 0; i < arraysize(kVideoFormats); ++i) {
+      default_formats.push_back(media::VideoCaptureFormat(
+          gfx::Size(kVideoFormats[i].width, kVideoFormats[i].height),
+          kVideoFormats[i].frame_rate,
+          media::PIXEL_FORMAT_I420));
+    }
+    source_formats_callback_.Run(default_formats);
+  }
+}
+
 void VideoCapturerDelegate::OnStarted(media::VideoCapture* capture) {
   DVLOG(3) << "VideoCapturerDelegate::OnStarted";
 }
 
 void VideoCapturerDelegate::OnStopped(media::VideoCapture* capture) {
 }
 
 void VideoCapturerDelegate::OnPaused(media::VideoCapture* capture) {
 }
 
@@ skipping 78 matching lines @@
                  base::Unretained(this)),
       base::Bind(&MediaStreamVideoCapturerSource::OnStartDone,
                  base::Unretained(this)));
 }
 
 void MediaStreamVideoCapturerSource::StopSourceImpl() {
   delegate_->StopDeliver();
 }
 
 }  // namespace content