Refactoring of chrome speech recognition architecture (CL1.3)

content/browser/speech/speech_recognition_manager_impl.cc

 // Copyright (c) 2012 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/render_view_host_impl.h"
 #include "content/browser/speech/input_tag_speech_dispatcher_host.h"
@@ skipping 185 matching lines @@
     NOTREACHED();
     return;
   }
 
   // We should not currently be recording for the caller.
   CHECK(recording_caller_id_ != caller_id);
 
   // If we are currently recording audio for another caller, abort that cleanly.
   if (recording_caller_id_)
     CancelRecognitionAndInformDelegate(recording_caller_id_);
-

[Primiano Tucci (use gerrit), 2012/03/15 09:14:36]

These checks have been moved into the speech_recognizer_impl, since it is the
one that takes care entirely of the audio capture process.

-  if (!HasAudioInputDevices()) {
-    if (delegate_) {
-      delegate_->ShowMicError(caller_id,
-          SpeechRecognitionManagerDelegate::MIC_ERROR_NO_DEVICE_AVAILABLE);
-    }
-  } else if (IsCapturingAudio()) {
-    if (delegate_) {
-      delegate_->ShowMicError(
-          caller_id, SpeechRecognitionManagerDelegate::MIC_ERROR_DEVICE_IN_USE);
-    }
-  } else {
-    recording_caller_id_ = caller_id;
-    requests_[caller_id].is_active = true;
-    requests_[caller_id].recognizer->StartRecognition();
-    if (delegate_)
-      delegate_->ShowWarmUp(caller_id);
-  }
+  recording_caller_id_ = caller_id;
+  requests_[caller_id].is_active = true;
+  requests_[caller_id].recognizer->StartRecognition();
+  if (delegate_)
+    delegate_->ShowWarmUp(caller_id);
 }
 
 void SpeechRecognitionManagerImpl::CancelRecognitionForRequest(int caller_id) {
   // Ignore if the caller id was not in our active recognizers list because the
   // user might have clicked more than once, or recognition could have been
   // ended due to other reasons before the user click was processed.
   if (!HasPendingRequest(caller_id))
     return;
 
   CancelRecognitionAndInformDelegate(caller_id);
@@ skipping 43 matching lines @@
     return;
 
   requests_[caller_id].recognizer->StopAudioCapture();
 }
 
 // -------- SpeechRecognitionEventListener interface implementation. ---------
 
 void SpeechRecognitionManagerImpl::OnRecognitionResult(
     int caller_id, const content::SpeechRecognitionResult& result) {
   DCHECK(HasPendingRequest(caller_id));
+  DCHECK(result.error == content::SPEECH_RECOGNITION_ERROR_NONE);
   GetDelegate(caller_id)->SetRecognitionResult(caller_id, result);
 }
 
 void SpeechRecognitionManagerImpl::OnAudioEnd(int caller_id) {
+  if (recording_caller_id_ != caller_id)

[Primiano Tucci (use gerrit), 2012/03/15 09:14:36]

Basically we need these checks because the new event_listener interface contract
is slightly changes, and the AudioEnd and RecognitionEnd events are always
raised even in case of error.

Spoiler: this code is transitional (just to make this CL work correctly). This
class will be reimplemented in the next CL as a FSM.

[hans, 2012/03/16 11:12:56]

maybe put a comment on the if statement about when this can happen?

[Primiano Tucci (use gerrit), 2012/03/16 15:03:42]

Spoiler: this code is going to disappear in the next CL :)

+    return;
   DCHECK_EQ(recording_caller_id_, caller_id);
   DCHECK(HasPendingRequest(caller_id));
+  if (!requests_[caller_id].is_active)
+    return;
   recording_caller_id_ = 0;
   GetDelegate(caller_id)->DidCompleteRecording(caller_id);
   if (delegate_)
     delegate_->ShowRecognizing(caller_id);
 }
 
 void SpeechRecognitionManagerImpl::OnRecognitionEnd(int caller_id) {
+  if (!HasPendingRequest(caller_id) || !requests_[caller_id].is_active)
+    return;
   GetDelegate(caller_id)->DidCompleteRecognition(caller_id);
   requests_.erase(caller_id);
   if (delegate_)
     delegate_->DoClose(caller_id);
 }
 
 void SpeechRecognitionManagerImpl::OnSoundStart(int caller_id) {
 }
 
 void SpeechRecognitionManagerImpl::OnSoundEnd(int caller_id) {
 }
 
 void SpeechRecognitionManagerImpl::OnRecognitionError(
-    int caller_id, const content::SpeechRecognitionErrorCode& error) {
+    int caller_id, const content::SpeechRecognitionError& error) {
+  DCHECK(HasPendingRequest(caller_id));
   if (caller_id == recording_caller_id_)
     recording_caller_id_ = 0;
   requests_[caller_id].is_active = false;
-  if (delegate_)
-    delegate_->ShowRecognizerError(caller_id, error);
+  if (delegate_) {
+    if (error.code == content::SPEECH_RECOGNITION_ERROR_AUDIO &&
+        error.details == content::AUDIO_ERROR_NO_MIC) {
+    delegate_->ShowMicError(caller_id,

[hans, 2012/03/16 11:12:56]

indentation looks wrong

[Primiano Tucci (use gerrit), 2012/03/16 15:03:42]

Done.

+        SpeechRecognitionManagerDelegate::MIC_ERROR_NO_DEVICE_AVAILABLE);
+    } else if (error.code == content::SPEECH_RECOGNITION_ERROR_AUDIO &&
+               error.details == content::AUDIO_ERROR_MIC_IN_USE) {
+      delegate_->ShowMicError(
+          caller_id, SpeechRecognitionManagerDelegate::MIC_ERROR_DEVICE_IN_USE);
+    } else {
+      delegate_->ShowRecognizerError(caller_id, error.code);
+    }
+  }
 }
 
 void SpeechRecognitionManagerImpl::OnAudioStart(int caller_id) {
   DCHECK(HasPendingRequest(caller_id));
   DCHECK_EQ(recording_caller_id_, caller_id);
   if (delegate_)
     delegate_->ShowRecording(caller_id);
 }
 
 void SpeechRecognitionManagerImpl::OnRecognitionStart(int caller_id) {
@@ skipping 23 matching lines @@
 
 SpeechRecognitionManagerImpl::Request::Request()
     : delegate(NULL),
       is_active(false) {
 }
 
 SpeechRecognitionManagerImpl::Request::~Request() {
 }
 
 }  // namespace speech