[flutter_runner] a11y updates, tests!

ci/licenses_golden/licenses_flutter

@@ -928,13 +928,15 @@ FILE: ../../../flutter/shell/platform/fuchsia/dart_runner/vmservice/empty.dart
 FILE: ../../../flutter/shell/platform/fuchsia/dart_runner/vmservice/meta/vmservice.cmx
 FILE: ../../../flutter/shell/platform/fuchsia/flutter/accessibility_bridge.cc
 FILE: ../../../flutter/shell/platform/fuchsia/flutter/accessibility_bridge.h
+FILE: ../../../flutter/shell/platform/fuchsia/flutter/accessibility_bridge_unittest.cc
 FILE: ../../../flutter/shell/platform/fuchsia/flutter/collect_traces.dart
 FILE: ../../../flutter/shell/platform/fuchsia/flutter/component.cc
 FILE: ../../../flutter/shell/platform/fuchsia/flutter/component.h
 FILE: ../../../flutter/shell/platform/fuchsia/flutter/compositor_context.cc
 FILE: ../../../flutter/shell/platform/fuchsia/flutter/compositor_context.h
 FILE: ../../../flutter/shell/platform/fuchsia/flutter/engine.cc
 FILE: ../../../flutter/shell/platform/fuchsia/flutter/engine.h
+FILE: ../../../flutter/shell/platform/fuchsia/flutter/flutter_runner_fakes.h
 FILE: ../../../flutter/shell/platform/fuchsia/flutter/isolate_configurator.cc
 FILE: ../../../flutter/shell/platform/fuchsia/flutter/isolate_configurator.h
 FILE: ../../../flutter/shell/platform/fuchsia/flutter/kernel/extract_far.dart
@@ -955,9 +957,9 @@ FILE: ../../../flutter/shell/platform/fuchsia/flutter/meta/jit_product_runtime
 FILE: ../../../flutter/shell/platform/fuchsia/flutter/meta/jit_runtime
 FILE: ../../../flutter/shell/platform/fuchsia/flutter/platform_view.cc
 FILE: ../../../flutter/shell/platform/fuchsia/flutter/platform_view.h
+FILE: ../../../flutter/shell/platform/fuchsia/flutter/platform_view_unittest.cc
 FILE: ../../../flutter/shell/platform/fuchsia/flutter/runner.cc
 FILE: ../../../flutter/shell/platform/fuchsia/flutter/runner.h
-FILE: ../../../flutter/shell/platform/fuchsia/flutter/sample_unittests.cc
 FILE: ../../../flutter/shell/platform/fuchsia/flutter/session_connection.cc
 FILE: ../../../flutter/shell/platform/fuchsia/flutter/session_connection.h
 FILE: ../../../flutter/shell/platform/fuchsia/flutter/surface.cc

shell/platform/fuchsia/flutter/BUILD.gn

@@ -247,14 +247,39 @@ executable("flutter_runner_unittests") {
   output_name = "flutter_runner_tests"
 
   sources = [
-    "sample_unittests.cc",
+    "accessibility_bridge.cc",
+    "accessibility_bridge.h",
+    "accessibility_bridge_unittest.cc",
+    "flutter_runner_fakes.h",
+    "logging.h",
+    "platform_view.cc",
+    "platform_view.h",
+    "platform_view_unittest.cc",
+    "surface.cc",
+    "surface.h",
+    "vsync_recorder.cc",
+    "vsync_recorder.h",
+    "vsync_waiter.cc",
+    "vsync_waiter.h",
   ]
 
   # This is needed for //third_party/googletest for linking zircon symbols.
   libs = [ "//fuchsia/sdk/$host_os/arch/$target_cpu/sysroot/lib/libzircon.so" ]
 
   deps = [
+    ":aot",
+    "//build/fuchsia/fidl:fuchsia.accessibility.semantics",
+    "//build/fuchsia/fidl:fuchsia.modular",
+    "//build/fuchsia/pkg:async-loop-cpp",
+    "//build/fuchsia/pkg:async-loop-default",
+    "//build/fuchsia/pkg:scenic_cpp",
+    "//build/fuchsia/pkg:sys_cpp_testing",
+    "//flutter/lib/ui",
+    "//flutter/shell/common",
+    "//flutter/shell/platform/fuchsia/runtime/dart/utils",
     "//flutter/testing",
+    "//third_party/dart/runtime:libdart_jit",
+    "//third_party/dart/runtime/platform:libdart_platform_jit",
   ]
 }
 

