getUserMeida: report device starting states

media/capture/content/android/screen_capture_machine_android.cc

 // Copyright 2016 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/android/screen_capture_machine_android.h"
 
 #include "base/android/context_utils.h"
 #include "base/android/jni_android.h"
 #include "base/android/scoped_java_ref.h"
 #include "jni/ScreenCapture_jni.h"
@@ skipping 178 matching lines @@
 }
 
 void ScreenCaptureMachineAndroid::OnActivityResult(JNIEnv* env,
                                                    jobject obj,
                                                    jboolean result) {
   if (!result) {
     oracle_proxy_->ReportError(FROM_HERE, "The user denied screen capture");
     return;
   }
 
-  Java_ScreenCapture_startCapture(env, obj);
+  if (Java_ScreenCapture_startCapture(env, obj))
+    oracle_proxy_->ReportStarted();
+  else
+    oracle_proxy_->ReportError(FROM_HERE, "Failed to start Screen Capture");
 }
 
 void ScreenCaptureMachineAndroid::OnOrientationChange(JNIEnv* env,
                                                       jobject obj,
                                                       jint rotation) {
   DeviceOrientation orientation = kDefault;
   switch (rotation) {
     case 0:
     case 180:
       orientation = kPortrait;
@@ skipping 41 matching lines @@
       Java_ScreenCapture_allocate(AttachCurrentThread(), j_capture_,
                                   params.requested_format.frame_size.width(),
                                   params.requested_format.frame_size.height());
   if (!ret) {
     DLOG(ERROR) << "Failed to init ScreenCaptureAndroid";
     callback.Run(ret);
     return;
   }
 
   ret = Java_ScreenCapture_startPrompt(AttachCurrentThread(), j_capture_);
-
-  callback.Run(ret);
+  // On Android, we must wait for user input to start capturing before we can
+  // report back device started state.

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

Maybe add: However, if the user-prompt failed to show, report a failed start
immediately.

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

Done.

+  if (!ret)
+    callback.Run(ret);
 }
 
 void ScreenCaptureMachineAndroid::Stop(const base::Closure& callback) {
   if (j_capture_.obj() != nullptr) {
     Java_ScreenCapture_stopCapture(AttachCurrentThread(), j_capture_);
   }
 
   callback.Run();
 }
 
@@ skipping 31 matching lines @@
       frame->visible_data(VideoFrame::kUPlane),
       frame->stride(VideoFrame::kUPlane),
       frame->visible_data(VideoFrame::kVPlane),
       frame->stride(VideoFrame::kVPlane), frame->visible_rect().width(),
       frame->visible_rect().height(), libyuv::kFilterBilinear);
 
   capture_frame_cb.Run(frame, start_time, true);
 }
 
 }  // namespace media