Remove unused parameters from MediaStreamVideoSource::StartCaptureImpl()

content/renderer/media/media_stream_video_source.h

Index: content/renderer/media/media_stream_video_source.h
diff --git a/content/renderer/media/media_stream_video_source.h b/content/renderer/media/media_stream_video_source.h
index 4e3139c05f952e3c365ae55f1e0dcf32488ab8a2..235f5fab3eccf08aeabd7037d8953ea578120f19 100644
--- a/content/renderer/media/media_stream_video_source.h
+++ b/content/renderer/media/media_stream_video_source.h
@@ -93,21 +93,15 @@ class CONTENT_EXPORT MediaStreamVideoSource : public MediaStreamSource {
   // Sets muted state and notifies it to all registered tracks.
   virtual void SetMutedState(bool state);
 
-  // TODO(guidou): Rename to GetCurrentFormat. http://crbug.com/706804
+  // An implementation must return its capture format if available.
   virtual base::Optional<media::VideoCaptureFormat> GetCurrentFormatImpl()
       const;

[hbos_chromium, 2017/07/04 14:13:51]

Optional: Make the public GetCurrentFormat virtual and remove this one, since
GetCurrentFormat just calls GetCurrentFromatImpl.

[Guido Urdaneta, 2017/07/04 14:29:26]

Done. See follow-up CL https://codereview.chromium.org/2970823003/

 
-  // An implementation must start capturing frames using the requested
-  // |format|. The fulfilled |constraints| are provided as additional context,
-  // and may be used to modify the behavior of the source. When the source has
-  // started or the source failed to start OnStartDone must be called. An
-  // implementation must call |frame_callback| on the IO thread with the
+  // An implementation must start capturing frames after this method is called.
+  // When the source has started or failed to start OnStartDone must be called.
+  // An implementation must call |frame_callback| on the IO thread with the
   // captured frames.
-  // TODO(guidou): Remove |format| and |constraints| parameters.
-  // http://crbug.com/706408
   virtual void StartSourceImpl(
-      const media::VideoCaptureFormat& format,
-      const blink::WebMediaConstraints& constraints,
       const VideoCaptureDeliverFrameCB& frame_callback) = 0;
   void OnStartDone(MediaStreamRequestResult result);