Host side implementation of ARC tts.

components/arc/tts/arc_tts_service.cc

+// Copyright 2016 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 "components/arc/tts/arc_tts_service.h"
+
+#include "base/logging.h"
+#include "chrome/browser/speech/tts_controller.h"
+#include "components/arc/arc_bridge_service.h"
+
+namespace arc {
+
+ArcTtsService::ArcTtsService(ArcBridgeService* bridge_service)
+    : ArcService(bridge_service), binding_(this) {
+  arc_bridge_service()->tts()->AddObserver(this);
+}
+
+ArcTtsService::~ArcTtsService() {}

[hidehiko, 2016/07/26 17:59:18]

arc_bridge_service()->tts()->RemoveObserver(this);

is needed.

[David Tseng, 2016/07/27 22:50:59]

Done.

+
+void ArcTtsService::OnInstanceReady() {
+  mojom::TtsInstance* tts_instance = arc_bridge_service()->tts()->instance();
+  if (!tts_instance) {
+    LOG(ERROR) << "OnTtsInstanceReady called, "
+               << "but no tts instance found";
+    return;
+  }
+  tts_instance->Init(binding_.CreateInterfacePtrAndBind());
+}
+
+void ArcTtsService::OnTtsEvent(uint32_t id, mojom::TtsEventType event_type) {
+  if (!TtsController::GetInstance())
+    return;
+
+  TtsEventType chrome_event_type;
+  switch (event_type) {
+    case mojom::TtsEventType::START:
+      chrome_event_type = TTS_EVENT_START;
+      break;
+    case mojom::TtsEventType::END:
+      chrome_event_type = TTS_EVENT_END;
+      break;
+    case mojom::TtsEventType::INTERRUPTED:
+      chrome_event_type = TTS_EVENT_INTERRUPTED;
+      break;
+    case mojom::TtsEventType::ERROR:
+      chrome_event_type = TTS_EVENT_ERROR;
+      break;
+  }
+  TtsController::GetInstance()->OnTtsEvent(id, chrome_event_type, 0, "");

[hidehiko, 2016/07/26 17:59:18]

Could you comment 0 and "" for what?

[dmazzoni, 2016/07/26 22:12:58]

To match other engines, I think the "end" event should be sent with the
utterance length as the char_index.

You can install TTS Debug from the web store and it will run through a bunch of
manual and automated tests to make sure you've implemented a bunch of things
correctly.

[hidehiko, 2016/07/29 09:25:53]

ping?

[David Tseng, 2016/08/01 20:21:41]

Passing this through now from the Android end.
Most of the logic is on the Android end. I suspect there will be some issues
with feature parity. Hoping to get the basic bindings in place first to avoid
constant rebasing.

+}
+
+}  // namespace arc