Support focus highlight in Android window

chrome/browser/chromeos/arc/focus_highlight/arc_focus_highlight_bridge.h

+// 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.
+
+#ifndef CHROME_BROWSER_CHROMEOS_ARC_FOCUS_HIGHLIGHT_ARC_FOCUS_HIGHLIGHT_BRIDGE_H_
+#define CHROME_BROWSER_CHROMEOS_ARC_FOCUS_HIGHLIGHT_ARC_FOCUS_HIGHLIGHT_BRIDGE_H_
+
+#include "components/arc/arc_service.h"
+#include "components/arc/common/focus_highlight.mojom.h"
+#include "components/arc/instance_holder.h"
+#include "mojo/public/cpp/bindings/binding.h"
+
+namespace arc {
+
+class ArcBridgeService;
+
+class ArcFocusHighlightBridge
+    : public ArcService,
+      public mojom::FocusHighlightHost,
+      public InstanceHolder<mojom::FocusHighlightInstance>::Observer {
+ public:
+  explicit ArcFocusHighlightBridge(ArcBridgeService* bridge_service);
+  ~ArcFocusHighlightBridge() override;
+
+  // InstanceHolder::Observer::OnInstanceReady overrides.

[Luis Héctor Chávez, 2016/12/07 17:58:56]

nit:

// InstanceHolder<mojom::FocusHighlightInstance>::Observer overrides:
void OnInstanceReady() override;
void OnInstanceClosed() override;

you just need to list the name of the interface, not the individual methods you
are overriding.

[yawano, 2017/01/11 10:31:33]

Done.

+  void OnInstanceReady() override;
+
+  // InstanceHolder::Observer::OnInstanceClosed overrides.
+  void OnInstanceClosed() override;
+
+  void OnViewFocused(const gfx::Rect& rect) override;

[hidehiko, 2016/12/08 14:54:59]

nit: "// mojom::FocusHighlightHost" too?

[yawano, 2017/01/11 10:31:33]

Done.

+
+ private:
+  mojo::Binding<mojom::FocusHighlightHost> binding_;
+
+  DISALLOW_COPY_AND_ASSIGN(ArcFocusHighlightBridge);
+};
+
+}  // namespace arc
+
+#endif  // CHROME_BROWSER_CHROMEOS_ARC_FOCUS_HIGHLIGHT_ARC_FOCUS_HIGHLIGHT_BRIDGE_H_