add voice interaction shortcut.

chrome/browser/chromeos/arc/voice_interaction/arc_voice_interaction_framework_service.cc

+// Copyright 2017 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/arc/voice_interaction/arc_voice_interaction_framework_service.h"
+
+#include <utility>
+#include <vector>
+
+#include "ash/common/accelerators/accelerator_controller.h"
+#include "ash/common/wm_shell.h"
+#include "ash/shell.h"
+#include "base/command_line.h"
+#include "base/logging.h"
+#include "base/optional.h"
+#include "components/arc/arc_bridge_service.h"
+#include "components/arc/instance_holder.h"
+#include "content/public/browser/browser_thread.h"
+
+namespace arc {
+
+ArcVoiceInteractionFrameworkService::ArcVoiceInteractionFrameworkService(
+    ArcBridgeService* bridge_service)
+    : ArcService(bridge_service), binding_(this) {
+  arc_bridge_service()->voice_interaction_framework()->AddObserver(this);
+}
+
+ArcVoiceInteractionFrameworkService::~ArcVoiceInteractionFrameworkService() {
+  arc_bridge_service()->voice_interaction_framework()->RemoveObserver(this);
+}
+
+void ArcVoiceInteractionFrameworkService::OnInstanceReady() {
+  DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
+  mojom::VoiceInteractionFrameworkInstance* framework_instance =
+      ARC_GET_INSTANCE_FOR_METHOD(
+          arc_bridge_service()->voice_interaction_framework(), Init);
+  DCHECK(framework_instance);
+  framework_instance->Init(binding_.CreateInterfacePtrAndBind());
+
+  ash::WmShell::Get()->accelerator_controller()->Register(
+      {ui::Accelerator(ui::VKEY_A, ui::EF_COMMAND_DOWN)}, this);
+}
+
+void ArcVoiceInteractionFrameworkService::OnInstanceClosed() {
+  DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
+  ash::WmShell::Get()->accelerator_controller()->UnregisterAll(this);
+}
+
+bool ArcVoiceInteractionFrameworkService::AcceleratorPressed(
+    const ui::Accelerator& accelerator) {
+  DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
+  mojom::VoiceInteractionFrameworkInstance* framework_instance =
+      ARC_GET_INSTANCE_FOR_METHOD(
+          arc_bridge_service()->voice_interaction_framework(),
+          StartVoiceInteractionSession);
+  DCHECK(framework_instance);
+  framework_instance->StartVoiceInteractionSession();
+  return true;
+}
+
+bool ArcVoiceInteractionFrameworkService::CanHandleAccelerators() const {
+  DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
+  return true;
+}
+
+void ArcVoiceInteractionFrameworkService::CaptureFocusedWindow(
+    const CaptureFocusedWindowCallback& callback) {
+  DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
+  NOTIMPLEMENTED();
+  // TODO(muyuanli): The logic below is only there to prevent blocking.
+  //                 details needs to be implemented in the future.

[dcheng, 2017/03/17 22:55:16]

I don't think this is likely to have a security impact, but please add an ipc
security owner on the followup CL that implements this.

[Muyuan, 2017/03/17 23:39:29]

Acknowledged.

+  callback.Run({});
+}
+
+}  // namespace arc