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..d2597a5e6b4bb010ee2b46b8dd94fcdade7d65ec 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,7 +400,6 @@ public class VideoCaptureCamera
             if (mIsRunning) {
                 return true;
             }
-            mIsRunning = true;
         } finally {
             mPreviewBufferLock.unlock();
         }
@@ -411,6 +410,8 @@ public class VideoCaptureCamera
             Log.e(TAG, "startCapture: Camera.startPreview: " + ex);
             return false;
         }
+        nativeOnStarted(mNativeVideoCaptureDeviceAndroid);
+        mIsRunning = true;

[chfremer, 2017/02/17 23:51:15]

I am not sure we can move mIsRunning = true out from the scope that has
mPreviewBufferLock locked.
Otherwise, setting mIsRunning = true may happen concurrently with the check if
(!mIsRunning) in method onPreviewFrame(). I don't know Java well enough to know
whether or not bools are safe to use concurrently. Could you please confirm that
this is intended and valid?

[braveyao, 2017/02/22 18:06:44]

Done.

         return true;
     }