Revert " Add parameters to SemanticActions; implement extend selection for a11y (flutter#4444)" (flutter#4448)

goderbauer · web-flow · commit 056fd4597f1e · 2017-12-12T10:53:15.000-08:00
This reverts commit 59c3a37.
diff --git a/lib/ui/hooks.dart b/lib/ui/hooks.dart
@@ -85,13 +85,12 @@ void _dispatchPointerDataPacket(ByteData packet) {
     _invoke1<PointerDataPacket>(window.onPointerDataPacket, window._onPointerDataPacketZone, _unpackPointerDataPacket(packet));
 }
 
-void _dispatchSemanticsAction(int id, int action, ByteData args) {
-  _invoke3<int, SemanticsAction, ByteData>(
+void _dispatchSemanticsAction(int id, int action) {
+  _invoke2<int, SemanticsAction>(
     window.onSemanticsAction,
     window._onSemanticsActionZone,
     id,
     SemanticsAction.values[action],
-    args,
   );
 }
 
diff --git a/lib/ui/window.dart b/lib/ui/window.dart
@@ -14,7 +14,7 @@ typedef void FrameCallback(Duration duration);
 typedef void PointerDataPacketCallback(PointerDataPacket packet);
 
 /// Signature for [Window.onSemanticsAction].
-typedef void SemanticsActionCallback(int id, SemanticsAction action, ByteData args);
+typedef void SemanticsActionCallback(int id, SemanticsAction action);
 
 /// Signature for responses to platform messages.
 ///
@@ -439,7 +439,7 @@ class Window {
 
   /// The setting indicating whether time should always be shown in the 24-hour
   /// format.
-  ///
+  /// 
   /// This option is used by [showTimePicker].
   bool get alwaysUse24HourFormat => _alwaysUse24HourFormat;
   bool _alwaysUse24HourFormat = false;
diff --git a/lib/ui/window/window.cc b/lib/ui/window/window.cc
@@ -223,22 +223,14 @@ void Window::DispatchPointerDataPacket(const PointerDataPacket& packet) {
                   {data_handle});
 }
 
-void Window::DispatchSemanticsAction(int32_t id,
-                                     SemanticsAction action,
-                                     std::vector<uint8_t> args) {
+void Window::DispatchSemanticsAction(int32_t id, SemanticsAction action) {
   tonic::DartState* dart_state = library_.dart_state().get();
   if (!dart_state)
     return;
   tonic::DartState::Scope scope(dart_state);
 
-  Dart_Handle args_handle = (args.empty()) ? Dart_Null() : ToByteData(args);
-
-  if (Dart_IsError(args_handle))
-    return;
-
-  DartInvokeField(
-      library_.value(), "_dispatchSemanticsAction",
-      {ToDart(id), ToDart(static_cast<int32_t>(action)), args_handle});
+  DartInvokeField(library_.value(), "_dispatchSemanticsAction",
+                  {ToDart(id), ToDart(static_cast<int32_t>(action))});
 }
 
 void Window::BeginFrame(fxl::TimePoint frameTime) {
diff --git a/lib/ui/window/window.h b/lib/ui/window/window.h
@@ -49,9 +49,7 @@ class Window {
   void UpdateSemanticsEnabled(bool enabled);
   void DispatchPlatformMessage(fxl::RefPtr<PlatformMessage> message);
   void DispatchPointerDataPacket(const PointerDataPacket& packet);
-  void DispatchSemanticsAction(int32_t id,
-                               SemanticsAction action,
-                               std::vector<uint8_t> args);
+  void DispatchSemanticsAction(int32_t id, SemanticsAction action);
   void BeginFrame(fxl::TimePoint frameTime);
 
   void CompletePlatformMessageResponse(int response_id,
diff --git a/runtime/runtime_controller.cc b/runtime/runtime_controller.cc
@@ -106,11 +106,10 @@ void RuntimeController::DispatchPointerDataPacket(
 }
 
 void RuntimeController::DispatchSemanticsAction(int32_t id,
-                                                SemanticsAction action,
-                                                std::vector<uint8_t> args) {
+                                                SemanticsAction action) {
   TRACE_EVENT1("flutter", "RuntimeController::DispatchSemanticsAction", "mode",
                "basic");
-  GetWindow()->DispatchSemanticsAction(id, action, std::move(args));
+  GetWindow()->DispatchSemanticsAction(id, action);
 }
 
 Window* RuntimeController::GetWindow() {
diff --git a/runtime/runtime_controller.h b/runtime/runtime_controller.h
@@ -42,9 +42,7 @@ class RuntimeController : public WindowClient, public IsolateClient {
 
   void DispatchPlatformMessage(fxl::RefPtr<PlatformMessage> message);
   void DispatchPointerDataPacket(const PointerDataPacket& packet);
-  void DispatchSemanticsAction(int32_t id,
-                               SemanticsAction action,
-                               std::vector<uint8_t> args);
+  void DispatchSemanticsAction(int32_t id, SemanticsAction action);
 
   Dart_Port GetMainPort();
   std::string GetIsolateName();
diff --git a/shell/common/engine.cc b/shell/common/engine.cc
@@ -554,11 +554,9 @@ void Engine::DispatchPointerDataPacket(const PointerDataPacket& packet) {
     runtime_->DispatchPointerDataPacket(packet);
 }
 
-void Engine::DispatchSemanticsAction(int id,
-                                     blink::SemanticsAction action,
-                                     std::vector<uint8_t> args) {
+void Engine::DispatchSemanticsAction(int id, blink::SemanticsAction action) {
   if (runtime_)
-    runtime_->DispatchSemanticsAction(id, action, std::move(args));
+    runtime_->DispatchSemanticsAction(id, action);
 }
 
 void Engine::SetSemanticsEnabled(bool enabled) {
diff --git a/shell/common/engine.h b/shell/common/engine.h
@@ -72,9 +72,7 @@ class Engine : public blink::RuntimeDelegate {
   void SetViewportMetrics(const blink::ViewportMetrics& metrics);
   void DispatchPlatformMessage(fxl::RefPtr<blink::PlatformMessage> message);
   void DispatchPointerDataPacket(const PointerDataPacket& packet);
-  void DispatchSemanticsAction(int id,
-                               blink::SemanticsAction action,
-                               std::vector<uint8_t> args);
+  void DispatchSemanticsAction(int id, blink::SemanticsAction action);
   void SetSemanticsEnabled(bool enabled);
   void ScheduleFrame(bool regenerate_layer_tree = true) override;
 
diff --git a/shell/common/platform_view.cc b/shell/common/platform_view.cc
@@ -55,13 +55,12 @@ void PlatformView::DispatchPlatformMessage(
 }
 
 void PlatformView::DispatchSemanticsAction(int32_t id,
-                                           blink::SemanticsAction action,
-                                           std::vector<uint8_t> args) {
+                                           blink::SemanticsAction action) {
   blink::Threads::UI()->PostTask(
-      [ engine = engine_->GetWeakPtr(), id, action, args = std::move(args) ] {
+      [ engine = engine_->GetWeakPtr(), id, action ] {
         if (engine) {
           engine->DispatchSemanticsAction(
-              id, static_cast<blink::SemanticsAction>(action), std::move(args));
+              id, static_cast<blink::SemanticsAction>(action));
         }
       });
 }
diff --git a/shell/common/platform_view.h b/shell/common/platform_view.h
@@ -41,9 +41,7 @@ class PlatformView : public std::enable_shared_from_this<PlatformView> {
   virtual void Attach() = 0;
 
   void DispatchPlatformMessage(fxl::RefPtr<blink::PlatformMessage> message);
-  void DispatchSemanticsAction(int32_t id,
-                               blink::SemanticsAction action,
-                               std::vector<uint8_t> args);
+  void DispatchSemanticsAction(int32_t id, blink::SemanticsAction action);
   void SetSemanticsEnabled(bool enabled);
 
   void NotifyCreated(std::unique_ptr<Surface> surface);
diff --git a/shell/platform/android/io/flutter/view/AccessibilityBridge.java b/shell/platform/android/io/flutter/view/AccessibilityBridge.java
@@ -312,16 +312,17 @@ public boolean performAction(int virtualViewId, int action, Bundle arguments) {
     boolean performCursorMoveAction(SemanticsObject object, int virtualViewId, Bundle arguments, boolean forward) {
         final int granularity = arguments.getInt(
             AccessibilityNodeInfo.ACTION_ARGUMENT_MOVEMENT_GRANULARITY_INT);
-        final boolean extendSelection = arguments.getBoolean(
-            AccessibilityNodeInfo.ACTION_ARGUMENT_EXTEND_SELECTION_BOOLEAN);
+        // TODO(goderbauer): support extending selections.
+        // final boolean extendSelection = arguments.getBoolean(
+        //     AccessibilityNodeInfo.ACTION_ARGUMENT_EXTEND_SELECTION_BOOLEAN);
         switch (granularity) {
             case AccessibilityNodeInfo.MOVEMENT_GRANULARITY_CHARACTER: {
                 if (forward && object.hasAction(Action.MOVE_CURSOR_FORWARD_BY_CHARACTER)) {
-                    mOwner.dispatchSemanticsAction(virtualViewId, Action.MOVE_CURSOR_FORWARD_BY_CHARACTER, extendSelection);
+                    mOwner.dispatchSemanticsAction(virtualViewId, Action.MOVE_CURSOR_FORWARD_BY_CHARACTER);
                     return true;
                 }
                 if (!forward && object.hasAction(Action.MOVE_CURSOR_BACKWARD_BY_CHARACTER)) {
-                    mOwner.dispatchSemanticsAction(virtualViewId, Action.MOVE_CURSOR_BACKWARD_BY_CHARACTER, extendSelection);
+                    mOwner.dispatchSemanticsAction(virtualViewId, Action.MOVE_CURSOR_BACKWARD_BY_CHARACTER);
                     return true;
                 }
             }
diff --git a/shell/platform/android/io/flutter/view/FlutterView.java b/shell/platform/android/io/flutter/view/FlutterView.java
@@ -682,7 +682,7 @@ private static native void nativeDispatchPointerDataPacket(long nativePlatformVi
         ByteBuffer buffer, int position);
 
     private static native void nativeDispatchSemanticsAction(long nativePlatformViewAndroid, int id,
-        int action, ByteBuffer args, int argsPosition);
+        int action);
 
     private static native void nativeSetSemanticsEnabled(long nativePlatformViewAndroid,
         boolean enabled);
@@ -743,19 +743,9 @@ public void onFirstFrame() {
     private TouchExplorationListener mTouchExplorationListener;
 
     protected void dispatchSemanticsAction(int id, AccessibilityBridge.Action action) {
-        dispatchSemanticsAction(id, action, null);
-    }
-
-    protected void dispatchSemanticsAction(int id, AccessibilityBridge.Action action, Object args) {
         if (!isAttached())
             return;
-        ByteBuffer encodedArgs = null;
-        int position = 0;
-        if (args != null) {
-            encodedArgs = StandardMessageCodec.INSTANCE.encodeMessage(args);
-            position = encodedArgs.position();
-        }
-        nativeDispatchSemanticsAction(mNativeView.get(), id, action.value, encodedArgs, position);
+        nativeDispatchSemanticsAction(mNativeView.get(), id, action.value);
     }
 
     @Override
diff --git a/shell/platform/android/platform_view_android.cc b/shell/platform/android/platform_view_android.cc
@@ -410,24 +410,9 @@ void PlatformViewAndroid::HandlePlatformMessageEmptyResponse(int response_id) {
                                            nullptr);
 }
 
-void PlatformViewAndroid::DispatchSemanticsAction(JNIEnv* env,
-                                                  jint id,
-                                                  jint action,
-                                                  jobject args,
-                                                  jint args_position) {
-  if (env->IsSameObject(args, NULL)) {
-    std::vector<uint8_t> args_vector;
-    PlatformView::DispatchSemanticsAction(
-        id, static_cast<blink::SemanticsAction>(action), args_vector);
-    return;
-  }
-
-  uint8_t* args_data = static_cast<uint8_t*>(env->GetDirectBufferAddress(args));
-  std::vector<uint8_t> args_vector =
-      std::vector<uint8_t>(args_data, args_data + args_position);
-
+void PlatformViewAndroid::DispatchSemanticsAction(jint id, jint action) {
   PlatformView::DispatchSemanticsAction(
-      id, static_cast<blink::SemanticsAction>(action), std::move(args_vector));
+      id, static_cast<blink::SemanticsAction>(action));
 }
 
 void PlatformViewAndroid::SetSemanticsEnabled(jboolean enabled) {
diff --git a/shell/platform/android/platform_view_android.h b/shell/platform/android/platform_view_android.h
@@ -79,11 +79,7 @@ class PlatformViewAndroid : public PlatformView {
   void InvokePlatformMessageEmptyResponseCallback(JNIEnv* env,
                                                   jint response_id);
 
-  void DispatchSemanticsAction(JNIEnv* env,
-                               jint id,
-                               jint action,
-                               jobject args,
-                               jint args_position);
+  void DispatchSemanticsAction(jint id, jint action);
 
   void SetSemanticsEnabled(jboolean enabled);
 
diff --git a/shell/platform/android/platform_view_android_jni.cc b/shell/platform/android/platform_view_android_jni.cc
@@ -236,11 +236,8 @@ static void DispatchSemanticsAction(JNIEnv* env,
                                     jobject jcaller,
                                     jlong platform_view,
                                     jint id,
-                                    jint action,
-                                    jobject args,
-                                    jint args_position) {
-  return PLATFORM_VIEW->DispatchSemanticsAction(env, id, action, args,
-                                                args_position);
+                                    jint action) {
+  return PLATFORM_VIEW->DispatchSemanticsAction(id, action);
 }
 
 static void SetSemanticsEnabled(JNIEnv* env,
@@ -416,7 +413,7 @@ bool PlatformViewAndroid::Register(JNIEnv* env) {
       },
       {
           .name = "nativeDispatchSemanticsAction",
-          .signature = "(JIILjava/nio/ByteBuffer;I)V",
+          .signature = "(JII)V",
           .fnPtr = reinterpret_cast<void*>(&shell::DispatchSemanticsAction),
       },
       {

Original file line number	Diff line number	Diff line change
`@@ -85,13 +85,12 @@ void _dispatchPointerDataPacket(ByteData packet) {`
`85`	`85`	`_invoke1<PointerDataPacket>(window.onPointerDataPacket, window._onPointerDataPacketZone, _unpackPointerDataPacket(packet));`
`86`	`86`	`}`
`87`	`87`
`88`		`-void _dispatchSemanticsAction(int id, int action, ByteData args) {`
`89`		`- _invoke3<int, SemanticsAction, ByteData>(`
	`88`	`+void _dispatchSemanticsAction(int id, int action) {`
	`89`	`+ _invoke2<int, SemanticsAction>(`
`90`	`90`	`window.onSemanticsAction,`
`91`	`91`	`window._onSemanticsActionZone,`
`92`	`92`	`id,`
`93`	`93`	`SemanticsAction.values[action],`
`94`		`- args,`
`95`	`94`	`);`
`96`	`95`	`}`
`97`	`96`
Original file line number	Diff line number	Diff line change
`@@ -106,11 +106,10 @@ void RuntimeController::DispatchPointerDataPacket(`
`106`	`106`	`}`
`107`	`107`
`108`	`108`	`void RuntimeController::DispatchSemanticsAction(int32_t id,`
`109`		`- SemanticsAction action,`
`110`		`- std::vector<uint8_t> args) {`
	`109`	`+ SemanticsAction action) {`
`111`	`110`	`TRACE_EVENT1("flutter", "RuntimeController::DispatchSemanticsAction", "mode",`
`112`	`111`	`"basic");`
`113`		`- GetWindow()->DispatchSemanticsAction(id, action, std::move(args));`
	`112`	`+ GetWindow()->DispatchSemanticsAction(id, action);`
`114`	`113`	`}`
`115`	`114`
`116`	`115`	`Window* RuntimeController::GetWindow() {`
Original file line number	Diff line number	Diff line change
`@@ -554,11 +554,9 @@ void Engine::DispatchPointerDataPacket(const PointerDataPacket& packet) {`
`554`	`554`	`runtime_->DispatchPointerDataPacket(packet);`
`555`	`555`	`}`
`556`	`556`
`557`		`-void Engine::DispatchSemanticsAction(int id,`
`558`		`- blink::SemanticsAction action,`
`559`		`- std::vector<uint8_t> args) {`
	`557`	`+void Engine::DispatchSemanticsAction(int id, blink::SemanticsAction action) {`
`560`	`558`	`if (runtime_)`
`561`		`- runtime_->DispatchSemanticsAction(id, action, std::move(args));`
	`559`	`+ runtime_->DispatchSemanticsAction(id, action);`
`562`	`560`	`}`
`563`	`561`
`564`	`562`	`void Engine::SetSemanticsEnabled(bool enabled) {`
Original file line number	Diff line number	Diff line change
`@@ -55,13 +55,12 @@ void PlatformView::DispatchPlatformMessage(`
`55`	`55`	`}`
`56`	`56`
`57`	`57`	`void PlatformView::DispatchSemanticsAction(int32_t id,`
`58`		`- blink::SemanticsAction action,`
`59`		`- std::vector<uint8_t> args) {`
	`58`	`+ blink::SemanticsAction action) {`
`60`	`59`	`blink::Threads::UI()->PostTask(`
`61`		`- [ engine = engine_->GetWeakPtr(), id, action, args = std::move(args) ] {`
	`60`	`+ [ engine = engine_->GetWeakPtr(), id, action ] {`
`62`	`61`	`if (engine) {`
`63`	`62`	`engine->DispatchSemanticsAction(`
`64`		`- id, static_cast<blink::SemanticsAction>(action), std::move(args));`
	`63`	`+ id, static_cast<blink::SemanticsAction>(action));`
`65`	`64`	`}`
`66`	`65`	`});`
`67`	`66`	`}`