Voice input

remoting/android/java/src/org/chromium/chromoting/CardboardDesktopRenderer.java

Index: remoting/android/java/src/org/chromium/chromoting/CardboardDesktopRenderer.java
diff --git a/remoting/android/java/src/org/chromium/chromoting/CardboardDesktopRenderer.java b/remoting/android/java/src/org/chromium/chromoting/CardboardDesktopRenderer.java
index a5db293d55078e1e9df2491058eb6abb51103158..305b63c1f03ccb6d24f2e1e1f0cd9a78b2712dbc 100644
--- a/remoting/android/java/src/org/chromium/chromoting/CardboardDesktopRenderer.java
+++ b/remoting/android/java/src/org/chromium/chromoting/CardboardDesktopRenderer.java
@@ -375,10 +375,8 @@ public class CardboardDesktopRenderer implements CardboardView.StereoRenderer {
         return value;
     }
 
-    /**
-     * Check whether user is looking at desktop or not.
-     * This can be called on any thread.
-     */
+
+     // This can be called on any thread.

[Lambros, 2015/08/05 00:25:52]

Undo this change?

[shichengfeng, 2015/08/05 23:06:31]

Done.

     public boolean isLookingAtDesktop() {
         synchronized (mEyePositionLock) {
             return Math.abs(mEyePositionVector[0]) <= (mHalfDesktopWidth + EDGE_MARGIN)
@@ -386,6 +384,20 @@ public class CardboardDesktopRenderer implements CardboardView.StereoRenderer {
         }
     }
 
+    // This can be called on any thread.

[Lambros, 2015/08/05 00:25:52]

I think Javadoc is better, since the method is public:
"Returns true if user is looking at the space to the left of the desktop. Can be
called on any thread."

[shichengfeng, 2015/08/05 23:06:31]

Done.

+    public boolean isLookingLeftToDesktop() {

[Lambros, 2015/08/05 00:31:14]

s/To/Of/

[shichengfeng, 2015/08/05 23:06:31]

Done.

+        synchronized (mEyePositionLock) {
+            return mEyePositionVector[0] >= (mHalfDesktopWidth + EDGE_MARGIN);
+        }
+    }
+
+    // This can be called on any thread.
+    public boolean isLookingRightToDesktop() {

[Lambros, 2015/08/05 00:25:52]

Same here.

[shichengfeng, 2015/08/05 23:06:31]

Done.

+        synchronized (mEyePositionLock) {
+            return mEyePositionVector[0] <= -(mHalfDesktopWidth + EDGE_MARGIN);
+        }
+    }
+
     /**
      * Get position on desktop where user is looking at.
      */