getUserMeida: report device starting states

media/capture/content/screen_capture_device_core.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 "media/capture/content/screen_capture_device_core.h"
 
 #include <memory>
 #include <utility>
 
 #include "base/bind.h"
@@ skipping 107 matching lines @@
     double utilization) {
   DCHECK(thread_checker_.CalledOnValidThread());
   DCHECK(oracle_proxy_);
   oracle_proxy_->OnConsumerReportingUtilization(frame_feedback_id, utilization);
 }
 
 void ScreenCaptureDeviceCore::CaptureStarted(bool success) {
   DCHECK(thread_checker_.CalledOnValidThread());
   if (!success)
     Error(FROM_HERE, "Failed to start capture machine.");
+  else if (oracle_proxy_.get())

[miu, 2017/02/09 22:21:49]

nit: Shouldn't need the ".get()". That's a legacy requirement on the smart
pointers that they fixed a while ago.

[braveyao, 2017/02/14 01:05:49]

Done.

+    oracle_proxy_->ReportStarted();
 }
 
 ScreenCaptureDeviceCore::ScreenCaptureDeviceCore(
     std::unique_ptr<VideoCaptureMachine> capture_machine)
     : state_(kIdle), capture_machine_(std::move(capture_machine)) {
   DCHECK(capture_machine_.get());
 }
 
 ScreenCaptureDeviceCore::~ScreenCaptureDeviceCore() {
   DCHECK(thread_checker_.CalledOnValidThread());
@@ skipping 28 matching lines @@
     return;
 
   if (oracle_proxy_.get())
     oracle_proxy_->ReportError(from_here, reason);
 
   StopAndDeAllocate();
   TransitionStateTo(kError);
 }
 
 }  // namespace media