shell/platform/fuchsia/flutter/accessibility_bridge.cc

@@ -14,9 +14,10 @@
 
 namespace flutter_runner {
 AccessibilityBridge::AccessibilityBridge(
+    Delegate& delegate,
     const std::shared_ptr<sys::ServiceDirectory> services,
     fuchsia::ui::views::ViewRef view_ref)
-    : binding_(this) {
+    : delegate_(delegate), binding_(this) {
   services->Connect(fuchsia::accessibility::semantics::SemanticsManager::Name_,
                     fuchsia_semantics_manager_.NewRequest().TakeChannel());
   fuchsia_semantics_manager_.set_error_handler([](zx_status_t status) {
@@ -240,4 +241,11 @@ void AccessibilityBridge::HitTest(
     fuchsia::accessibility::semantics::SemanticListener::HitTestCallback
         callback) {}
 
+// |fuchsia::accessibility::semantics::SemanticListener|
+void AccessibilityBridge::OnSemanticsModeChanged(
+    bool enabled,
+    OnSemanticsModeChangedCallback callback) {
+  delegate_.SetSemanticsEnabled(enabled);
+}
+
 }  // namespace flutter_runner

shell/platform/fuchsia/flutter/accessibility_bridge.h

@@ -36,6 +36,12 @@ namespace flutter_runner {
 class AccessibilityBridge
     : public fuchsia::accessibility::semantics::SemanticListener {
  public:
+  // A delegate to call when semantics are enabled or disabled.
+  class Delegate {
+   public:
+    virtual void SetSemanticsEnabled(bool enabled) = 0;
+  };
+
   // TODO(MI4-2531, FIDL-718): Remove this. We shouldn't be worried about
   // batching messages at this level.
   // FIDL may encode a C++ struct as larger than the sizeof the C++ struct.
@@ -55,7 +61,8 @@ class AccessibilityBridge
       "flutter::SemanticsNode::id and "
       "fuchsia::accessibility::semantics::Node::node_id differ in size.");
 
-  AccessibilityBridge(const std::shared_ptr<sys::ServiceDirectory> services,
+  AccessibilityBridge(Delegate& delegate,
+                      const std::shared_ptr<sys::ServiceDirectory> services,
                       fuchsia::ui::views::ViewRef view_ref);
 
   // Returns true if accessible navigation is enabled.
@@ -75,6 +82,8 @@ class AccessibilityBridge
   zx_status_t OnHoverMove(double x, double y);
 
  private:
+  AccessibilityBridge::Delegate& delegate_;
+
   static constexpr int32_t kRootNodeId = 0;
   fidl::Binding<fuchsia::accessibility::semantics::SemanticListener> binding_;
   fuchsia::accessibility::semantics::SemanticsManagerPtr
@@ -128,9 +137,8 @@ class AccessibilityBridge
           callback) override;
 
   // |fuchsia::accessibility::semantics::SemanticListener|
-  void OnSemanticsModeChanged(
-      bool enabled,
-      OnSemanticsModeChangedCallback callback) override {}
+  void OnSemanticsModeChanged(bool enabled,
+                              OnSemanticsModeChangedCallback callback) override;
 
   FML_DISALLOW_COPY_AND_ASSIGN(AccessibilityBridge);
 };