[webview_flutter] Apply texture api change

Signed-off-by: MuHong Byun <[email protected]>

Author: bwikbs

packages/webview_flutter/tizen/src/webview.cc

@@ -7,10 +7,12 @@
 #include <Ecore_IMF_Evas.h>
 #include <Ecore_Input_Evas.h>
 #include <flutter_platform_view.h>
-#include <flutter_tizen_texture_registrar.h>
+#include <flutter_texture_registrar.h>
+#include <texture_registrar.h>
 
 #include <map>
 #include <memory>
+#include <mutex>
 #include <sstream>
 #include <string>
 
@@ -154,7 +156,7 @@ double ExtractDoubleFromMap(const flutter::EncodableValue& arguments,
 }
 
 WebView::WebView(flutter::PluginRegistrar* registrar, int viewId,
-                 FlutterTextureRegistrar* texture_registrar, double width,
+                 flutter::TextureRegistrar* texture_registrar, double width,
                  double height, flutter::EncodableMap& params)
     : PlatformView(registrar, viewId),
       texture_registrar_(texture_registrar),
@@ -166,7 +168,14 @@ WebView::WebView(flutter::PluginRegistrar* registrar, int viewId,
       has_navigation_delegate_(false),
       has_progress_tracking_(false),
       context_(nullptr) {
-  SetTextureId(FlutterRegisterExternalTexture(texture_registrar_));
+  flutter::TextureVariant* textureVariant_ =
+      new flutter::TextureVariant(flutter::GpuBufferTexture(
+          [this](size_t width,
+                 size_t height) -> const FlutterDesktopGpuBuffer* {
+            return this->CopyGpuBuffer(width, height);
+          },
+          [this]() -> void { this->Destruction(); }));
+  SetTextureId(texture_registrar_->RegisterTexture(textureVariant_));
   InitWebView();
 
   channel_ = std::make_unique<flutter::MethodChannel<flutter::EncodableValue>>(
@@ -375,7 +384,8 @@ std::string WebView::GetChannelName() {
 }
 
 void WebView::Dispose() {
-  FlutterUnregisterExternalTexture(texture_registrar_, GetTextureId());
+  // FlutterUnregisterExternalTexture(texture_registrar_, GetTextureId());
+  texture_registrar_->UnregisterTexture(GetTextureId());
 
   if (webview_instance_) {
     webview_instance_->Destroy();
@@ -754,19 +764,17 @@ void WebView::InitWebView() {
       "Asia/Seoul",
       [this]() -> LWE::WebContainer::ExternalImageInfo {
         LWE::WebContainer::ExternalImageInfo result;
-        if (!tbm_surface_) {
-          tbm_surface_ =
+        if (candidate_surfaces_.size() < 2) {
+          tbm_surface_h tbm_surface =
               tbm_surface_create(width_, height_, TBM_FORMAT_ARGB8888);
+          candidate_surfaces_.push(tbm_surface);
         }
-        result.imageAddress = (void*)tbm_surface_;
+        result.imageAddress = (void*)candidate_surfaces_.top();
         return result;
       },
       [this](LWE::WebContainer* c, bool isRendered) {
         if (isRendered) {
-          FlutterMarkExternalTextureFrameAvailable(
-              texture_registrar_, GetTextureId(), tbm_surface_);
-          tbm_surface_destroy(tbm_surface_);
-          tbm_surface_ = nullptr;
+          texture_registrar_->MarkTextureFrameAvailable(GetTextureId());
         }
       });
 #ifndef TV_PROFILE
@@ -834,6 +842,7 @@ void WebView::HandleMethodCall(
           });
     } else {
       result->Error("Invalid Arguments", "Invalid Arguments");
+      //  result->Success();
     }
   } else if (method_name.compare("addJavascriptChannels") == 0) {
     if (std::holds_alternative<flutter::EncodableList>(arguments)) {
@@ -900,3 +909,26 @@ void WebView::HandleCookieMethodCall(
     result->NotImplemented();
   }
 }
+
+FlutterDesktopGpuBuffer* WebView::CopyGpuBuffer(size_t width, size_t height) {
+  if (candidate_surfaces_.size() == 0) {
+    return nullptr;
+  }
+  std::lock_guard<std::mutex> lock(mutex_);
+  tbm_surface_ = candidate_surfaces_.top();
+  candidate_surfaces_.pop();
+
+  FlutterDesktopGpuBuffer* gpuBuffer = new FlutterDesktopGpuBuffer();
+  gpuBuffer->buffer = tbm_surface_;
+  gpuBuffer->width = width;
+  gpuBuffer->height = height;
+  return gpuBuffer;
+}
+
+void WebView::Destruction() {
+  if (tbm_surface_) {
+    std::lock_guard<std::mutex> lock(mutex_);
+    tbm_surface_destroy(tbm_surface_);
+    tbm_surface_ = nullptr;
+  }
+}

packages/webview_flutter/tizen/src/webview.h

@@ -10,9 +10,11 @@
 #include <flutter/standard_message_codec.h>
 #include <flutter/standard_method_codec.h>
 #include <flutter_platform_view.h>
-#include <flutter_tizen_texture_registrar.h>
 #include <tbm_surface.h>
 
+#include <mutex>
+#include <stack>
+
 namespace LWE {
 class WebContainer;
 }
@@ -22,7 +24,7 @@ class TextInputChannel;
 class WebView : public PlatformView {
  public:
   WebView(flutter::PluginRegistrar* registrar, int viewId,
-          FlutterTextureRegistrar* textureRegistrar, double width,
+          flutter::TextureRegistrar* textureRegistrar, double width,
           double height, flutter::EncodableMap& params);
   ~WebView();
   virtual void Dispose() override;
@@ -46,6 +48,9 @@ class WebView : public PlatformView {
   void HidePanel();
   void ShowPanel();
 
+  FlutterDesktopGpuBuffer* CopyGpuBuffer(size_t width, size_t height);
+  void Destruction();
+
  private:
   void HandleMethodCall(
       const flutter::MethodCall<flutter::EncodableValue>& method_call,
@@ -59,16 +64,18 @@ class WebView : public PlatformView {
   void RegisterJavaScriptChannelName(const std::string& name);
   void ApplySettings(flutter::EncodableMap);
 
-  FlutterTextureRegistrar* texture_registrar_;
+  flutter::TextureRegistrar* texture_registrar_;
   LWE::WebContainer* webview_instance_;
   double width_;
   double height_;
+  std::stack<tbm_surface_h> candidate_surfaces_;
   tbm_surface_h tbm_surface_;
   bool is_mouse_lbutton_down_;
   bool has_navigation_delegate_;
   bool has_progress_tracking_;
   std::unique_ptr<flutter::MethodChannel<flutter::EncodableValue>> channel_;
   Ecore_IMF_Context* context_;
+  std::mutex mutex_;
 };
 
 #endif  // FLUTTER_PLUGIN_WEBVIEW_FLUTTER_TIZEN_WEVIEW_H_

packages/webview_flutter/tizen/src/webview_factory.cc

@@ -22,7 +22,7 @@
 #include "webview_flutter_tizen_plugin.h"
 
 WebViewFactory::WebViewFactory(flutter::PluginRegistrar* registrar,
-                               FlutterTextureRegistrar* textureRegistrar)
+                               flutter::TextureRegistrar* textureRegistrar)
     : PlatformViewFactory(registrar), textureRegistrar_(textureRegistrar) {
   char* path = app_get_data_path();
   if (!path || strlen(path) == 0) {

packages/webview_flutter/tizen/src/webview_factory.h

@@ -9,14 +9,14 @@
 class WebViewFactory : public PlatformViewFactory {
  public:
   WebViewFactory(flutter::PluginRegistrar* registrar,
-                 FlutterTextureRegistrar* textureRegistrar);
+                 flutter::TextureRegistrar* textureRegistrar);
   virtual void Dispose() override;
   virtual PlatformView* Create(
       int viewId, double width, double height,
       const std::vector<uint8_t>& createParams) override;
 
  private:
-  FlutterTextureRegistrar* textureRegistrar_;
+  flutter::TextureRegistrar* textureRegistrar_;
 };
 
 #endif  // FLUTTER_PLUGIN_WEBVIEW_FLUTTER_TIZEN_WEVIEW_FACTORY_H_

packages/webview_flutter/tizen/src/webview_flutter_tizen_plugin.cc

@@ -38,7 +38,7 @@ void WebviewFlutterTizenPluginRegisterWithRegistrar(
       flutter::PluginRegistrarManager::GetInstance()
           ->GetRegistrar<flutter::PluginRegistrar>(registrar);
   auto factory = std::make_unique<WebViewFactory>(
-      core_registrar, FlutterPluginRegistrarGetTexture(registrar));
+      core_registrar, core_registrar->texture_registrar());
   FlutterRegisterViewFactory(registrar, kViewType, std::move(factory));
   WebviewFlutterTizenPlugin::RegisterWithRegistrar(core_registrar);
 }