[video_player] Adds platform view support on macOS (flutter#9576)

Adds macOS support for the platform view display option, brining macOS in line with the iOS implementation.

This:
- extracts the macOS additions from flutter#8558 as cherry picks,
- updates it for some changes that happened during iOS review, and 
- makes some minor adjustments (mostly combining the factory classes into one shared file)

Part of flutter/flutter#86613

## Pre-Review Checklist

[^1]: Regular contributors who have demonstrated familiarity with the repository guidelines only need to comment if the PR is not auto-exempted by repo tooling.

Author: stuartmorgan-g

packages/video_player/video_player_avfoundation/CHANGELOG.md

@@ -1,3 +1,7 @@
+## 2.8.0
+
+* Adds platform view support for macOS.
+
 ## 2.7.3
 
 * Restructures the communication between Dart and native code.

packages/video_player/video_player_avfoundation/README.md

@@ -13,7 +13,3 @@ should add it to your `pubspec.yaml` as usual.
 
 [1]: https://pub.dev/packages/video_player
 [2]: https://flutter.dev/to/endorsed-federated-plugin
-
-## Platform limitations
-
-On macOS, the plugin does not currently support platform views. Instead, a texture view is always used to display the video player, even if `VideoViewType.platformView` is specified as a parameter.

packages/video_player/video_player_avfoundation/darwin/RunnerTests/VideoPlayerTests.m

@@ -8,13 +8,10 @@
 
 #import <OCMock/OCMock.h>
 #import <video_player_avfoundation/AVAssetTrackUtils.h>
+#import <video_player_avfoundation/FVPNativeVideoViewFactory.h>
 #import <video_player_avfoundation/FVPTextureBasedVideoPlayer_Test.h>
 #import <video_player_avfoundation/FVPVideoPlayerPlugin_Test.h>
 
-#if TARGET_OS_IOS
-#import <video_player_avfoundation/FVPNativeVideoViewFactory.h>
-#endif
-
 #if TARGET_OS_IOS
 @interface FakeAVAssetTrack : AVAssetTrack
 @property(readonly, nonatomic) CGAffineTransform preferredTransform;
@@ -801,7 +798,6 @@ - (void)testHotReloadDoesNotCrash {
                                handler:nil];  // No assertions needed. Lack of crash is a success.
 }
 
