getUserMeida: report device starting states

media/capture/video/win/video_capture_device_mf_win.cc

 // 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.
 
 #include "media/capture/video/win/video_capture_device_mf_win.h"
 
 #include <mfapi.h>
 #include <mferror.h>
 #include <stddef.h>
 
@@ skipping 221 matching lines @@
       ScopedComPtr<IMFMediaType> type;
       hr = reader_->GetNativeMediaType(
           kFirstVideoStream, found_capability.stream_index, type.Receive());
       if (SUCCEEDED(hr)) {
         hr = reader_->SetCurrentMediaType(kFirstVideoStream, NULL, type.get());
         if (SUCCEEDED(hr)) {
           hr =
               reader_->ReadSample(kFirstVideoStream, 0, NULL, NULL, NULL, NULL);
           if (SUCCEEDED(hr)) {
             capture_format_ = found_capability.supported_format;
+            client_->OnStarted();
             capture_ = true;
             return;
           }
         }
       }
     }
   }
 
   OnError(FROM_HERE, hr);
 }
@@ skipping 27 matching lines @@
     flushed.TimedWait(base::TimeDelta::FromMilliseconds(kFlushTimeOutInMs));
 }
 
 void VideoCaptureDeviceMFWin::OnIncomingCapturedData(
     const uint8_t* data,
     int length,
     int rotation,
     base::TimeTicks reference_time,
     base::TimeDelta timestamp) {
   base::AutoLock lock(lock_);
+  if (!capture_)
+    return;
+
   if (data && client_.get()) {
     client_->OnIncomingCapturedData(data, length, capture_format_, rotation,
                                     reference_time, timestamp);
   }
 
-  if (capture_) {
-    HRESULT hr =
-        reader_->ReadSample(kFirstVideoStream, 0, NULL, NULL, NULL, NULL);
-    if (FAILED(hr)) {
-      // If running the *VideoCap* unit tests on repeat, this can sometimes
-      // fail with HRESULT_FROM_WINHRESULT_FROM_WIN32(ERROR_INVALID_FUNCTION).
-      // It's not clear to me why this is, but it is possible that it has
-      // something to do with this bug:
-      // http://support.microsoft.com/kb/979567
-      OnError(FROM_HERE, hr);
-    }
+  HRESULT hr =
+      reader_->ReadSample(kFirstVideoStream, 0, NULL, NULL, NULL, NULL);
+  if (FAILED(hr)) {
+    // If running the *VideoCap* unit tests on repeat, this can sometimes
+    // fail with HRESULT_FROM_WINHRESULT_FROM_WIN32(ERROR_INVALID_FUNCTION).
+    // It's not clear to me why this is, but it is possible that it has
+    // something to do with this bug:
+    // http://support.microsoft.com/kb/979567
+    OnError(FROM_HERE, hr);
   }
 }
 
 void VideoCaptureDeviceMFWin::OnError(
     const tracked_objects::Location& from_here,
     HRESULT hr) {
   if (client_.get()) {
     client_->OnError(
         from_here,
         base::StringPrintf("VideoCaptureDeviceMFWin: %s",
                            logging::SystemErrorCodeToString(hr).c_str()));
   }
 }
 
 }  // namespace media