Move chromecast accessibility manager.

  * Move the cast AccessibilityManager into
    chromecast/browser/accessibility.
  * Only compile and create AccessibilityManager if extensions
    enabled.

Bug: none
Test: manual
Change-Id: I83c30b8259caa97093cdd48baafd26b2750d1da7
Reviewed-on: https://chromium-review.googlesource.com/1081234Reviewed-by: Sadrul Chowdhury <sadrul@chromium.org>
Reviewed-by: Kevin Schoedel <kpschoedel@chromium.org>
Reviewed-by: Alex Sakhartchouk <alexst@chromium.org>
Commit-Queue: Ryan Daum <rdaum@chromium.org>
Cr-Commit-Position: refs/heads/master@{#564692}

chromecast/browser/BUILD.gn

@@ -284,6 +284,8 @@ cast_source_set("browser") {
     ]
     if (use_aura) {
       sources += [
+        "accessibility/accessibility_manager.cc",
+        "accessibility/accessibility_manager.h",
         "cast_web_view_extension.h",
         "cast_web_view_extension.h",
         "ui/aura/accessibility/automation_manager_aura.cc",
@@ -309,6 +311,7 @@ cast_source_set("browser") {
       "//extensions/common/api",
       "//google_apis:google_apis",
       "//ui/views:views",
+      "//ui/wm/public:public",
     ]
   }
 

chromecast/browser/DEPS

@@ -49,6 +49,7 @@ include_rules = [
   "+ui/display",
   "+ui/ozone",
   "+ui/views",
+  "+ui/wm",
 
   # TODO(sanfin): Remove this by fixing the crash handler on android.
   "!chromecast/app",

chromecast/graphics/accessibility/accessibility_manager.cc → chromecast/browser/accessibility/accessibility_manager.cc

@@ -2,7 +2,7 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-#include "chromecast/graphics/accessibility/accessibility_manager.h"
+#include "chromecast/browser/accessibility/accessibility_manager.h"
 
 #include "chromecast/graphics/accessibility/focus_ring_controller.h"
 #include "ui/aura/window.h"
@@ -10,6 +10,7 @@
 #include "ui/wm/public/activation_client.h"
 
 namespace chromecast {
+namespace shell {
 
 AccessibilityManager::AccessibilityManager(
     aura::WindowTreeHost* window_tree_host) {
@@ -65,4 +66,5 @@ void AccessibilityManager::SetTouchAccessibilityAnchorPoint(
   // TODO(rdaum): Implement
 }
 
+}  // namespace shell
 }  // namespace chromecast

chromecast/graphics/accessibility/accessibility_manager.h → chromecast/browser/accessibility/accessibility_manager.h

@@ -2,8 +2,8 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-#ifndef CHROMECAST_GRAPHICS_ACCESSIBILITY_ACCESSIBILITY_MANAGER_H_
-#define CHROMECAST_GRAPHICS_ACCESSIBILITY_ACCESSIBILITY_MANAGER_H_
+#ifndef CHROMECAST_BROWSER_ACCESSIBILITY_ACCESSIBILITY_MANAGER_H_
+#define CHROMECAST_BROWSER_ACCESSIBILITY_ACCESSIBILITY_MANAGER_H_
 
 #include <memory>
 #include <vector>
@@ -18,6 +18,8 @@ namespace chromecast {
 
 class FocusRingController;
 
+namespace shell {
+
 // Responsible for delegating chromecast browser process accessibility functions
 // to the responsible party.
 class AccessibilityManager {
@@ -60,6 +62,7 @@ class AccessibilityManager {
       accessibility_focus_ring_controller_;
 };
 
+}  // namespace shell
 }  // namespace chromecast
 
-#endif  // CHROMECAST_GRAPHICS_ACCESSIBILITY_ACCESSIBILITY_MANAGER_H_
+#endif  // CHROMECAST_BROWSER_ACCESSIBILITY_ACCESSIBILITY_MANAGER_H_

chromecast/browser/cast_browser_main_parts.cc

@@ -91,8 +91,8 @@
 // gn check ignored on OverlayManagerCast as it's not a public ozone
 // header, but is exported to allow injecting the overlay-composited
 // callback.
+#include "chromecast/browser/accessibility/accessibility_manager.h"
 #include "chromecast/browser/cast_display_configurator.h"
-#include "chromecast/graphics/accessibility/accessibility_manager.h"
 #include "chromecast/graphics/cast_screen.h"
 #include "chromecast/graphics/cast_window_manager_aura.h"
 #include "components/viz/service/display/overlay_strategy_underlay_cast.h"  // nogncheck
@@ -535,12 +535,16 @@ void CastBrowserMainParts::PreMainMessageLoopRun() {
       CAST_IS_DEBUG_BUILD() ||
       GetSwitchValueBoolean(switches::kEnableInput, false));
   window_manager_->Setup();
+
+#if BUILDFLAG(ENABLE_CHROMECAST_EXTENSIONS)
   cast_browser_process_->SetAccessibilityManager(
       std::make_unique<AccessibilityManager>(
           window_manager_->window_tree_host()));
-#else
+#endif  // BUILDFLAG(ENABLE_CHROMECAST_EXTENSIONS)
+
+#else   // defined(USE_AURA)
   window_manager_ = std::make_unique<CastWindowManagerDefault>();
-#endif
+#endif  // defined(USE_AURA)
 
   cast_browser_process_->SetCastService(
       cast_browser_process_->browser_client()->CreateCastService(

chromecast/browser/cast_browser_process.cc

@@ -17,7 +17,7 @@
 #include "components/prefs/pref_service.h"
 
 #if defined(USE_AURA)
-#include "chromecast/graphics/accessibility/accessibility_manager.h"
+#include "chromecast/browser/accessibility/accessibility_manager.h"
 #include "chromecast/graphics/cast_screen.h"
 #endif  // defined(USE_AURA)
 

chromecast/browser/cast_browser_process.h

@@ -19,9 +19,6 @@ class NetLog;
 }  // namespace net
 
 namespace chromecast {
-#if defined(USE_AURA)
-class AccessibilityManager;
-#endif  // defined(USE_AURA)
 class CastService;
 class CastScreen;
 class ConnectivityChecker;
@@ -31,6 +28,9 @@ class CastMetricsServiceClient;
 }  // namespace metrics
 
 namespace shell {
+#if defined(USE_AURA)
+class AccessibilityManager;
+#endif  // defined(USE_AURA)
 class CastBrowserContext;
 class CastContentBrowserClient;
 class RemoteDebuggingServer;

chromecast/browser/extensions/api/accessibility_private/accessibility_extension_api.cc

@@ -4,9 +4,9 @@
 
 #include "chromecast/browser/extensions/api/accessibility_private/accessibility_extension_api.h"
 
+#include "chromecast/browser/accessibility/accessibility_manager.h"
 #include "chromecast/browser/cast_browser_process.h"
 #include "chromecast/common/extensions_api/accessibility_private.h"
-#include "chromecast/graphics/accessibility/accessibility_manager.h"
 #include "content/public/browser/browser_accessibility_state.h"
 #include "content/public/browser/render_frame_host.h"
 #include "content/public/common/service_manager_connection.h"

chromecast/graphics/BUILD.gn

@@ -32,8 +32,6 @@ cast_source_set("graphics") {
       "accessibility/accessibility_highlight_layer.h",
       "accessibility/accessibility_layer.cc",
       "accessibility/accessibility_layer.h",
-      "accessibility/accessibility_manager.cc",
-      "accessibility/accessibility_manager.h",
       "accessibility/accessibility_observer.h",
       "accessibility/focus_ring_controller.cc",
       "accessibility/focus_ring_controller.h",