Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(625)

Unified Diff: chrome/browser/chromeos/arc/accessibility_helper/arc_accessibility_helper_bridge.cc

Issue 2559663002: Support focus highlight in Android window (Closed)
Patch Set: Fix comments and style. Created 3 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/chromeos/arc/accessibility_helper/arc_accessibility_helper_bridge.cc
diff --git a/chrome/browser/chromeos/arc/accessibility_helper/arc_accessibility_helper_bridge.cc b/chrome/browser/chromeos/arc/accessibility_helper/arc_accessibility_helper_bridge.cc
new file mode 100644
index 0000000000000000000000000000000000000000..682deff9d68893886c22b8f1bcabab2d91d664c4
--- /dev/null
+++ b/chrome/browser/chromeos/arc/accessibility_helper/arc_accessibility_helper_bridge.cc
@@ -0,0 +1,65 @@
+// 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/accessibility_helper/arc_accessibility_helper_bridge.h"
+
+#include "chrome/browser/chromeos/accessibility/accessibility_manager.h"
+#include "components/arc/arc_bridge_service.h"
+#include "components/exo/wm_helper.h"
+#include "ui/aura/window.h"
+
+namespace arc {
+
+ArcAccessibilityHelperBridge::ArcAccessibilityHelperBridge(
+ ArcBridgeService* bridge_service)
+ : ArcService(bridge_service), binding_(this) {
+ bridge_service->accessibility_helper()->AddObserver(this);
+}
+
+ArcAccessibilityHelperBridge::~ArcAccessibilityHelperBridge() {
+ arc_bridge_service()->accessibility_helper()->RemoveObserver(this);
+}
+
+void ArcAccessibilityHelperBridge::OnInstanceReady() {
+ auto* instance = ARC_GET_INSTANCE_FOR_METHOD(
+ arc_bridge_service()->accessibility_helper(), Init);
+ DCHECK(instance);
+ instance->Init(binding_.CreateInterfacePtrAndBind());
+}
+
+void ArcAccessibilityHelperBridge::OnAccessibilityEvent(
+ mojom::AccessibilityEventType eventType,
+ mojom::AccessibilityNodeInfoDataPtr eventSource) {
+ if (eventType != mojom::AccessibilityEventType::VIEW_FOCUSED ||
+ eventSource.is_null()) {
+ return;
+ }
+
+ chromeos::AccessibilityManager* accessibility_manager =
+ chromeos::AccessibilityManager::Get();
+
+ if (!accessibility_manager ||
+ !accessibility_manager->IsFocusHighlightEnabled()) {
+ return;
+ }
+
+ exo::WMHelper* wmHelper = exo::WMHelper::GetInstance();
+
+ aura::Window* focused_window = wmHelper->GetFocusedWindow();
+ if (!focused_window)
+ return;
+
+ aura::Window* toplevel_window = focused_window->GetToplevelWindow();
+
+ gfx::Rect bounds_in_screen = gfx::ScaleToEnclosingRect(
Luis Héctor Chávez 2017/01/19 17:17:56 nit: #include "ui/gfx/geometry/rect.h"
yawano 2017/01/20 06:52:30 Done.
+ eventSource.get()->boundsInScreen,
+ 1 / toplevel_window->layer()->device_scale_factor());
Luis Héctor Chávez 2017/01/19 17:17:56 nit: maybe 1.0f to avoid the implicit conversion?
yawano 2017/01/20 06:52:30 Done.
+
+ const gfx::Rect& window_bounds = toplevel_window->GetBoundsInScreen();
+ bounds_in_screen.Offset(window_bounds.x(), window_bounds.y());
+
+ accessibility_manager->OnViewFocusedInArc(bounds_in_screen);
+}
+
+} // namespace arc

Powered by Google App Engine
This is Rietveld 408576698