[webview_flutter] Update minor things #2

* Update lightweightwebengine binary (aarch64/armel, tizen 5.5)
* Introduce buffer pool

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

Author: bwikbs

packages/webview_flutter/CHANGELOG.md

@@ -14,4 +14,5 @@
 
 ## 0.2.2
 * Update lightweight web engine binary & header file
-* Activate resizing function
+* Activate resizing function
+* Apply embedder's texture APIs change

packages/webview_flutter/tizen/lib/aarch64/liblightweight-web-engine.flutter.so

@@ -6,7 +6,7 @@ type = sharedLib
 profile = common-5.5
 
 # Source files
-USER_SRCS += src/webview_flutter_tizen_plugin.cc  src/webview.cc src/webview_factory.cc
+USER_SRCS += src/webview_flutter_tizen_plugin.cc  src/webview.cc src/webview_factory.cc src/buffer_pool.cc
 
 # User defines
 USER_DEFS =

packages/webview_flutter/tizen/lib/armel/liblightweight-web-engine.flutter.so

@@ -0,0 +1,100 @@
+// Copyright 2021 Samsung Electronics Co., Ltd. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "buffer_pool.h"
+
+#include "log.h"
+
+#define BUFFER_POOL_SIZE 5
+
+BufferUnit::BufferUnit(int index, int width, int height)
+    : isAllocated_(false),
+      index_(index),
+      width_(0),
+      height_(0),
+      tbm_surface_(nullptr) {
+  Reset(width, height);
+}
+
+BufferUnit::~BufferUnit() {
+  if (tbm_surface_) {
+    tbm_surface_destroy(tbm_surface_);
+    tbm_surface_ = nullptr;
+  }
+}
+
+bool BufferUnit::Allocate() {
+  if (!isAllocated_) {
+    isAllocated_ = true;
+    return true;
+  }
+  return false;
+}
+
+int BufferUnit::Index() { return index_; }
+bool BufferUnit::isAllocate() { return isAllocated_ && tbm_surface_; }
+tbm_surface_h BufferUnit::Surface() {
+  if (isAllocate()) {
+    return tbm_surface_;
+  }
+  return nullptr;
+}
+
+void BufferUnit::Release() { isAllocated_ = false; }
+
+void BufferUnit::Reset(int width, int height) {
+  if (width_ == width && height_ == height) {
+    return;
+  }
+  width_ = width;
+  height_ = height;
+  if (tbm_surface_) {
+    tbm_surface_destroy(tbm_surface_);
+    tbm_surface_ = nullptr;
+  }
+  tbm_surface_ = tbm_surface_create(width_, height_, TBM_FORMAT_ARGB8888);
+}
+
+BufferPool::BufferPool(int width, int height) {
+  for (int idx = 0; idx < BUFFER_POOL_SIZE; idx++) {
+    pool_.emplace_back(new BufferUnit(idx, width, height));
+  }
+  Prepare(width, height);
+}
+
+BufferPool::~BufferPool() {}
+
+BufferUnit* BufferPool::Get(tbm_surface_h surface) {
+  for (int idx = 0; idx < BUFFER_POOL_SIZE; idx++) {
+    BufferUnit* buffer = pool_[idx].get();
+    if (buffer->Surface() == surface) {
+      return buffer;
+    }
+  }
+  return nullptr;
+}
+
+BufferUnit* BufferPool::AllocateBuffer() {
+  std::lock_guard<std::mutex> lock(mutex_);
+  for (int idx = 0; idx < BUFFER_POOL_SIZE; idx++) {
+    BufferUnit* buffer = pool_[idx].get();
+    if (buffer->Allocate()) {
+      return buffer;
+    }
+  }
+  return nullptr;
+}
+
+void BufferPool::Release(BufferUnit* unit) {
+  std::lock_guard<std::mutex> lock(mutex_);
+  unit->Release();
+}
+
+void BufferPool::Prepare(int width, int height) {
+  std::lock_guard<std::mutex> lock(mutex_);
+  for (int idx = 0; idx < BUFFER_POOL_SIZE; idx++) {
+    BufferUnit* buffer = pool_[idx].get();
+    buffer->Reset(width, height);
+  }
+}

packages/webview_flutter/tizen/project_def.prop

@@ -0,0 +1,49 @@
+// Copyright 2021 Samsung Electronics Co., Ltd. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef FLUTTER_PLUGIN_WEBVIEW_FLUTTER_TIZEN_WEBVIEW_BUFFER_POOL_H_
+#define FLUTTER_PLUGIN_WEBVIEW_FLUTTER_TIZEN_WEBVIEW_BUFFER_POOL_H_
+
+#include <tbm_surface.h>
+
+#include <memory>
+#include <mutex>
+#include <string>
+#include <vector>
+
+class BufferUnit {
+ public:
+  explicit BufferUnit(int index, int width, int height);
+  ~BufferUnit();
+  void Reset(int width, int height);
+  bool Allocate();
+  void Release();
+  int Index();
+  bool isAllocate();
+  tbm_surface_h Surface();
+
+ private:
+  bool isAllocated_;
+  int index_;
+  int width_;
+  int height_;
+  tbm_surface_h tbm_surface_;
+};
+
+class BufferPool {
+ public:
+  explicit BufferPool(int width, int height);
+  ~BufferPool();
+
+  BufferUnit* AllocateBuffer();
+  BufferUnit* Get(tbm_surface_h surface);
+  void Release(BufferUnit* unit);
+  void Prepare(int with, int height);
+
+ private:
+  std::mutex mutex_;
+  std::vector<std::unique_ptr<BufferUnit>> pool_;
+};
+
+#endif

