Revert "Migrate darwin common "framework_shared" target to ARC (#37049)" (#37121)

Author: Chris Yang

shell/platform/darwin/BUILD.gn

@@ -26,20 +26,6 @@ source_set("flutter_channels") {
   sources = [
     "common/buffer_conversions.h",
     "common/buffer_conversions.mm",
-  ]
-
-  deps = [ "//flutter/fml" ]
-
-  public_deps = [ ":flutter_channels_arc" ]
-
-  public_configs = [ "//flutter:config" ]
-}
-
-source_set("flutter_channels_arc") {
-  cflags_objc = flutter_cflags_objc_arc
-  cflags_objcc = flutter_cflags_objcc_arc
-
-  sources = [
     "common/framework/Headers/FlutterBinaryMessenger.h",
     "common/framework/Headers/FlutterChannels.h",
     "common/framework/Headers/FlutterCodecs.h",
@@ -50,12 +36,13 @@ source_set("flutter_channels_arc") {
     "common/framework/Source/FlutterStandardCodec_Internal.h",
   ]
 
-  public = [
-    "common/framework/Headers/FlutterBinaryMessenger.h",
-    "common/framework/Headers/FlutterChannels.h",
-    "common/framework/Headers/FlutterCodecs.h",
-    "common/framework/Headers/FlutterMacros.h",
-    "common/framework/Source/FlutterStandardCodec_Internal.h",
+  deps = [
+    "//flutter/common",
+    "//flutter/flow",
+    "//flutter/fml",
+    "//flutter/runtime",
+    "//flutter/shell/common",
+    "//third_party/skia",
   ]
 
   public_configs = [ "//flutter:config" ]

shell/platform/darwin/common/BUILD.gn

@@ -16,7 +16,15 @@ source_set("common") {
     "command_line.mm",
   ]
 
-  deps = [ "//flutter/fml" ]
+  deps = [
+    "//flutter/common",
+    "//flutter/flow",
+    "//flutter/fml",
+    "//flutter/runtime",
+    "//flutter/shell/common",
+    "//third_party/dart/runtime:dart_api",
+    "//third_party/skia",
+  ]
 
   public_configs = [ "//flutter:config" ]
 }
@@ -30,8 +38,8 @@ config("framework_relative_headers") {
 
 # Framework code shared between iOS and macOS.
 source_set("framework_shared") {
-  cflags_objc = flutter_cflags_objc_arc
-  cflags_objcc = flutter_cflags_objcc_arc
+  cflags_objc = flutter_cflags_objc
+  cflags_objcc = flutter_cflags_objcc
 
   sources = [
     "framework/Source/FlutterChannels.mm",

shell/platform/darwin/common/framework/Headers/FlutterCodecs.h

@@ -344,17 +344,17 @@ FLUTTER_DARWIN_EXPORT
 /**
  * The type of the encoded values.
  */
-@property(readonly, nonatomic, assign) FlutterStandardDataType type;
+@property(readonly, nonatomic) FlutterStandardDataType type;
 
 /**
  * The number of value items encoded.
  */
-@property(readonly, nonatomic, assign) UInt32 elementCount;
+@property(readonly, nonatomic) UInt32 elementCount;
 
 /**
  * The number of bytes used by the encoding of a single value item.
  */
-@property(readonly, nonatomic, assign) UInt8 elementSize;
+@property(readonly, nonatomic) UInt8 elementSize;
 @end
 
 /**

shell/platform/darwin/common/framework/Source/FlutterChannels.mm

@@ -4,8 +4,6 @@
 
 #import "flutter/shell/platform/darwin/common/framework/Headers/FlutterChannels.h"
 
-FLUTTER_ASSERT_ARC
-
 #pragma mark - Basic message channel
 
 static NSString* const kFlutterChannelBuffersChannel = @"dev.flutter/channel-buffers";
@@ -53,9 +51,9 @@ + (instancetype)messageChannelWithName:(NSString*)name
 + (instancetype)messageChannelWithName:(NSString*)name
                        binaryMessenger:(NSObject<FlutterBinaryMessenger>*)messenger
                                  codec:(NSObject<FlutterMessageCodec>*)codec {
-  return [[FlutterBasicMessageChannel alloc] initWithName:name
-                                          binaryMessenger:messenger
-                                                    codec:codec];
+  return [[[FlutterBasicMessageChannel alloc] initWithName:name
+                                           binaryMessenger:messenger
+                                                     codec:codec] autorelease];
 }
 
 - (instancetype)initWithName:(NSString*)name
@@ -71,13 +69,21 @@ - (instancetype)initWithName:(NSString*)name
                    taskQueue:(NSObject<FlutterTaskQueue>*)taskQueue {
   self = [super init];
   NSAssert(self, @"Super init cannot be nil");
-  _name = [name copy];
-  _messenger = messenger;
-  _codec = codec;
-  _taskQueue = taskQueue;
+  _name = [name retain];
+  _messenger = [messenger retain];
+  _codec = [codec retain];
+  _taskQueue = [taskQueue retain];
   return self;
 }
 
+- (void)dealloc {
+  [_name release];
+  [_messenger release];
+  [_codec release];
+  [_taskQueue release];
+  [super dealloc];
+}
+
 - (void)sendMessage:(id)message {
   [_messenger sendOnChannel:_name message:[_codec encode:message]];
 }
@@ -102,7 +108,7 @@ - (void)setMessageHandler:(FlutterMessageHandler)handler {
     return;
   }
   // Grab reference to avoid retain on self.
-  __weak NSObject<FlutterMessageCodec>* codec = _codec;
+  NSObject<FlutterMessageCodec>* codec = _codec;
   FlutterBinaryMessageHandler messageHandler = ^(NSData* message, FlutterBinaryReply callback) {
     handler([codec decode:message], ^(id reply) {
       callback([codec encode:reply]);
@@ -122,19 +128,26 @@ - (void)resizeChannelBuffer:(NSInteger)newSize {
 ////////////////////////////////////////////////////////////////////////////////
 @implementation FlutterError
 + (instancetype)errorWithCode:(NSString*)code message:(NSString*)message details:(id)details {
-  return [[FlutterError alloc] initWithCode:code message:message details:details];
+  return [[[FlutterError alloc] initWithCode:code message:message details:details] autorelease];
 }
 
 - (instancetype)initWithCode:(NSString*)code message:(NSString*)message details:(id)details {
   NSAssert(code, @"Code cannot be nil");
   self = [super init];
   NSAssert(self, @"Super init cannot be nil");
-  _code = [code copy];
-  _message = [message copy];
-  _details = details;
+  _code = [code retain];
+  _message = [message retain];
+  _details = [details retain];
   return self;
 }
 
+- (void)dealloc {
+  [_code release];
+  [_message release];
+  [_details release];
+  [super dealloc];
+}
+
 - (BOOL)isEqual:(id)object {
   if (self == object) {
     return YES;
@@ -156,18 +169,24 @@ - (NSUInteger)hash {
 ////////////////////////////////////////////////////////////////////////////////
 @implementation FlutterMethodCall
 + (instancetype)methodCallWithMethodName:(NSString*)method arguments:(id)arguments {
-  return [[FlutterMethodCall alloc] initWithMethodName:method arguments:arguments];
+  return [[[FlutterMethodCall alloc] initWithMethodName:method arguments:arguments] autorelease];
 }
 
 - (instancetype)initWithMethodName:(NSString*)method arguments:(id)arguments {
   NSAssert(method, @"Method name cannot be nil");
   self = [super init];
   NSAssert(self, @"Super init cannot be nil");
-  _method = [method copy];
-  _arguments = arguments;
+  _method = [method retain];
+  _arguments = [arguments retain];
   return self;
 }
 
+- (void)dealloc {
+  [_method release];
+  [_arguments release];
+  [super dealloc];
+}
+
 - (BOOL)isEqual:(id)object {
   if (self == object) {
     return YES;
@@ -205,7 +224,8 @@ + (instancetype)methodChannelWithName:(NSString*)name
 + (instancetype)methodChannelWithName:(NSString*)name
                       binaryMessenger:(NSObject<FlutterBinaryMessenger>*)messenger
                                 codec:(NSObject<FlutterMethodCodec>*)codec {
-  return [[FlutterMethodChannel alloc] initWithName:name binaryMessenger:messenger codec:codec];
+  return [[[FlutterMethodChannel alloc] initWithName:name binaryMessenger:messenger
+                                               codec:codec] autorelease];
 }
 
 - (instancetype)initWithName:(NSString*)name
@@ -220,13 +240,21 @@ - (instancetype)initWithName:(NSString*)name
                    taskQueue:(NSObject<FlutterTaskQueue>*)taskQueue {
   self = [super init];
   NSAssert(self, @"Super init cannot be nil");
-  _name = [name copy];
-  _messenger = messenger;
-  _codec = codec;
-  _taskQueue = taskQueue;
+  _name = [name retain];
+  _messenger = [messenger retain];
+  _codec = [codec retain];
+  _taskQueue = [taskQueue retain];
   return self;
 }
 
+- (void)dealloc {
+  [_name release];
+  [_messenger release];
+  [_codec release];
+  [_taskQueue release];
+  [super dealloc];
+}
+
 - (void)invokeMethod:(NSString*)method arguments:(id)arguments {
   FlutterMethodCall* methodCall = [FlutterMethodCall methodCallWithMethodName:method
                                                                     arguments:arguments];
@@ -257,7 +285,7 @@ - (void)setMethodCallHandler:(FlutterMethodCallHandler)handler {
     return;
   }
   // Make sure the block captures the codec, not self.
-  __weak NSObject<FlutterMethodCodec>* codec = _codec;
+  NSObject<FlutterMethodCodec>* codec = _codec;
   FlutterBinaryMessageHandler messageHandler = ^(NSData* message, FlutterBinaryReply callback) {
     FlutterMethodCall* call = [codec decodeMethodCall:message];
     handler(call, ^(id result) {
@@ -300,7 +328,8 @@ + (instancetype)eventChannelWithName:(NSString*)name
 + (instancetype)eventChannelWithName:(NSString*)name
                      binaryMessenger:(NSObject<FlutterBinaryMessenger>*)messenger
                                codec:(NSObject<FlutterMethodCodec>*)codec {
-  return [[FlutterEventChannel alloc] initWithName:name binaryMessenger:messenger codec:codec];
+  return [[[FlutterEventChannel alloc] initWithName:name binaryMessenger:messenger
+                                              codec:codec] autorelease];
 }
 
 - (instancetype)initWithName:(NSString*)name
@@ -315,13 +344,21 @@ - (instancetype)initWithName:(NSString*)name
                    taskQueue:(NSObject<FlutterTaskQueue>* _Nullable)taskQueue {
   self = [super init];
   NSAssert(self, @"Super init cannot be nil");
-  _name = [name copy];
-  _messenger = messenger;
-  _codec = codec;
-  _taskQueue = taskQueue;
+  _name = [name retain];
+  _messenger = [messenger retain];
+  _codec = [codec retain];
+  _taskQueue = [taskQueue retain];
   return self;
 }
 
+- (void)dealloc {
+  [_name release];
+  [_codec release];
+  [_messenger release];
+  [_taskQueue release];
+  [super dealloc];
+}
+
 static FlutterBinaryMessengerConnection SetStreamHandlerMessageHandlerOnChannel(
     NSObject<FlutterStreamHandler>* handler,
     NSString* name,

shell/platform/darwin/common/framework/Source/FlutterCodecs.mm

@@ -6,8 +6,6 @@
 
 #include <cstring>
 
-FLUTTER_ASSERT_ARC
-
 @implementation FlutterBinaryCodec
 + (instancetype)sharedInstance {
   static id _sharedInstance = nil;
@@ -50,7 +48,7 @@ - (NSString*)decode:(NSData*)message {
   if (message == nil) {
     return nil;
   }
-  return [[NSString alloc] initWithData:message encoding:NSUTF8StringEncoding];
+  return [[[NSString alloc] initWithData:message encoding:NSUTF8StringEncoding] autorelease];
 }
 @end