[ios_platform_view, a11y] Make FlutterPlatformViewSemanticsContainer a SemanticsObject.

shell/platform/darwin/ios/framework/Source/SemanticsObject.h

@@ -61,11 +61,6 @@ constexpr float kScrollExtentMaxForInf = 1000;
  */
 @property(nonatomic, strong) NSArray<SemanticsObject*>* children;
 
-/**
- * Used if this SemanticsObject is for a platform view.
- */
-@property(strong, nonatomic) FlutterPlatformViewSemanticsContainer* platformViewSemanticsContainer;
-
 /**
  * The UIAccessibility that represents this object.
  *
@@ -159,16 +154,15 @@ constexpr float kScrollExtentMaxForInf = 1000;
  * * `SemanticsObject` for the other type of semantics objects.
  * * `FlutterSemanticsObject` for default implementation of `SemanticsObject`.
  */
-@interface FlutterPlatformViewSemanticsContainer : UIAccessibilityElement
-
-/**
- * The position inside an accessibility container.
- */
-@property(nonatomic) NSInteger index;
+@interface FlutterPlatformViewSemanticsContainer : SemanticsObject
 
-- (instancetype)init __attribute__((unavailable("Use initWithAccessibilityContainer: instead")));
+- (instancetype)initWithBridge:(fml::WeakPtr<flutter::AccessibilityBridgeIos>)bridge
+                           uid:(int32_t)uid
+    __attribute__((unavailable("Use initWithBridge:uid:platformViewId instead")));
 
-- (instancetype)initWithSemanticsObject:(SemanticsObject*)object;
+- (instancetype)initWithBridge:(fml::WeakPtr<flutter::AccessibilityBridgeIos>)bridge
+                           uid:(int32_t)uid
+                platformViewId:(int32_t)platformViewId;
 
 @end
 

shell/platform/darwin/ios/framework/Source/SemanticsObject.mm

@@ -300,7 +300,6 @@ - (void)dealloc {
   [_children release];
   _parent = nil;
   _container.get().semanticsObject = nil;
-  [_platformViewSemanticsContainer release];
   _inDealloc = YES;
   [super dealloc];
 }
@@ -319,9 +318,6 @@ - (void)setChildren:(NSArray<SemanticsObject*>*)children {
 }
 
 - (BOOL)hasChildren {
-  if (_node.IsPlatformViewNode()) {
-    return YES;
-  }
   return [self.children count] != 0;
 }
 