packages/webview_flutter/tizen/src/buffer_pool.cc

@@ -15,6 +15,7 @@
 #include <sstream>
 #include <string>
 
+#include "buffer_pool.h"
 #include "log.h"
 #include "lwe/LWEWebView.h"
 #include "lwe/PlatformIntegrationData.h"
@@ -170,6 +171,7 @@ WebView::WebView(flutter::PluginRegistrar* registrar, int viewId,
       context_(nullptr),
       texture_variant_(nullptr),
       gpu_buffer_(nullptr) {
+  tbm_pool_ = std::make_unique<BufferPool>(width, height);
   texture_variant_ = new flutter::TextureVariant(flutter::GpuBufferTexture(
       [this](size_t width, size_t height) -> const FlutterDesktopGpuBuffer* {
         return this->ObtainGpuBuffer(width, height);
@@ -408,9 +410,9 @@ void WebView::Resize(double width, double height) {
   height_ = height;
 
   if (candidate_surface_) {
-    tbm_surface_destroy(candidate_surface_);
     candidate_surface_ = nullptr;
   }
+  tbm_pool_->Prepare(width_, height_);
   webview_instance_->ResizeTo(width_, height_);
 }
 
@@ -786,10 +788,13 @@ void WebView::InitWebView() {
         std::lock_guard<std::mutex> lock(mutex_);
         LWE::WebContainer::ExternalImageInfo result;
         if (!candidate_surface_) {
-          candidate_surface_ =
-              tbm_surface_create(width_, height_, TBM_FORMAT_ARGB8888);
+          candidate_surface_ = tbm_pool_->AllocateBuffer();
+        }
+        if (candidate_surface_) {
+          result.imageAddress = (void*)candidate_surface_->Surface();
+        } else {
+          result.imageAddress = nullptr;
         }
-        result.imageAddress = (void*)candidate_surface_;
         return result;
       },
       [this](LWE::WebContainer* c, bool isRendered) {
@@ -931,20 +936,17 @@ void WebView::HandleCookieMethodCall(
 
 FlutterDesktopGpuBuffer* WebView::ObtainGpuBuffer(size_t width, size_t height) {
   if (!candidate_surface_) {
-    return nullptr;
+    if (!rendered_surface_) {
+      return nullptr;
+    } else {
+      return gpu_buffer_;
+    }
   }
-
   std::lock_guard<std::mutex> lock(mutex_);
-  if (rendered_surface_) {
-    tbm_surface_destroy(rendered_surface_);
-    rendered_surface_ = nullptr;
-  }
-
   rendered_surface_ = candidate_surface_;
   candidate_surface_ = nullptr;
-
   if (gpu_buffer_) {
-    gpu_buffer_->buffer = rendered_surface_;
+    gpu_buffer_->buffer = (void*)rendered_surface_->Surface();
     gpu_buffer_->width = width;
     gpu_buffer_->height = height;
   }
@@ -953,10 +955,9 @@ FlutterDesktopGpuBuffer* WebView::ObtainGpuBuffer(size_t width, size_t height) {
 
 void WebView::DestructBuffer(void* buffer) {
   if (buffer) {
-    std::lock_guard<std::mutex> lock(mutex_);
-    tbm_surface_destroy((tbm_surface_h)buffer);
-    if (rendered_surface_ == buffer) {
-      rendered_surface_ = nullptr;
+    BufferUnit* unit = tbm_pool_->Get((tbm_surface_h)buffer);
+    if (unit) {
+      tbm_pool_->Release(unit);
     }
   }
 }

packages/webview_flutter/tizen/src/buffer_pool.h

@@ -20,6 +20,8 @@ class WebContainer;
 }
 
 class TextInputChannel;
+class BufferPool;
+class BufferUnit;
 
 class WebView : public PlatformView {
  public:
@@ -68,8 +70,10 @@ class WebView : public PlatformView {
   LWE::WebContainer* webview_instance_;
   double width_;
   double height_;
-  tbm_surface_h candidate_surface_;
-  tbm_surface_h rendered_surface_;
+  //   tbm_surface_h candidate_surface_;
+  //   tbm_surface_h rendered_surface_;
+  BufferUnit* candidate_surface_;
+  BufferUnit* rendered_surface_;
   bool is_mouse_lbutton_down_;
   bool has_navigation_delegate_;
   bool has_progress_tracking_;
@@ -78,6 +82,7 @@ class WebView : public PlatformView {
   flutter::TextureVariant* texture_variant_;
   FlutterDesktopGpuBuffer* gpu_buffer_;
   std::mutex mutex_;
+  std::unique_ptr<BufferPool> tbm_pool_;
 };
 
 #endif  // FLUTTER_PLUGIN_WEBVIEW_FLUTTER_TIZEN_WEVIEW_H_