Remove SpeechRecognitionSessionConfig::is_legacy_api

content/browser/speech/speech_recognition_manager_impl.cc

Index: content/browser/speech/speech_recognition_manager_impl.cc
diff --git a/content/browser/speech/speech_recognition_manager_impl.cc b/content/browser/speech/speech_recognition_manager_impl.cc
index 47b5995f0256f359882df908509841fda8d0a945..511251278dbcca62105ce8a01f6c7ff48177415f 100644
--- a/content/browser/speech/speech_recognition_manager_impl.cc
+++ b/content/browser/speech/speech_recognition_manager_impl.cc
@@ -113,9 +113,6 @@ int SpeechRecognitionManagerImpl::CreateSession(
   if (delegate_)
     delegate_->GetDiagnosticInformation(&can_report_metrics, &hardware_info);
 
-  // The legacy api cannot use continuous mode.
-  DCHECK(!config.is_legacy_api || !config.continuous);
-
 #if !defined(OS_ANDROID)
   // A SpeechRecognitionEngine (and corresponding Config) is required only
   // when using SpeechRecognizerImpl, which performs the audio capture and
@@ -141,15 +138,8 @@ int SpeechRecognitionManagerImpl::CreateSession(
   remote_engine_config.auth_scope = config.auth_scope;
   remote_engine_config.preamble = config.preamble;
 
-  SpeechRecognitionEngine* google_remote_engine;
-  if (config.is_legacy_api) {
-    google_remote_engine =
-        new GoogleOneShotRemoteEngine(config.url_request_context_getter.get());

[Primiano Tucci (use gerrit), 2016/04/13 17:40:59]

At this point in theory we could remove the GoogleOneShotRemoteEngine.
In practice IIRC that is still used by the recognizer unittest, so might not be
that easy. :)

[hans, 2016/04/13 17:41:50]

I have a CL for that :-) https://codereview.chromium.org/1886813002

-  } else {
-    google_remote_engine = new GoogleStreamingRemoteEngine(
-        config.url_request_context_getter.get());
-  }
-
+  SpeechRecognitionEngine* google_remote_engine =
+      new GoogleStreamingRemoteEngine(config.url_request_context_getter.get());
   google_remote_engine->SetConfig(remote_engine_config);
 
   session->recognizer = new SpeechRecognizerImpl(