From 4ff8dfc9e5d92bcb190efaee771cb6c338029e56 Mon Sep 17 00:00:00 2001
From: loongliu <liujilong.me@gmail.com>
Date: Fri, 12 Jul 2019 11:35:09 +0800
Subject: [PATCH] Reuse texture cache in ios_external_texture_gl.

In current implementation, external texture data flow is a producer-consumer model. When painting external texture, it always asks registered external texture object to produce new CVPixelBuffer, then transforms it to texture. `MarkNewFrameAvailable` function is ignored. This commit changes the dataflow. Now ios_external_texture_gl caches previous opengl texture, if no new frame are available, it do not `copyPixelBuffer` method, just uses cached opengl texture to draw.
---
 shell/platform/darwin/ios/ios_external_texture_gl.h |  1 +
 .../platform/darwin/ios/ios_external_texture_gl.mm  | 13 +++++++++++--
 2 files changed, 12 insertions(+), 2 deletions(-)

diff --git a/shell/platform/darwin/ios/ios_external_texture_gl.h b/shell/platform/darwin/ios/ios_external_texture_gl.h
index 923f7410632e8..e74caad2d9c66 100644
--- a/shell/platform/darwin/ios/ios_external_texture_gl.h
+++ b/shell/platform/darwin/ios/ios_external_texture_gl.h
@@ -31,6 +31,7 @@ class IOSExternalTextureGL : public flutter::Texture {
 
   void EnsureTextureCacheExists();
 
+  bool new_frame_ready_ = false;
   NSObject<FlutterTexture>* external_texture_;
   fml::CFRef<CVOpenGLESTextureCacheRef> cache_ref_;
   fml::CFRef<CVOpenGLESTextureRef> texture_ref_;
diff --git a/shell/platform/darwin/ios/ios_external_texture_gl.mm b/shell/platform/darwin/ios/ios_external_texture_gl.mm
index e7e15d5719196..a5e729195ce34 100644
--- a/shell/platform/darwin/ios/ios_external_texture_gl.mm
+++ b/shell/platform/darwin/ios/ios_external_texture_gl.mm
@@ -53,17 +53,24 @@
   }
 }
 
+bool IOSExternalTextureGL::NeedUpdateTexture(bool freeze) {
+  // Update texture if `texture_ref_` is reset to `nullptr` when GrContext
+  // is destroied or new frame is ready.
+  return (!freeze && new_frame_ready_) || !texture_ref_;
+}
+
 void IOSExternalTextureGL::Paint(SkCanvas& canvas,
                                  const SkRect& bounds,
                                  bool freeze,
                                  GrContext* context) {
   EnsureTextureCacheExists();
-  if (!freeze) {
+  if (NeedUpdateTexture(freeze)) {
     auto pixelBuffer = [external_texture_ copyPixelBuffer];
     if (pixelBuffer) {
       buffer_ref_.Reset(pixelBuffer);
     }
     CreateTextureFromPixelBuffer();
+    new_frame_ready_ = false;
   }
   if (!texture_ref_) {
     return;
@@ -93,6 +100,8 @@
   cache_ref_.Reset(nullptr);
 }
 
-void IOSExternalTextureGL::MarkNewFrameAvailable() {}
+void IOSExternalTextureGL::MarkNewFrameAvailable() {
+  new_frame_ready_ = true;
+}
 
 }  // namespace flutter