Make FLEViewController's view an internal detail (flutter#9741)

Rather than clients needing to create an appropriate view in a XIB and
attach it to an FLEViewController, which is error-prone, have
FLEViewController create its own view programatically. The view is now
an internal detail, so calling setView: on an FLEViewController will no
longer work.

As a result of the view being internal, the public API surface is
simplified.

This is a breaking change for macOS Runners.

Author: stuartmorgan-g

ci/licenses_golden/licenses_flutter

@@ -750,9 +750,6 @@ FILE: ../../../flutter/shell/platform/darwin/ios/platform_view_ios.mm
 FILE: ../../../flutter/shell/platform/darwin/macos/framework/FlutterMacOS.podspec
 FILE: ../../../flutter/shell/platform/darwin/macos/framework/Headers/FLEDartProject.h
 FILE: ../../../flutter/shell/platform/darwin/macos/framework/Headers/FLEEngine.h
-FILE: ../../../flutter/shell/platform/darwin/macos/framework/Headers/FLEOpenGLContextHandling.h
-FILE: ../../../flutter/shell/platform/darwin/macos/framework/Headers/FLEReshapeListener.h
-FILE: ../../../flutter/shell/platform/darwin/macos/framework/Headers/FLEView.h
 FILE: ../../../flutter/shell/platform/darwin/macos/framework/Headers/FLEViewController.h
 FILE: ../../../flutter/shell/platform/darwin/macos/framework/Headers/FlutterMacOS.h
 FILE: ../../../flutter/shell/platform/darwin/macos/framework/Headers/FlutterPluginMacOS.h
@@ -766,9 +763,10 @@ FILE: ../../../flutter/shell/platform/darwin/macos/framework/Source/FLETextInput
 FILE: ../../../flutter/shell/platform/darwin/macos/framework/Source/FLETextInputModel.mm
 FILE: ../../../flutter/shell/platform/darwin/macos/framework/Source/FLETextInputPlugin.h
 FILE: ../../../flutter/shell/platform/darwin/macos/framework/Source/FLETextInputPlugin.mm
-FILE: ../../../flutter/shell/platform/darwin/macos/framework/Source/FLEView.mm
 FILE: ../../../flutter/shell/platform/darwin/macos/framework/Source/FLEViewController.mm
 FILE: ../../../flutter/shell/platform/darwin/macos/framework/Source/FLEViewController_Internal.h
+FILE: ../../../flutter/shell/platform/darwin/macos/framework/Source/FlutterView.h
+FILE: ../../../flutter/shell/platform/darwin/macos/framework/Source/FlutterView.mm
 FILE: ../../../flutter/shell/platform/darwin/macos/framework/module.modulemap
 FILE: ../../../flutter/shell/platform/embedder/assets/EmbedderInfo.plist
 FILE: ../../../flutter/shell/platform/embedder/assets/embedder.modulemap

shell/platform/darwin/macos/BUILD.gn

@@ -35,9 +35,6 @@ _flutter_framework_headers = [
   "framework/Headers/FlutterPluginRegistrarMacOS.h",
   "framework/Headers/FLEDartProject.h",
   "framework/Headers/FLEEngine.h",
-  "framework/Headers/FLEOpenGLContextHandling.h",
-  "framework/Headers/FLEReshapeListener.h",
-  "framework/Headers/FLEView.h",
   "framework/Headers/FLEViewController.h",
 ]
 
@@ -58,9 +55,10 @@ shared_library("create_flutter_framework_dylib") {
     "framework/Source/FLETextInputModel.mm",
     "framework/Source/FLETextInputPlugin.h",
     "framework/Source/FLETextInputPlugin.mm",
-    "framework/Source/FLEView.mm",
     "framework/Source/FLEViewController.mm",
     "framework/Source/FLEViewController_Internal.h",
+    "framework/Source/FlutterView.h",
+    "framework/Source/FlutterView.mm",
   ]
 
   sources += _flutter_framework_headers

shell/platform/darwin/macos/framework/Headers/FLEOpenGLContextHandling.h

@@ -5,8 +5,6 @@
 #import <Cocoa/Cocoa.h>
 
 #import "FLEEngine.h"
-#import "FLEOpenGLContextHandling.h"
-#import "FLEReshapeListener.h"
 #import "FlutterMacros.h"
 #import "FlutterPluginRegistrarMacOS.h"
 
