Remove cbhm dependency (#64)

Author: JSUYA

flutter/shell/platform/tizen/BUILD.gn

@@ -13,7 +13,6 @@ config("flutter_tizen_config") {
   include_dirs = [
     "${sysroot_path}/usr/include/appfw",
     "${sysroot_path}/usr/include/base",
-    "${sysroot_path}/usr/include/cbhm",
     "${sysroot_path}/usr/include/dali",
     "${sysroot_path}/usr/include/dali-adaptor",
     "${sysroot_path}/usr/include/dali-toolkit",
@@ -170,11 +169,6 @@ template("embedder") {
       ]
     }
 
-    if (target_name == "flutter_tizen_mobile" ||
-        target_name == "flutter_tizen_common") {
-      libs += [ "cbhm" ]
-    }
-
     defines += invoker.defines
     defines += [ "FLUTTER_ENGINE_NO_PROTOTYPES" ]
 

flutter/shell/platform/tizen/channels/platform_channel.cc

@@ -81,25 +81,14 @@ PlatformChannel::PlatformChannel(BinaryMessenger* messenger,
           kChannelName,
           &JsonMethodCodec::GetInstance())),
       view_(view) {
-#if defined(MOBILE_PROFILE) || defined(COMMON_PROFILE)
-  int ret = cbhm_open_service(&cbhm_handle_);
-  if (ret != CBHM_ERROR_NONE) {
-    FT_LOG(Error) << "Failed to initialize the clipboard service.";
-  }
-#endif
-
   channel_->SetMethodCallHandler(
       [this](const MethodCall<rapidjson::Document>& call,
              std::unique_ptr<MethodResult<rapidjson::Document>> result) {
         HandleMethodCall(call, std::move(result));
       });
 }
 
-PlatformChannel::~PlatformChannel() {
-#if defined(MOBILE_PROFILE) || defined(COMMON_PROFILE)
-  cbhm_close_service(cbhm_handle_);
-#endif
-}
+PlatformChannel::~PlatformChannel() {}
 
 void PlatformChannel::HandleMethodCall(
     const MethodCall<rapidjson::Document>& method_call,
@@ -231,56 +220,15 @@ void PlatformChannel::HapticFeedbackVibrate(const std::string& feedback_type) {
 }
 
 void PlatformChannel::GetClipboardData(ClipboardCallback on_data) {
-  on_clipboard_data_ = std::move(on_data);
-
-#if defined(MOBILE_PROFILE) || defined(COMMON_PROFILE)
-  int ret = cbhm_selection_get(
-      cbhm_handle_, CBHM_SEL_TYPE_TEXT,
-      [](cbhm_h cbhm_handle, const char* buf, size_t len,
-         void* user_data) -> int {
-        auto* self = static_cast<PlatformChannel*>(user_data);
-        std::string data;
-        if (buf) {
-          data = std::string(buf, len);
-        }
-        self->on_clipboard_data_(data);
-        self->on_clipboard_data_ = nullptr;
-        return CBHM_ERROR_NONE;
-      },
-      this);
-  if (ret != CBHM_ERROR_NONE) {
-    if (ret == CBHM_ERROR_NO_DATA) {
-      FT_LOG(Info) << "No clipboard data available.";
-    } else {
-      FT_LOG(Error) << "Failed to get clipboard data.";
-    }
-    on_clipboard_data_("");
-    on_clipboard_data_ = nullptr;
-  }
-#else
-  on_clipboard_data_(clipboard_);
-  on_clipboard_data_ = nullptr;
-#endif
+  on_data(clipboard_);
 }
 
 void PlatformChannel::SetClipboardData(const std::string& data) {
-#if defined(MOBILE_PROFILE) || defined(COMMON_PROFILE)
-  int ret = cbhm_selection_set(cbhm_handle_, CBHM_SEL_TYPE_TEXT, data.c_str(),
-                               data.length());
-  if (ret != CBHM_ERROR_NONE) {
-    FT_LOG(Error) << "Failed to set clipboard data.";
-  }
-#else
   clipboard_ = data;
-#endif
 }
 
 bool PlatformChannel::ClipboardHasStrings() {
-#if defined(MOBILE_PROFILE) || defined(COMMON_PROFILE)
-  return cbhm_item_count_get(cbhm_handle_) > 0;
-#else
   return !clipboard_.empty();
-#endif
 }
 
 void PlatformChannel::RestoreSystemUiOverlays() {

flutter/shell/platform/tizen/channels/platform_channel.h

@@ -5,10 +5,6 @@
 #ifndef EMBEDDER_PLATFORM_CHANNEL_H_
 #define EMBEDDER_PLATFORM_CHANNEL_H_
 
-#if defined(MOBILE_PROFILE) || defined(COMMON_PROFILE)
-#include <cbhm.h>
-#endif
-
 #include <functional>
 #include <memory>
 #include <string>
@@ -53,18 +49,10 @@ class PlatformChannel {
   // A reference to the native view managed by FlutterTizenView.
   TizenViewBase* view_ = nullptr;
 
-#if defined(MOBILE_PROFILE) || defined(COMMON_PROFILE)
-  // The clipboard history manager.
-  cbhm_h cbhm_handle_ = nullptr;
-#else
   // A container that holds clipboard data during the engine lifetime.
   //
-  // Only used by profiles that do not support the Tizen clipboard API
-  // (wearable and TV).
+  // TODO(JSUYA): Remove after implementing the ecore_wl2 based clipboard.
   std::string clipboard_;
-#endif
-
-  ClipboardCallback on_clipboard_data_ = nullptr;
 };
 
 }  // namespace flutter

tools/generate_sysroot.py

@@ -43,8 +43,6 @@
   'capi-system-system-settings-devel',
   'capi-ui-efl-util',
   'capi-ui-efl-util-devel',
-  'cbhm',
-  'cbhm-devel',
   'coregl',
   'coregl-devel',
   'ecore-con-devel',