Implement SpeechRecognizerImplAndroid

content/browser/speech/speech_recognition_manager_impl.cc

 // Copyright (c) 2013 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.
 
 #include "content/browser/speech/speech_recognition_manager_impl.h"
 
 #include "base/bind.h"
 #include "content/browser/browser_main_loop.h"
 #include "content/browser/renderer_host/media/media_stream_manager.h"
 #include "content/browser/speech/google_one_shot_remote_engine.h"
 #include "content/browser/speech/google_streaming_remote_engine.h"
 #include "content/browser/speech/speech_recognition_engine.h"
 #include "content/browser/speech/speech_recognizer_impl.h"
+#include "content/browser/speech/speech_recognizer_impl_android.h"
 #include "content/public/browser/browser_thread.h"
 #include "content/public/browser/content_browser_client.h"
 #include "content/public/browser/resource_context.h"
 #include "content/public/browser/speech_recognition_event_listener.h"
 #include "content/public/browser/speech_recognition_manager_delegate.h"
 #include "content/public/browser/speech_recognition_session_config.h"
 #include "content/public/browser/speech_recognition_session_context.h"
 #include "content/public/common/speech_recognition_error.h"
 #include "content/public/common/speech_recognition_result.h"
 #include "media/audio/audio_manager.h"
@@ skipping 101 matching lines @@
   }
 
   google_remote_engine->SetConfig(remote_engine_config);
 
   session.recognizer = new SpeechRecognizerImpl(
       this,
       session_id,
       !config.continuous,
       google_remote_engine);
 #else
-  // TODO(janx): Implement a SpeechRecognizerImplAndroid with a JNI interface
-  // forwarding calls to Android's platform speech recognition service (see
-  // crbug.com/222352).
-  session.recognizer = NULL;
+  session.recognizer = new SpeechRecognizerImplAndroid(
+      this,
+      config,

[Primiano Tucci (use gerrit), 2013/06/03 16:24:11]

Can you just pass session_id and use
SpeechRecognitionManager.GetSessionConfig(id) to get the config inside
SpeechRecognizerImplAndroid?

[janx, 2013/06/05 13:51:00]

Since the config is used only in StartRecognition, I'll use the getter instead
of saving the config as an attribute. Nice catch.

+      session_id);
 #endif
   return session_id;
 }
 
 void SpeechRecognitionManagerImpl::StartSession(int session_id) {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO));
   if (!SessionExists(session_id))
     return;
 
   // If there is another active session, abort that.
@@ skipping 523 matching lines @@
 
 SpeechRecognitionManagerImpl::Session::Session()
   : id(kSessionIDInvalid),
     listener_is_active(true) {
 }
 
 SpeechRecognitionManagerImpl::Session::~Session() {
 }
 
 }  // namespace content