-#if TARGET_OS_IOS
 - (void)testNativeVideoViewFactoryRegistration {
   NSObject<FlutterPluginRegistrar> *registrar = OCMProtocolMock(@protocol(FlutterPluginRegistrar));
 
@@ -811,7 +807,6 @@ - (void)testNativeVideoViewFactoryRegistration {
 
   OCMVerifyAll(registrar);
 }
-#endif
 
 - (void)testPublishesInRegistration {
   NSObject<FlutterPluginRegistrar> *registrar = OCMProtocolMock(@protocol(FlutterPluginRegistrar));

packages/video_player/video_player_avfoundation/darwin/video_player_avfoundation/Sources/video_player_avfoundation_ios/FVPNativeVideoViewFactory.m renamed to packages/video_player/video_player_avfoundation/darwin/video_player_avfoundation/Sources/video_player_avfoundation/FVPNativeVideoViewFactory.m

@@ -25,9 +25,16 @@ - (instancetype)initWithMessenger:(NSObject<FlutterBinaryMessenger> *)messenger
   return self;
 }
 
+#pragma mark - FlutterPlatformViewFactory
+
+#if TARGET_OS_OSX
+- (NSView *)createWithViewIdentifier:(int64_t)viewIdentifier
+                           arguments:(FVPPlatformVideoViewCreationParams *)args {
+#else
 - (NSObject<FlutterPlatformView> *)createWithFrame:(CGRect)frame
                                     viewIdentifier:(int64_t)viewIdentifier
                                          arguments:(FVPPlatformVideoViewCreationParams *)args {
+#endif
   NSNumber *playerIdentifier = @(args.playerId);
   FVPVideoPlayer *player = self.playerByIdProvider(playerIdentifier);
   return [[FVPNativeVideoView alloc] initWithPlayer:player.player];

packages/video_player/video_player_avfoundation/darwin/video_player_avfoundation/Sources/video_player_avfoundation/FVPVideoPlayerPlugin.m

@@ -10,17 +10,13 @@
 #import "./include/video_player_avfoundation/FVPAVFactory.h"
 #import "./include/video_player_avfoundation/FVPDisplayLink.h"
 #import "./include/video_player_avfoundation/FVPFrameUpdater.h"
+#import "./include/video_player_avfoundation/FVPNativeVideoViewFactory.h"
 #import "./include/video_player_avfoundation/FVPTextureBasedVideoPlayer.h"
 #import "./include/video_player_avfoundation/FVPVideoPlayer.h"
 // Relative path is needed for messages.g.h. See
 // https://github.com/flutter/packages/pull/6675/#discussion_r1591210702
 #import "./include/video_player_avfoundation/messages.g.h"
 
-#if TARGET_OS_IOS
-// Platform views are only supported on iOS as of now.
-#import "./include/video_player_avfoundation/FVPNativeVideoViewFactory.h"
-#endif
-
 #if !__has_feature(objc_arc)
 #error Code Requires ARC.
 #endif
@@ -61,15 +57,12 @@ @implementation FVPVideoPlayerPlugin
 + (void)registerWithRegistrar:(NSObject<FlutterPluginRegistrar> *)registrar {
   FVPVideoPlayerPlugin *instance = [[FVPVideoPlayerPlugin alloc] initWithRegistrar:registrar];
   [registrar publish:instance];
-#if TARGET_OS_IOS
-  // Platform views are only supported on iOS as of now.
   FVPNativeVideoViewFactory *factory = [[FVPNativeVideoViewFactory alloc]
                initWithMessenger:registrar.messenger
       playerByIdentifierProvider:^FVPVideoPlayer *(NSNumber *playerIdentifier) {
         return instance->_playersByIdentifier[playerIdentifier];
       }];
   [registrar registerViewFactory:factory withId:@"plugins.flutter.dev/video_player_ios"];
-#endif
   SetUpFVPAVFoundationVideoPlayerApi(registrar.messenger, instance);
 }
 

packages/video_player/video_player_avfoundation/darwin/video_player_avfoundation/Sources/video_player_avfoundation/include/video_player_avfoundation/FVPNativeVideoView.h

@@ -2,8 +2,6 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-// Platform views are only supported on iOS as of now. Ifdefs are used to avoid compilation errors.
-
 #import <AVFoundation/AVFoundation.h>
 
 #if TARGET_OS_OSX

packages/video_player/video_player_avfoundation/darwin/video_player_avfoundation/Sources/video_player_avfoundation/include/video_player_avfoundation/FVPNativeVideoViewFactory.h

@@ -2,8 +2,6 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-// Platform views are only supported on iOS as of now. Ifdef is used to avoid compilation errors.
-
 #import <Foundation/Foundation.h>
 
 #import "FVPVideoPlayer.h"

packages/video_player/video_player_avfoundation/darwin/video_player_avfoundation/Sources/video_player_avfoundation_macos/FVPNativeVideoView.m

@@ -0,0 +1,25 @@
+// Copyright 2013 The Flutter Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#import "../video_player_avfoundation/include/video_player_avfoundation/FVPNativeVideoView.h"
+
+#import <AVFoundation/AVFoundation.h>
+#import <QuartzCore/QuartzCore.h>
+
+@implementation FVPNativeVideoView
+
+- (instancetype)initWithPlayer:(AVPlayer *)player {
+  self = [super init];
+  if (self) {
+    self.wantsLayer = YES;
+    ((AVPlayerLayer *)self.layer).player = player;
+  }
+  return self;
+}
+
+- (CALayer *)makeBackingLayer {
+  return [[AVPlayerLayer alloc] init];
+}
+
+@end

packages/video_player/video_player_avfoundation/example/lib/main.dart

@@ -4,8 +4,6 @@
 
 // ignore_for_file: public_member_api_docs
 
-import 'dart:io';
-
 import 'package:flutter/material.dart';
 import 'package:video_player_platform_interface/video_player_platform_interface.dart';
 
@@ -44,27 +42,20 @@ class _App extends StatelessWidget {
           ),
         ),
         body: TabBarView(
-          children: Platform.isIOS
-              ? <Widget>[
-                  _ViewTypeTabBar(
-                    builder: (VideoViewType viewType) =>
-                        _BumbleBeeRemoteVideo(viewType),
-                  ),
-                  _ViewTypeTabBar(
-                    builder: (VideoViewType viewType) =>
-                        _BumbleBeeEncryptedLiveStream(viewType),
-                  ),
-                  _ViewTypeTabBar(
-                    builder: (VideoViewType viewType) =>
-                        _ButterFlyAssetVideo(viewType),
-                  ),
-                ]
-              // Platform views are only supported on iOS as of now.
-              : const <Widget>[
-                  _BumbleBeeRemoteVideo(VideoViewType.textureView),
-                  _BumbleBeeEncryptedLiveStream(VideoViewType.textureView),
-                  _ButterFlyAssetVideo(VideoViewType.textureView),
-                ],
+          children: <Widget>[
+            _ViewTypeTabBar(
+              builder: (VideoViewType viewType) =>
+                  _BumbleBeeRemoteVideo(viewType),
+            ),
+            _ViewTypeTabBar(
+              builder: (VideoViewType viewType) =>
+                  _BumbleBeeEncryptedLiveStream(viewType),
+            ),
+            _ViewTypeTabBar(
+              builder: (VideoViewType viewType) =>
+                  _ButterFlyAssetVideo(viewType),
+            ),
+          ],
         ),
       ),
     );

packages/video_player/video_player_avfoundation/lib/src/avfoundation_video_player.dart

@@ -4,7 +4,6 @@
 
 import 'dart:async';
 
-import 'package:flutter/foundation.dart';
 import 'package:flutter/services.dart';
 import 'package:flutter/widgets.dart';
 import 'package:video_player_platform_interface/video_player_platform_interface.dart';
@@ -73,10 +72,7 @@ class AVFoundationVideoPlayer extends VideoPlayerPlatform {
   @override
   Future<int?> createWithOptions(VideoCreationOptions options) async {
     final DataSource dataSource = options.dataSource;
-    // Platform views are not supported on macOS yet. Use texture view instead.
-    final VideoViewType viewType = defaultTargetPlatform == TargetPlatform.macOS
-        ? VideoViewType.textureView
-        : options.viewType;
+    final VideoViewType viewType = options.viewType;
 
     String? asset;
     String? packageName;