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;
 
+import java.util.ArrayList;
+
 /**
  * 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;
+
+    // Flag to indicate whether the speech recognizer is listening or not.
+    private boolean mIsListening;
 
     @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);
+        mIsListening = false;
 
         // 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.isLookingLeftOfDesktop()) {
+            // TODO(shichengfeng): Give a more polished UI(including menu icons

[Lambros, 2015/08/05 23:59:37]

nit: space after "UI"

[shichengfeng, 2015/08/06 19:13:41]

Done.

+            // and visual indicator during when speech recognizer is listening).
             mSwitchToDesktopActivity = true;
             finish();
+        } else if (mRenderer.isLookingRightOfDesktop() && !mIsListening) {

[Lambros, 2015/08/05 23:59:37]

Maybe move the !listening check inside sendVoiceInput() ?

[shichengfeng, 2015/08/06 19:13:41]

Done.

+            mIsListening = true;

[Lambros, 2015/08/05 23:59:37]

Move this inside sendVoiceInput() ?

[shichengfeng, 2015/08/06 19:13:41]

Done.

+            sendVoiceInput();
         }
     }
 
     @Override
     protected void onStart() {
         super.onStart();
         JniInterface.enableVideoChannel(true);
         mRenderer.attachRedrawCallback();
     }
 
     @Override
     protected void onPause() {
         super.onPause();
         if (!mSwitchToDesktopActivity) {
             JniInterface.enableVideoChannel(false);
         }
+        if (mSpeechRecognizer != null) {
+            mSpeechRecognizer.stopListening();
+        }
     }
 
     @Override
     protected void onResume() {
         super.onResume();
         JniInterface.enableVideoChannel(true);
     }
 
     @Override
     protected void onStop() {
         super.onStop();
         if (mSwitchToDesktopActivity) {
             mSwitchToDesktopActivity = false;
         } else {
             JniInterface.enableVideoChannel(false);
         }
+        if (mSpeechRecognizer != null) {
+            mSpeechRecognizer.stopListening();
+        }
+    }
+
+    @Override
+    protected void onDestroy() {
+        super.onDestroy();
+        if (mSpeechRecognizer != null) {
+            mSpeechRecognizer.cancel();
+            mSpeechRecognizer.destroy();
+        }
+    }
+
+    private void sendVoiceInput() {

[Lambros, 2015/08/05 23:59:37]

This doesn't really "send" anything, it just starts listening. Maybe rename to
"listenForVoiceInput" ?

[shichengfeng, 2015/08/06 19:13:41]

Done.

+        if (mSpeechRecognizer == null) {
+            mSpeechRecognizer = SpeechRecognizer.createSpeechRecognizer(this);

[Lambros, 2015/08/05 23:59:37]

Check to see if the service is available before creating the recognizer. See
http://developer.android.com/reference/android/speech/SpeechRecognizer.html#i...

[shichengfeng, 2015/08/06 19:13:41]

Done.

+            mSpeechRecognizer.setRecognitionListener(new VoiceInputRecognitionListener());
+        }
+
+        Intent intent = new Intent(RecognizerIntent.ACTION_RECOGNIZE_SPEECH);
+        intent.putExtra(RecognizerIntent.EXTRA_LANGUAGE_MODEL,
+                RecognizerIntent.LANGUAGE_MODEL_FREE_FORM);
+
+        mSpeechRecognizer.startListening(intent);
+    }
+
+    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() {

[Lambros, 2015/08/05 23:59:37]

Set mIsListening = false here? The recognizer stops listening automatically when
it detects the speech has ended - see
http://developer.android.com/reference/android/speech/SpeechRecognizer.html#s....
In a followup CL, you could give user feedback here.

[shichengfeng, 2015/08/06 19:13:41]

Done.

+        }
+
+        public void onError(int error) {
+        }
+
+        public void onResults(Bundle results) {
+            // TODO(shichengfeng): If necessary, provide a list of choices for user to pick.
+            ArrayList<String> data =
+                    results.getStringArrayList(SpeechRecognizer.RESULTS_RECOGNITION);
+            if (!data.isEmpty()) {
+                JniInterface.sendTextEvent(data.get(0));
+            }
+            mIsListening = false;

[Lambros, 2015/08/05 23:59:37]

I wonder if this belongs in onEndOfSpeech() ?

[shichengfeng, 2015/08/06 19:13:41]

Done.

+        }
+
+        public void onPartialResults(Bundle partialResults) {
+        }
+
+        public void onEvent(int eventType, Bundle params) {
+        }
     }
 }