Revert "[macOS] Move the glContext generation to FlutterOpenGLRenderer (flutter#22572)"

This reverts commit be5cf15.

Author: iskakaushik

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

@@ -23,11 +23,6 @@ NS_ASSUME_NONNULL_BEGIN
  */
 @property(nonatomic, readonly, nullable) NSOpenGLContext* resourceContext;
 
-/**
- * The main OpenGL which will be used for rendering contents to the FlutterView.
- */
-@property(readwrite, nonatomic, nonnull) NSOpenGLContext* openGLContext;
-
 /**
  * Intializes the renderer with the given FlutterEngine.
  */

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

@@ -72,6 +72,7 @@ - (instancetype)initWithFlutterEngine:(FLUTTER_API_SYMBOL(FlutterEngine))engine
 
 - (void)attachToFlutterView:(FlutterView*)view {
   _flutterView = view;
+  _openGLContext = view.openGLContext;
 }
 
 - (bool)makeCurrent {
@@ -111,15 +112,6 @@ - (NSOpenGLContext*)resourceContext {
   return _resourceContext;
 }
 
-- (NSOpenGLContext*)openGLContext {
-  if (!_openGLContext) {
-    NSOpenGLContext* shareContext = [self resourceContext];
-    _openGLContext = [[NSOpenGLContext alloc] initWithFormat:shareContext.pixelFormat
-                                                shareContext:shareContext];
-  }
-  return _openGLContext;
-}
-
 - (bool)makeResourceCurrent {
   [self.resourceContext makeCurrentContext];
   return true;

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

@@ -20,14 +20,19 @@
  */
 @interface FlutterView : NSView
 
+/**
+ * The OpenGL context of backing surface.
+ */
+@property(readwrite, nonatomic, nonnull) NSOpenGLContext* openGLContext;
+
 - (nullable instancetype)initWithFrame:(NSRect)frame
-                           mainContext:(nonnull NSOpenGLContext*)mainContext
+                          shareContext:(nonnull NSOpenGLContext*)shareContext
                        reshapeListener:(nonnull id<FlutterViewReshapeListener>)reshapeListener
     NS_DESIGNATED_INITIALIZER;
 
-- (nullable instancetype)initWithMainContext:(nonnull NSOpenGLContext*)mainContext
-                             reshapeListener:
-                                 (nonnull id<FlutterViewReshapeListener>)reshapeListener;
+- (nullable instancetype)initWithShareContext:(nonnull NSOpenGLContext*)shareContext
+                              reshapeListener:
+                                  (nonnull id<FlutterViewReshapeListener>)reshapeListener;
 
 - (nullable instancetype)initWithFrame:(NSRect)frameRect
                            pixelFormat:(nullable NSOpenGLPixelFormat*)format NS_UNAVAILABLE;

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

@@ -15,37 +15,38 @@ @interface FlutterView () <FlutterResizeSynchronizerDelegate> {
   __weak id<FlutterViewReshapeListener> _reshapeListener;
   FlutterResizeSynchronizer* _resizeSynchronizer;
   FlutterSurfaceManager* _surfaceManager;
-  NSOpenGLContext* _openGLContext;
 }
 
 @end
 
 @implementation FlutterView
 
-- (instancetype)initWithMainContext:(NSOpenGLContext*)mainContext
-                    reshapeListener:(id<FlutterViewReshapeListener>)reshapeListener {
-  return [self initWithFrame:NSZeroRect mainContext:mainContext reshapeListener:reshapeListener];
+- (instancetype)initWithShareContext:(NSOpenGLContext*)shareContext
+                     reshapeListener:(id<FlutterViewReshapeListener>)reshapeListener {
+  return [self initWithFrame:NSZeroRect shareContext:shareContext reshapeListener:reshapeListener];
 }
 
 - (instancetype)initWithFrame:(NSRect)frame
-                  mainContext:(NSOpenGLContext*)mainContext
+                 shareContext:(NSOpenGLContext*)shareContext
               reshapeListener:(id<FlutterViewReshapeListener>)reshapeListener {
   self = [super initWithFrame:frame];
   if (self) {
-    _openGLContext = mainContext;
+    self.openGLContext = [[NSOpenGLContext alloc] initWithFormat:shareContext.pixelFormat
+                                                    shareContext:shareContext];
+
     [self setWantsLayer:YES];
 
     _resizeSynchronizer = [[FlutterResizeSynchronizer alloc] initWithDelegate:self];
     _surfaceManager = [[FlutterSurfaceManager alloc] initWithLayer:self.layer
-                                                     openGLContext:_openGLContext];
+                                                     openGLContext:self.openGLContext];
 
     _reshapeListener = reshapeListener;
   }
   return self;
 }
 
 - (void)resizeSynchronizerFlush:(FlutterResizeSynchronizer*)synchronizer {
-  MacOSGLContextSwitch context_switch(_openGLContext);
+  MacOSGLContextSwitch context_switch(self.openGLContext);
   glFlush();
 }
 

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

@@ -239,13 +239,13 @@ - (instancetype)initWithProject:(nullable FlutterDartProject*)project {
 }
 
 - (void)loadView {
-  NSOpenGLContext* mainContext = _engine.openGLRenderer.openGLContext;
-  if (!mainContext) {
-    NSLog(@"Unable to create FlutterView; no GL context available.");
+  NSOpenGLContext* resourceContext = _engine.openGLRenderer.resourceContext;
+  if (!resourceContext) {
+    NSLog(@"Unable to create FlutterView; no resource context available.");
     return;
   }
-  FlutterView* flutterView = [[FlutterView alloc] initWithMainContext:mainContext
-                                                      reshapeListener:self];
+  FlutterView* flutterView = [[FlutterView alloc] initWithShareContext:resourceContext
+                                                       reshapeListener:self];
   self.view = flutterView;
 }