@@ -753,30 +749,27 @@ - (UIAccessibilityTraits)accessibilityTraits {
 @end
 
 @interface FlutterPlatformViewSemanticsContainer ()
-@property(nonatomic, assign) SemanticsObject* semanticsObject;
 @property(nonatomic, strong) UIView* platformView;
 @end
 
 @implementation FlutterPlatformViewSemanticsContainer
 
 // Method declared as unavailable in the interface
-- (instancetype)init {
+- (instancetype)initWithBridge:(fml::WeakPtr<flutter::AccessibilityBridgeIos>)bridge
+                           uid:(int32_t)uid {
   [self release];
   [super doesNotRecognizeSelector:_cmd];
   return nil;
 }
 
-- (instancetype)initWithSemanticsObject:(SemanticsObject*)object {
-  FML_CHECK(object);
-  // Initialize with the UIView as the container.
-  // The UIView will not necessarily be accessibility parent for this object.
-  // The bridge informs the OS of the actual structure via
-  // `accessibilityContainer` and `accessibilityElementAtIndex`.
-  if (self = [super initWithAccessibilityContainer:object.bridge->view()]) {
-    _semanticsObject = object;
-    auto controller = object.bridge->GetPlatformViewsController();
+- (instancetype)initWithBridge:(fml::WeakPtr<flutter::AccessibilityBridgeIos>)bridge
+                           uid:(int32_t)uid
+                platformViewId:(int32_t)platformViewId {
+  if (self = [super initWithBridge:bridge uid:uid]) {
+    auto controller = bridge->GetPlatformViewsController();
+    FML_DCHECK(controller);
     if (controller) {
-      _platformView = [controller->GetPlatformViewByID(object.node.platformViewId) retain];
+      _platformView = [controller->GetPlatformViewByID(platformViewId) retain];
     }
   }
   return self;
@@ -790,47 +783,8 @@ - (void)dealloc {
 
 #pragma mark - UIAccessibilityContainer overrides
 
-- (NSInteger)accessibilityElementCount {
-  // This container should only contain 2 elements:
-  // 1. The semantic object that represents this container.
-  // 2. The platform view object.
-  return 2;
-}
-
-- (nullable id)accessibilityElementAtIndex:(NSInteger)index {
-  FML_DCHECK(index < 2);
-  if (index == 0) {
-    return _semanticsObject.nativeAccessibility;
-  } else {
-    return _platformView;
-  }
-}
-
-- (NSInteger)indexOfAccessibilityElement:(id)element {
-  FML_DCHECK(element == _semanticsObject || element == _platformView);
-  if (element == _semanticsObject) {
-    return 0;
-  } else {
-    return 1;
-  }
-}
-
-#pragma mark - UIAccessibilityElement overrides
-
-- (CGRect)accessibilityFrame {
-  return _semanticsObject.accessibilityFrame;
-}
-
-- (BOOL)isAccessibilityElement {
-  return NO;
-}
-
-- (id)accessibilityContainer {
-  return [_semanticsObject accessibilityContainer];
-}
-
-- (BOOL)accessibilityScroll:(UIAccessibilityScrollDirection)direction {
-  return [_platformView accessibilityScroll:direction];
+- (NSArray*)accessibilityElements {
+  return @[ _platformView ];
 }
 
 @end
@@ -882,12 +836,6 @@ - (nullable id)accessibilityElementAtIndex:(NSInteger)index {
 
   SemanticsObject* child = [_semanticsObject children][index - 1];
 
-  // Swap the original `SemanticsObject` to a `PlatformViewSemanticsContainer`
-  if (child.node.IsPlatformViewNode()) {
-    child.platformViewSemanticsContainer.index = index;
-    return child.platformViewSemanticsContainer;
-  }
-
   if ([child hasChildren])
     return [child accessibilityContainer];
   return child.nativeAccessibility;
@@ -897,11 +845,6 @@ - (NSInteger)indexOfAccessibilityElement:(id)element {
   if (element == _semanticsObject)
     return 0;
 
-  // FlutterPlatformViewSemanticsContainer is always the last element of its parent.
-  if ([element isKindOfClass:[FlutterPlatformViewSemanticsContainer class]]) {
-    return ((FlutterPlatformViewSemanticsContainer*)element).index;
-  }
-
   NSArray<SemanticsObject*>* children = [_semanticsObject children];
   for (size_t i = 0; i < [children count]; i++) {
     SemanticsObject* child = children[i];

shell/platform/darwin/ios/framework/Source/accessibility_bridge.mm

@@ -126,15 +126,6 @@ void PostAccessibilityNotification(UIAccessibilityNotifications notification,
       object.accessibilityCustomActions = accessibilityCustomActions;
     }
 
-    if (object.node.IsPlatformViewNode()) {
-      auto controller = GetPlatformViewsController();
-      if (controller) {
-        object.platformViewSemanticsContainer = [[[FlutterPlatformViewSemanticsContainer alloc]
-            initWithSemanticsObject:object] autorelease];
-      }
-    } else if (object.platformViewSemanticsContainer) {
-      object.platformViewSemanticsContainer = nil;
-    }
     if (needsAnnouncement) {
       // Try to be more polite - iOS 11+ supports
       // UIAccessibilitySpeechAttributeQueueAnnouncement which should avoid
@@ -268,6 +259,13 @@ static void ReplaceSemanticsObject(SemanticsObject* oldObject,
   } else if (node.HasFlag(flutter::SemanticsFlags::kHasImplicitScrolling)) {
     return [[[FlutterScrollableSemanticsObject alloc] initWithBridge:weak_ptr
                                                                  uid:node.id] autorelease];
+  } else if (node.IsPlatformViewNode()) {
+    FlutterPlatformViewSemanticsContainer* platformViewSemanticsContainer =
+        [[[FlutterPlatformViewSemanticsContainer alloc] initWithBridge:weak_ptr
+                                                                   uid:node.id
+                                                        platformViewId:node.platformViewId]
+            autorelease];
+    return platformViewSemanticsContainer;
   } else {
     return [[[FlutterSemanticsObject alloc] initWithBridge:weak_ptr uid:node.id] autorelease];
   }