Block focus actions for accessibility-blocked nodes

Author: JSUYA

flutter/shell/platform/common/flutter_platform_node_delegate.cc

@@ -38,6 +38,9 @@ bool FlutterPlatformNodeDelegate::AccessibilityPerformAction(
           target, FlutterSemanticsAction::kFlutterSemanticsActionTap, {});
       return true;
     case ax::mojom::Action::kFocus:
+      if (!CanReceiveAccessibilityFocus()) {
+        return false;
+      }
       bridge_ptr->SetLastFocusedId(target);
       bridge_ptr->DispatchAccessibilityAction(
           target,
@@ -57,6 +60,11 @@ bool FlutterPlatformNodeDelegate::AccessibilityPerformAction(
   return false;
 }
 
+bool FlutterPlatformNodeDelegate::CanReceiveAccessibilityFocus() const {
+  return !GetData().IsIgnored() &&
+         GetData().HasState(ax::mojom::State::kFocusable);
+}
+
 const ui::AXNodeData& FlutterPlatformNodeDelegate::GetData() const {
   return ax_node_->data();
 }

flutter/shell/platform/common/flutter_platform_node_delegate.h

@@ -172,6 +172,8 @@ class FlutterPlatformNodeDelegate : public ui::AXPlatformNodeDelegateBase {
   virtual const ui::AXTree::Selection GetUnignoredSelection() const override;
 
  private:
+  bool CanReceiveAccessibilityFocus() const;
+
   ui::AXNode* ax_node_;
   std::weak_ptr<OwnerBridge> bridge_;
   ui::AXUniqueId unique_id_;

flutter/shell/platform/tizen/BUILD.gn

@@ -246,6 +246,7 @@ executable("flutter_tizen_unittests") {
   testonly = true
 
   sources = [
+    "accessibility_bridge_unittests.cc",
     "channels/lifecycle_channel_unittests.cc",
     "channels/settings_channel_unittests.cc",
     "flutter_project_bundle_unittests.cc",

flutter/shell/platform/tizen/accessibility_bridge_unittests.cc

@@ -0,0 +1,104 @@
+// Copyright 2026 Samsung Electronics Co., Ltd. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "flutter/shell/platform/common/test_accessibility_bridge.h"
+
+#include "flutter/third_party/accessibility/ax/ax_action_data.h"
+#include "gtest/gtest.h"
+
+namespace flutter {
+namespace {
+
+FlutterTransformation IdentityTransform() {
+  return FlutterTransformation{
+      1.0, 0.0, 0.0,
+      0.0, 1.0, 0.0,
+      0.0, 0.0, 1.0,
+  };
+}
+
+FlutterSemanticsFlags MakeFlags(bool blocked) {
+  FlutterSemanticsFlags flags = {};
+  flags.struct_size = sizeof(FlutterSemanticsFlags);
+  flags.is_accessibility_focus_blocked = blocked;
+  return flags;
+}
+
+FlutterSemanticsNode2 MakeNode(int32_t id, FlutterSemanticsFlags* flags) {
+  FlutterSemanticsNode2 node = {};
+  node.struct_size = sizeof(FlutterSemanticsNode2);
+  node.id = id;
+  node.actions = kFlutterSemanticsActionTap;
+  node.label = "target";
+  node.text_direction = kFlutterTextDirectionLTR;
+  node.rect = FlutterRect{0.0, 0.0, 10.0, 10.0};
+  node.transform = IdentityTransform();
+  node.platform_view_id = -1;
+  node.flags2 = flags;
+  return node;
+}
+
+std::shared_ptr<FlutterPlatformNodeDelegate> GetDelegate(
+    TestAccessibilityBridge& bridge,
+    int32_t node_id) {
+  auto delegate = bridge.GetFlutterPlatformNodeDelegateFromID(node_id).lock();
+  EXPECT_NE(delegate, nullptr);
+  return delegate;
+}
+
+}  // namespace
+
+TEST(AccessibilityBridgeTest, BlockedNodeDoesNotBecomeFocusable) {
+  TestAccessibilityBridge bridge;
+  auto flags = MakeFlags(true);
+  auto node = MakeNode(1, &flags);
+
+  bridge.AddFlutterSemanticsNodeUpdate(node);
+  bridge.CommitUpdates();
+
+  auto* ax_node = bridge.GetNodeFromTree(1);
+  ASSERT_NE(ax_node, nullptr);
+  EXPECT_FALSE(ax_node->data().HasState(ax::mojom::State::kFocusable));
+}
+
+TEST(AccessibilityBridgeTest, BlockedNodeRejectsAccessibilityFocusAction) {
+  TestAccessibilityBridge bridge;
+  auto flags = MakeFlags(true);
+  auto node = MakeNode(1, &flags);
+
+  bridge.AddFlutterSemanticsNodeUpdate(node);
+  bridge.CommitUpdates();
+
+  auto delegate = GetDelegate(bridge, node.id);
+  ASSERT_NE(delegate, nullptr);
+
+  ui::AXActionData action_data;
+  action_data.action = ax::mojom::Action::kFocus;
+
+  EXPECT_FALSE(delegate->AccessibilityPerformAction(action_data));
+  EXPECT_TRUE(bridge.performed_actions.empty());
+  EXPECT_EQ(bridge.GetLastFocusedId(), ui::AXNode::kInvalidAXID);
+}
+
+TEST(AccessibilityBridgeTest, UnblockedNodeAcceptsAccessibilityFocusAction) {
+  TestAccessibilityBridge bridge;
+  auto flags = MakeFlags(false);
+  auto node = MakeNode(1, &flags);
+
+  bridge.AddFlutterSemanticsNodeUpdate(node);
+  bridge.CommitUpdates();
+
+  auto delegate = GetDelegate(bridge, node.id);
+  ASSERT_NE(delegate, nullptr);
+
+  ui::AXActionData action_data;
+  action_data.action = ax::mojom::Action::kFocus;
+
+  EXPECT_TRUE(delegate->AccessibilityPerformAction(action_data));
+  ASSERT_EQ(bridge.performed_actions.size(), 1u);
+  EXPECT_EQ(
+      bridge.performed_actions.front(),
+      FlutterSemanticsAction::kFlutterSemanticsActionDidGainAccessibilityFocus);
+  EXPECT_EQ(bridge.GetLastFocusedId(), node.id);
+}

flutter/third_party/accessibility/ax/platform/ax_platform_node_auralinux.cc

@@ -4358,6 +4358,9 @@ AXPlatformNodeAuraLinux::HitTestSync(gint x, gint y, AtkCoordType coord_type) {
 }
 
 bool AXPlatformNodeAuraLinux::GrabFocus() {
+  if (!GetData().HasState(ax::mojom::State::kFocusable))
+    return false;
+
   AXActionData action_data;
   action_data.action = ax::mojom::Action::kFocus;
   return delegate_->AccessibilityPerformAction(action_data);