Voice input

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

 // Copyright 2015 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.
 
 package org.chromium.chromoting;
 
+import android.content.Intent;
 import android.graphics.PointF;
 import android.os.Bundle;
+import android.speech.RecognitionListener;
+import android.speech.RecognizerIntent;
+import android.speech.SpeechRecognizer;
 
 import com.google.vrtoolkit.cardboard.CardboardActivity;
 import com.google.vrtoolkit.cardboard.CardboardView;
 
 import org.chromium.chromoting.jni.JniInterface;
 
 /**
  * Virtual desktop activity for Cardboard.
  */
 public class CardboardDesktopActivity extends CardboardActivity {
     // Flag to indicate whether the current activity is going to switch to normal
     // desktop activity.
     private boolean mSwitchToDesktopActivity;
 
     private CardboardDesktopRenderer mRenderer;
+    private SpeechRecognizer mSpeechRecognizer;
 
     @Override
     public void onCreate(Bundle savedInstanceState) {
         super.onCreate(savedInstanceState);
         setContentView(R.layout.cardboard_desktop);
         mSwitchToDesktopActivity = false;
         CardboardView cardboardView = (CardboardView) findViewById(R.id.cardboard_view);
         mRenderer = new CardboardDesktopRenderer(this);
+        mSpeechRecognizer = SpeechRecognizer.createSpeechRecognizer(this);

[Sergey Ulanov, 2015/08/05 17:53:47]

It might be good idea to create this lazily the first time the user tries to use
speech input.

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

Done.

+        mSpeechRecognizer.setRecognitionListener(new VoiceInputRecognitionListener());
 
         // Associate a CardboardView.StereoRenderer with cardboardView.
         cardboardView.setRenderer(mRenderer);
 
         // Associate the cardboardView with this activity.
         setCardboardView(cardboardView);
     }
 
     @Override
     public void onCardboardTrigger() {
         if (mRenderer.isLookingAtDesktop()) {
             PointF coordinates = mRenderer.getMouseCoordinates();
             JniInterface.sendMouseEvent((int) coordinates.x, (int) coordinates.y,
                     TouchInputHandler.BUTTON_LEFT, true);
             JniInterface.sendMouseEvent((int) coordinates.x, (int) coordinates.y,
                     TouchInputHandler.BUTTON_LEFT, false);
-        } else {
+        } else if (mRenderer.isLookingLeftToDesktop()) {
             mSwitchToDesktopActivity = true;
             finish();
+        } else if (mRenderer.isLookingRightToDesktop()) {

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

This is OK for now, but add a TODO for a more polished UI :)

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

Done.

+            sendVoiceInput();
         }
     }
 
     @Override
     protected void onStart() {
         super.onStart();
         JniInterface.enableVideoChannel(true);
         mRenderer.attachRedrawCallback();
     }
 
@@ skipping 13 matching lines @@
 
     @Override
     protected void onStop() {
         super.onStop();
         if (mSwitchToDesktopActivity) {
             mSwitchToDesktopActivity = false;
         } else {
             JniInterface.enableVideoChannel(false);
         }
     }
+
+    private void sendVoiceInput() {
+        Intent intent = new Intent(RecognizerIntent.ACTION_RECOGNIZE_SPEECH);
+        intent.putExtra(RecognizerIntent.EXTRA_LANGUAGE_MODEL,
+                RecognizerIntent.LANGUAGE_MODEL_FREE_FORM);
+
+        mSpeechRecognizer.startListening(intent);

[Sergey Ulanov, 2015/08/05 17:53:47]

Do we need to verify that the recognizer is not already listening before calling
startListening()?

[Sergey Ulanov, 2015/08/05 17:53:47]

I think one problem when I tried using the feature is that there is no
indication that speech input is active. It would be great to add some visual
indicator for that.

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

Sure, I will put all UI polish in another CL. Put a TODO here. 

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

Yes, we have to. I added a flag to prevent calling multiple startListening at
the same time.

+    }
+
+    private class VoiceInputRecognitionListener implements RecognitionListener {
+        public void onReadyForSpeech(Bundle params) {
+        }
+
+        public void onBeginningOfSpeech() {
+        }
+
+        public void onRmsChanged(float rmsdB){
+        }
+
+        public void onBufferReceived(byte[] buffer) {
+        }
+
+        public void onEndOfSpeech() {
+        }
+
+        public void onError(int error) {
+        }
+
+        public void onResults(Bundle results) {
+            // TODO(shichengfeng): If necessary, provide a list of choices for user to pick.
+            String text = results.getStringArrayList(SpeechRecognizer.RESULTS_RECOGNITION).get(0);

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

I don't know if the result array is guaranteed to be non-empty? Maybe check it
anyway?

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

Done.

+            JniInterface.sendTextEvent(text);
+        }
+
+        public void onPartialResults(Bundle partialResults) {
+        }
+
+        public void onEvent(int eventType, Bundle params) {
+        }
+    }
 }