Don't start speech synthesis until after the audio mixer is initialized.

chrome/browser/chromeos/accessibility_util.cc

 // Copyright (c) 2011 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 "chrome/browser/chromeos/accessibility_util.h"
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/logging.h"
 #include "chrome/browser/browser_process.h"
-#include "chrome/browser/chromeos/dbus/dbus_thread_manager.h"
-#include "chrome/browser/chromeos/dbus/speech_synthesizer_client.h"
 #include "chrome/browser/extensions/extension_accessibility_api.h"
+#include "chrome/browser/extensions/extension_tts_api_platform.h"
 #include "chrome/browser/extensions/component_loader.h"
 #include "chrome/browser/extensions/extension_service.h"
 #include "chrome/browser/extensions/file_reader.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/profiles/profile_manager.h"
 #include "chrome/common/extensions/extension_messages.h"
 #include "chrome/common/extensions/extension_resource.h"
 #include "chrome/common/pref_names.h"
 #include "content/browser/renderer_host/render_view_host.h"
 #include "content/browser/tab_contents/tab_contents.h"
@@ skipping 118 matching lines @@
 }
 
 void ToggleAccessibility(WebUI* login_web_ui) {
   bool accessibility_enabled = g_browser_process &&
       g_browser_process->local_state()->GetBoolean(
           prefs::kAccessibilityEnabled);
   accessibility_enabled = !accessibility_enabled;
   EnableAccessibility(accessibility_enabled, login_web_ui);
 };
 
 void Speak(const char* speak_str, bool queue, bool interruptible) {

[oshima, 2011/11/09 23:47:32]

are "queue" and "interruptable" parameters obsolete?
If so, can we remove them?

[dmazzoni, 2011/11/10 07:19:57]

Done. They're obsolete assuming that the old views-based oobe/login is obsolete.
Is there a plan to get rid of it soon?

-  if (queue || !interruptible) {
-    std::string props = "";
-    props.append("enqueue=");
-    props.append(queue ? "1;" : "0;");
-    props.append("interruptible=");
-    props.append(interruptible ? "1;" : "0;");
-    chromeos::DBusThreadManager::Get()->GetSpeechSynthesizerClient()->
-        SetSpeakProperties(props);
-  }
-  chromeos::DBusThreadManager::Get()->GetSpeechSynthesizerClient()->
-      Speak(speak_str);
+  UtteranceContinuousParameters params;
+  ExtensionTtsPlatformImpl::GetInstance()->Speak(
+      -1,
+      speak_str,
+      "",
+      params);

[oshima, 2011/11/09 23:47:32]

can you add comment for -1 and ""? 
(like -1 /* xyz */)

[dmazzoni, 2011/11/10 07:19:57]

Done. Actually the "" should have been the language/locale, I fixed that. (""
just gave the default.)

 }
 
 
 }  // namespace accessibility
 }  // namespace chromeos