@@ -29,13 +27,7 @@ typedef NS_ENUM(NSInteger, FlutterMouseTrackingMode) {
  * Flutter engine in non-interactive mode, or with a drawable Flutter canvas.
  */
 FLUTTER_EXPORT
-@interface FLEViewController : NSViewController <FlutterPluginRegistry, FLEReshapeListener>
-
-/**
- * The view this controller manages. Must be capable of handling text input events, and the OpenGL
- * context handling protocols.
- */
-@property(nullable) NSView<FLEOpenGLContextHandling>* view;
+@interface FLEViewController : NSViewController <FlutterPluginRegistry>
 
 /**
  * The Flutter engine associated with this view controller.

shell/platform/darwin/macos/framework/Headers/FLEReshapeListener.h

@@ -4,9 +4,6 @@
 
 #import "FLEDartProject.h"
 #import "FLEEngine.h"
-#import "FLEOpenGLContextHandling.h"
-#import "FLEReshapeListener.h"
-#import "FLEView.h"
 #import "FLEViewController.h"
 #import "FlutterBinaryMessenger.h"
 #import "FlutterChannels.h"

shell/platform/darwin/macos/framework/Headers/FLEView.h

@@ -210,31 +210,31 @@ - (void)sendPointerEvent:(const FlutterPointerEvent&)event {
 #pragma mark - Private methods
 
 - (bool)engineCallbackOnMakeCurrent {
-  if (!_viewController.view) {
+  if (!_viewController.flutterView) {
     return false;
   }
-  [_viewController.view makeCurrentContext];
+  [_viewController.flutterView makeCurrentContext];
   return true;
 }
 
 - (bool)engineCallbackOnClearCurrent {
-  if (!_viewController.view) {
+  if (!_viewController.flutterView) {
     return false;
   }
   [NSOpenGLContext clearCurrentContext];
   return true;
 }
 
 - (bool)engineCallbackOnPresent {
-  if (!_viewController.view) {
+  if (!_viewController.flutterView) {
     return false;
   }
-  [_viewController.view onPresent];
+  [_viewController.flutterView onPresent];
   return true;
 }
 
 - (bool)engineCallbackOnMakeResourceCurrent {
-  if (!_viewController.view) {
+  if (!_viewController.flutterView) {
     return false;
   }
   [_viewController makeResourceContextCurrent];

shell/platform/darwin/macos/framework/Headers/FLEViewController.h

@@ -8,10 +8,9 @@
 #import "flutter/shell/platform/darwin/common/framework/Headers/FlutterChannels.h"
 #import "flutter/shell/platform/darwin/common/framework/Headers/FlutterCodecs.h"
 #import "flutter/shell/platform/darwin/macos/framework/Headers/FLEEngine.h"
-#import "flutter/shell/platform/darwin/macos/framework/Headers/FLEReshapeListener.h"
-#import "flutter/shell/platform/darwin/macos/framework/Headers/FLEView.h"
 #import "flutter/shell/platform/darwin/macos/framework/Source/FLEEngine_Internal.h"
 #import "flutter/shell/platform/darwin/macos/framework/Source/FLETextInputPlugin.h"
+#import "flutter/shell/platform/darwin/macos/framework/Source/FlutterView.h"
 #import "flutter/shell/platform/embedder/embedder.h"
 
 namespace {
@@ -68,7 +67,7 @@ void Reset() {
 /**
  * Private interface declaration for FLEViewController.
  */
-@interface FLEViewController ()
+@interface FLEViewController () <FlutterViewReshapeListener>
 
 /**
  * A list of additional responders to keyboard events. Keybord events are forwarded to all of them.
@@ -195,16 +194,13 @@ - (instancetype)initWithNibName:(NSString*)nibNameOrNil bundle:(NSBundle*)nibBun
   return self;
 }
 
-- (void)setView:(NSView*)view {
-  if (_trackingArea) {
-    [self.view removeTrackingArea:_trackingArea];
-  }
-  [super setView:view];
-  [self configureTrackingArea];
+- (void)loadView {
+  FlutterView* flutterView = [[FlutterView alloc] initWithReshapeListener:self];
+  self.view = flutterView;
 }
 
-- (void)loadView {
-  self.view = [[FLEView alloc] init];
+- (void)viewDidLoad {
+  [self configureTrackingArea];
 }
 
 #pragma mark - Public methods
@@ -238,6 +234,10 @@ - (BOOL)launchEngineWithProject:(nullable FLEDartProject*)project {
 
 #pragma mark - Framework-internal methods
 
+- (FlutterView*)flutterView {
+  return static_cast<FlutterView*>(self.view);
+}
+
 - (void)addKeyResponder:(NSResponder*)responder {
   [self.additionalKeyResponders addObject:responder];
 }
@@ -449,12 +449,12 @@ - (void)setClipboardData:(NSDictionary*)data {
   }
 }
 
-#pragma mark - FLEReshapeListener
+#pragma mark - FlutterViewReshapeListener
 
 /**
  * Responds to view reshape by notifying the engine of the change in dimensions.
  */
-- (void)viewDidReshape:(NSOpenGLView*)view {
+- (void)viewDidReshape:(NSView*)view {
   CGSize scaledSize = [view convertRectToBacking:view.bounds].size;
   double pixelRatio = view.bounds.size.width == 0 ? 1 : scaledSize.width / view.bounds.size.width;
   [_engine updateWindowMetricsWithSize:scaledSize pixelRatio:pixelRatio];