small changes

Author: knopp

shell/platform/darwin/macos/framework/Source/FlutterSurfaceManager.h

@@ -67,8 +67,8 @@
  * Must be called on raster thread. This will schedule a commit on platform
  * thread and block raster thread until the commit is done.
  * `notify` block will be invoked on platform thread and can be used to perform
- * additional work after commit is done, such as mutating platform views.
- * It is guaranteed be called in same CATransaction.
+ * additional work, such as mutating platform views. It is guaranteed be called in
+ * same CATransaction.
  */
 - (void)present:(nonnull NSArray<FlutterSurfacePresentInfo*>*)surfaces
          notify:(nullable dispatch_block_t)notify;

shell/platform/darwin/macos/framework/Source/FlutterSurfaceManager.mm

@@ -26,10 +26,11 @@ - (instancetype)init {
   return self;
 }
 
-- (nullable FlutterSurface*)getSurfaceForSize:(CGSize)size {
+- (nullable FlutterSurface*)removeSurfaceForSize:(CGSize)size {
   @synchronized(self) {
     for (FlutterSurface* surface in _surfaces) {
       if (CGSizeEqualToSize(surface.size, size)) {
+        // By default ARC doesn't retain enumartion iteration variables.
         FlutterSurface* res = surface;
         [_surfaces removeObject:surface];
         return res;
@@ -138,7 +139,7 @@ - (FlutterSurface*)lookupSurface:(nonnull const FlutterMetalTexture*)texture {
 }
 
 - (FlutterSurface*)backBufferForSize:(CGSize)size {
-  FlutterSurface* res = [_backBufferCache getSurfaceForSize:size];
+  FlutterSurface* res = [_backBufferCache removeSurfaceForSize:size];
   if (res == nil) {
     res = [[FlutterSurface alloc] initWithSize:size device:_device];
   }

shell/platform/darwin/macos/framework/Source/FlutterSurfaceManager_Internal.h

@@ -14,7 +14,7 @@
 /**
  * Removes surface with given size from cache (if available) and returns it.
  */
-- (nullable FlutterSurface*)getSurfaceForSize:(CGSize)size;
+- (nullable FlutterSurface*)removeSurfaceForSize:(CGSize)size;
 
 /**
  * Removes all cached surface replacing them with new ones.