[VideoCapture] Limit logging of first frame in VideoCaptureImpl

Bug: 974149
Change-Id: Ie4753ea473fbfd8448d772c1e07d440e74085ace
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2377718Reviewed-by: Guido Urdaneta <guidou@chromium.org>
Commit-Queue: Palak Agarwal <agpalak@chromium.org>
Cr-Commit-Position: refs/heads/master@{#802885}

third_party/blink/renderer/platform/video_capture/video_capture_impl.cc

@@ -37,6 +37,8 @@
 
 namespace blink {
 
+constexpr int kMaxFirstFrameLogs = 5;
+
 using VideoFrameBufferHandleType = media::mojom::blink::VideoBufferHandle::Tag;
 
 // A collection of all types of handles that we use to reference a camera buffer
@@ -514,7 +516,15 @@ void VideoCaptureImpl::OnBufferReady(
 
   if (first_frame_ref_time_.is_null()) {
     first_frame_ref_time_ = reference_time;
-    OnLog("First frame received at VideoCaptureImpl");
+    if (num_first_frame_logs_ < kMaxFirstFrameLogs) {
+      OnLog("First frame received for this VideoCaptureImpl instance");
+      num_first_frame_logs_++;
+    } else if (num_first_frame_logs_ == kMaxFirstFrameLogs) {
+      OnLog(
+          "First frame received for this VideoCaptureImpl instance. This will "
+          "not be logged anymore for this VideoCaptureImpl instance.");
+      num_first_frame_logs_++;
+    }
   }
 
   // If the timestamp is not prepared, we use reference time to make a rough

third_party/blink/renderer/platform/video_capture/video_capture_impl.h

@@ -174,6 +174,8 @@ class PLATFORM_EXPORT VideoCaptureImpl
 
   VideoCaptureState state_;
 
+  int num_first_frame_logs_ = 0;
+
   // Methods of |gpu_factories_| need to run on |media_task_runner_|.
   media::GpuVideoAcceleratorFactories* gpu_factories_;
   scoped_refptr<base::SingleThreadTaskRunner> media_task_runner_;