Uses a system-wide notion of isSpeaking in the Mac extension TTS api.

chrome/browser/speech/extension_api/tts_extension_api_controller.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 "chrome/browser/speech/extension_api/tts_extension_api_controller.h"
 
 #include <string>
 #include <vector>
 
 #include "base/float_util.h"
@@ skipping 265 matching lines @@
           constants::kEventTypeError));
     result_voice->Set(constants::kEventTypesKey, event_types);
     result_voices->Append(result_voice);
   }
 
   GetExtensionVoices(profile, result_voices);
 
   return result_voices;
 }
 
-bool ExtensionTtsController::IsSpeaking() const {
-  return current_utterance_ != NULL;
+bool ExtensionTtsController::IsSpeaking() {
+  return current_utterance_ != NULL && GetPlatformImpl()->IsSpeaking();

[dmazzoni, 2012/04/11 20:44:18]

Shouldn't this be || instead of &&

[David Tseng, 2012/04/11 22:20:47]

On 2012/04/11 20:44:18, Dominic Mazzoni wrote:

You're right; the current && logic adds nothing. I don't think we expect
utterance to be non-null when isSpeaking returns true elsewhere?

 }
 
 void ExtensionTtsController::FinishCurrentUtterance() {
   if (current_utterance_) {
     if (!current_utterance_->finished())
       current_utterance_->OnTtsEvent(TTS_EVENT_INTERRUPTED, kInvalidCharIndex,
                                      std::string());
     delete current_utterance_;
     current_utterance_ = NULL;
   }
@@ skipping 29 matching lines @@
 
 int ExtensionTtsController::QueueSize() {
   return static_cast<int>(utterance_queue_.size());
 }
 
 ExtensionTtsPlatformImpl* ExtensionTtsController::GetPlatformImpl() {
   if (!platform_impl_)
     platform_impl_ = ExtensionTtsPlatformImpl::GetInstance();
   return platform_impl_;
 }