[webview_flutter] Refactor the C++ code

Author: swift-kim

packages/webview_flutter/CHANGELOG.md

@@ -1,3 +1,7 @@
+## NEXT
+
+* Code refactoring.
+
 ## 0.4.4
 
 * Update LWE binary (645719ed084d899ec7d53de1758db71a8974e446).

packages/webview_flutter/tizen/src/buffer_pool.cc

@@ -6,13 +6,7 @@
 
 #include "log.h"
 
-BufferUnit::BufferUnit(int index, int width, int height)
-    : isUsed_(false),
-      index_(index),
-      width_(0),
-      height_(0),
-      tbm_surface_(nullptr),
-      gpu_buffer_(nullptr) {
+BufferUnit::BufferUnit(int index, int width, int height) : index_(index) {
   Reset(width, height);
 }
 
@@ -28,18 +22,14 @@ BufferUnit::~BufferUnit() {
 }
 
 bool BufferUnit::MarkInUse() {
-  if (!isUsed_) {
-    isUsed_ = true;
+  if (!is_used_) {
+    is_used_ = true;
     return true;
   }
   return false;
 }
 
-FlutterDesktopGpuBuffer* BufferUnit::GpuBuffer() { return gpu_buffer_; }
-
-int BufferUnit::Index() { return index_; }
-
-bool BufferUnit::IsUsed() { return isUsed_ && tbm_surface_; }
+void BufferUnit::UnmarkInUse() { is_used_ = false; }
 
 tbm_surface_h BufferUnit::Surface() {
   if (IsUsed()) {
@@ -48,14 +38,13 @@ tbm_surface_h BufferUnit::Surface() {
   return nullptr;
 }
 
-void BufferUnit::UnmarkInUse() { isUsed_ = 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;
@@ -64,14 +53,15 @@ void BufferUnit::Reset(int width, int height) {
     delete gpu_buffer_;
     gpu_buffer_ = nullptr;
   }
+
   tbm_surface_ = tbm_surface_create(width_, height_, TBM_FORMAT_ARGB8888);
   gpu_buffer_ = new FlutterDesktopGpuBuffer();
   gpu_buffer_->width = width_;
   gpu_buffer_->height = height_;
   gpu_buffer_->buffer = tbm_surface_;
   gpu_buffer_->release_callback = [](void* release_context) {
-    BufferUnit* bu = (BufferUnit*)release_context;
-    bu->UnmarkInUse();
+    BufferUnit* buffer = reinterpret_cast<BufferUnit*>(release_context);
+    buffer->UnmarkInUse();
   };
   gpu_buffer_->release_context = this;
 }
@@ -98,9 +88,9 @@ BufferUnit* BufferPool::GetAvailableBuffer() {
   return nullptr;
 }
 
-void BufferPool::Release(BufferUnit* unit) {
+void BufferPool::Release(BufferUnit* buffer) {
   std::lock_guard<std::mutex> lock(mutex_);
-  unit->UnmarkInUse();
+  buffer->UnmarkInUse();
 }
 
 void BufferPool::Prepare(int width, int height) {
@@ -117,27 +107,28 @@ SingleBufferPool::SingleBufferPool(int width, int height)
 SingleBufferPool::~SingleBufferPool() {}
 
 BufferUnit* SingleBufferPool::GetAvailableBuffer() {
-  pool_[0].get()->MarkInUse();
-  return pool_[0].get();
+  BufferUnit* buffer = pool_[0].get();
+  buffer->MarkInUse();
+  return buffer;
 }
 
-void SingleBufferPool::Release(BufferUnit* unit) {}
+void SingleBufferPool::Release(BufferUnit* buffer) {}
 
 #ifndef NDEBUG
 #include <cairo.h>
 void BufferUnit::DumpToPng(int file_name) {
-  char filePath[256];
-  sprintf(filePath, "/tmp/dump%d.png", file_name);
-  tbm_surface_info_s surfaceInfo;
-  tbm_surface_map(tbm_surface_, TBM_SURF_OPTION_WRITE, &surfaceInfo);
-  void* buffer = surfaceInfo.planes[0].ptr;
-
-  cairo_surface_t* png_buffer;
-  png_buffer = cairo_image_surface_create_for_data(
-      (unsigned char*)buffer, CAIRO_FORMAT_ARGB32, width_, height_,
-      surfaceInfo.planes[0].stride);
-
-  cairo_surface_write_to_png(png_buffer, filePath);
+  char file_path[256];
+  sprintf(file_path, "/tmp/dump%d.png", file_name);
+
+  tbm_surface_info_s surface_info;
+  tbm_surface_map(tbm_surface_, TBM_SURF_OPTION_WRITE, &surface_info);
+
+  unsigned char* buffer = surface_info.planes[0].ptr;
+  cairo_surface_t* png_buffer = cairo_image_surface_create_for_data(
+      buffer, CAIRO_FORMAT_ARGB32, width_, height_,
+      surface_info.planes[0].stride);
+
+  cairo_surface_write_to_png(png_buffer, file_path);
 
   tbm_surface_unmap(tbm_surface_);
   cairo_surface_destroy(png_buffer);

packages/webview_flutter/tizen/src/buffer_pool.h

@@ -2,8 +2,8 @@
 // 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_
+#ifndef FLUTTER_PLUGIN_BUFFER_POOL_H_
+#define FLUTTER_PLUGIN_BUFFER_POOL_H_
 
 #include <flutter_texture_registrar.h>
 #include <tbm_surface.h>
@@ -16,24 +16,30 @@ class BufferUnit {
  public:
   explicit BufferUnit(int index, int width, int height);
   ~BufferUnit();
+
   void Reset(int width, int height);
+
   bool MarkInUse();
   void UnmarkInUse();
-  int Index();
-  bool IsUsed();
+
+  bool IsUsed() { return is_used_ && tbm_surface_; }
+
   tbm_surface_h Surface();
-  FlutterDesktopGpuBuffer* GpuBuffer();
+
+  FlutterDesktopGpuBuffer* GpuBuffer() { return gpu_buffer_; }
+
 #ifndef NDEBUG
+  // TODO: Unused code.
   void DumpToPng(int file_name);
 #endif
 
  private:
-  bool isUsed_;
+  bool is_used_ = false;
   int index_;
-  int width_;
-  int height_;
-  tbm_surface_h tbm_surface_;
-  FlutterDesktopGpuBuffer* gpu_buffer_;
+  int width_ = 0;
+  int height_ = 0;
+  tbm_surface_h tbm_surface_ = nullptr;
+  FlutterDesktopGpuBuffer* gpu_buffer_ = nullptr;
 };
 
 class BufferPool {
@@ -42,7 +48,8 @@ class BufferPool {
   virtual ~BufferPool();
 
   virtual BufferUnit* GetAvailableBuffer();
-  virtual void Release(BufferUnit* unit);
+  virtual void Release(BufferUnit* buffer);
+
   void Prepare(int with, int height);
 
  protected:
@@ -58,8 +65,8 @@ class SingleBufferPool : public BufferPool {
   explicit SingleBufferPool(int width, int height);
   ~SingleBufferPool();
 
-  virtual BufferUnit* GetAvailableBuffer();
-  virtual void Release(BufferUnit* unit);
+  virtual BufferUnit* GetAvailableBuffer() override;
+  virtual void Release(BufferUnit* buffer) override;
 };
 
-#endif
+#endif  // FLUTTER_PLUGIN_BUFFER_POOL_H_