getUserMedia: handle the device starting status report.

content/browser/renderer_host/media/video_capture_controller.h

 // Copyright (c) 2012 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.
 
 #ifndef CONTENT_BROWSER_RENDERER_HOST_MEDIA_VIDEO_CAPTURE_CONTROLLER_H_
 #define CONTENT_BROWSER_RENDERER_HOST_MEDIA_VIDEO_CAPTURE_CONTROLLER_H_
 
 #include <list>
 #include <memory>
 
@@ skipping 173 matching lines @@
   // state which stops the flow of data to clients.
   VideoCaptureState state_;
 
   int next_buffer_context_id_ = 0;
 
   // True if the controller has received a video frame from the device.
   bool has_received_frames_;
 
   media::VideoCaptureFormat video_capture_format_;
 
+  // True if OnStarted has been reported from the device.
+  bool has_received_onstarted_;

[miu, 2017/03/06 22:37:33]

Instead of adding this separate boolean, how about adding a VideoCaptureState
called VIDEO_CAPTURE_STATE_STARTING? This seems to be an actual concept that is
known and used in both the browser and renderer code.

https://cs.chromium.org/chromium/src/content/common/media/video_capture.h?rcl...

[braveyao, 2017/03/08 22:02:52]

Done.

+
   base::WeakPtrFactory<VideoCaptureController> weak_ptr_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(VideoCaptureController);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_RENDERER_HOST_MEDIA_VIDEO_CAPTURE_CONTROLLER_H_