getUserMeida: report device starting states

media/capture/video/android/java/src/org/chromium/media/VideoCaptureCamera.java

Index: media/capture/video/android/java/src/org/chromium/media/VideoCaptureCamera.java
diff --git a/media/capture/video/android/java/src/org/chromium/media/VideoCaptureCamera.java b/media/capture/video/android/java/src/org/chromium/media/VideoCaptureCamera.java
index 929f041150de28ccec086a696f2bc2f219dc9537..db3f6e3cae640953f9ecefdb8b913adea91cb474 100644
--- a/media/capture/video/android/java/src/org/chromium/media/VideoCaptureCamera.java
+++ b/media/capture/video/android/java/src/org/chromium/media/VideoCaptureCamera.java
@@ -400,10 +400,10 @@ public class VideoCaptureCamera
             if (mIsRunning) {
                 return true;
             }
-            mIsRunning = true;
         } finally {
             mPreviewBufferLock.unlock();
         }
+
         setPreviewCallback(this);
         try {
             mCamera.startPreview();
@@ -411,6 +411,14 @@ public class VideoCaptureCamera
             Log.e(TAG, "startCapture: Camera.startPreview: " + ex);
             return false;
         }
+
+        mPreviewBufferLock.lock();
+        try {
+            nativeOnStarted(mNativeVideoCaptureDeviceAndroid);
+            mIsRunning = true;

[chfremer, 2017/02/23 17:49:53]

Why are we moving the mIsRunning = true down here? I thought this was only
needed if the contract was that no frames are allowed to be sent before
OnStarted(). Now that we agreed to allow frames being sent before OnStarted(),
this should no longer be needed. Couldn't we just call nativeOnStarted() in line
410 right after mCamera.startPreview()?

[braveyao, 2017/02/23 18:08:09]

If startPreview fails, it's not right to set mIsRunning = true.

[chfremer, 2017/02/23 19:07:08]

Ah, I see. Then this is a fix for a bug that existed before and is unrelated to
this CL? If so, should we create a bug entry for visibility and tracking?

+        } finally {
+            mPreviewBufferLock.unlock();
+        }
         return true;
     }