[audioplayers] Remove Ecore API

Replace the Ecore main-loop and timer APIs with their GLib APIs.

Author: JSUYA

packages/audioplayers/CHANGELOG.md

@@ -1,3 +1,7 @@
+## 3.1.4
+
+* Remove Ecore API.
+
 ## 3.1.3
 
 * Adds compatibility with `http` 1.0 in example.

packages/audioplayers/README.md

@@ -11,7 +11,7 @@ This package is not an _endorsed_ implementation of `audioplayers`. Therefore, y
 ```yaml
 dependencies:
   audioplayers: ^6.6.0
-  audioplayers_tizen: ^3.1.3
+  audioplayers_tizen: ^3.1.4
 
 ```
 

packages/audioplayers/pubspec.yaml

@@ -2,7 +2,7 @@ name: audioplayers_tizen
 description: Tizen implementation of the audioplayers plugin.
 homepage: https://github.com/flutter-tizen/plugins
 repository: https://github.com/flutter-tizen/plugins/tree/master/packages/audioplayers
-version: 3.1.3
+version: 3.1.4
 
 environment:
   sdk: ^3.6.0

packages/audioplayers/tizen/src/audio_player.cc

@@ -30,9 +30,9 @@ AudioPlayer::~AudioPlayer() {
     player_destroy(player_);
     player_ = nullptr;
   }
-  if (timer_) {
-    ecore_timer_del(timer_);
-    timer_ = nullptr;
+  if (timer_id_ != 0) {
+    g_source_remove(timer_id_);
+    timer_id_ = 0;
   }
 }
 
@@ -346,8 +346,8 @@ void AudioPlayer::OnPrepared(void *data) {
   // On TV devices, callbacks are not executed on the main loop. Therefore
   // we explicitly transfer the callback to the main loop to avoid any race
   // conditions and to allow creating timer objects in StartPositionUpdates.
-  ecore_main_loop_thread_safe_call_async(
-      [](void *data) {
+  g_idle_add(
+      [](gpointer data) -> gboolean {
         auto *player = reinterpret_cast<AudioPlayer *>(data);
         player->preparing_ = false;
 
@@ -356,15 +356,15 @@ void AudioPlayer::OnPrepared(void *data) {
           player->prepared_listener_(player->player_id_, true);
         } catch (const AudioPlayerError &error) {
           player->log_listener_(player->player_id_, error.code());
-          return;
+          return G_SOURCE_REMOVE;
         }
         player_set_playback_rate(player->player_, player->playback_rate_);
 
         if (player->should_play_) {
           int ret = player_start(player->player_);
           if (ret != PLAYER_ERROR_NONE) {
             player->log_listener_(player->player_id_, "player_start failed.");
-            return;
+            return G_SOURCE_REMOVE;
           }
           player->StartPositionUpdates();
           player->should_play_ = false;
@@ -379,10 +379,11 @@ void AudioPlayer::OnPrepared(void *data) {
             player->seeking_ = false;
             player->log_listener_(player->player_id_,
                                   "player_set_play_position failed.");
-            return;
+            return G_SOURCE_REMOVE;
           }
           player->should_seek_to_ = -1;
         }
+        return G_SOURCE_REMOVE;
       },
       data);
 }
@@ -391,11 +392,12 @@ void AudioPlayer::OnSeekCompleted(void *data) {
   // On TV devices, callbacks are not executed on the main loop. Therefore
   // we explicitly transfer the callback to the main loop to avoid any race
   // conditions.
-  ecore_main_loop_thread_safe_call_async(
-      [](void *data) {
+  g_idle_add(
+      [](gpointer data) -> gboolean {
         auto *player = reinterpret_cast<AudioPlayer *>(data);
         player->seek_completed_listener_(player->player_id_);
         player->seeking_ = false;
+        return G_SOURCE_REMOVE;
       },
       data);
 }
@@ -404,8 +406,8 @@ void AudioPlayer::OnPlayCompleted(void *data) {
   // On TV devices, callbacks are not executed on the main loop. Therefore
   // we explicitly transfer the callback to the main loop to avoid any race
   // conditions.
-  ecore_main_loop_thread_safe_call_async(
-      [](void *data) {
+  g_idle_add(
+      [](gpointer data) -> gboolean {
         auto *player = reinterpret_cast<AudioPlayer *>(data);
         try {
           player->Seek(0);
@@ -414,6 +416,7 @@ void AudioPlayer::OnPlayCompleted(void *data) {
         } catch (const AudioPlayerError &error) {
           player->log_listener_(player->player_id_, error.code());
         }
+        return G_SOURCE_REMOVE;
       },
       data);
 }
@@ -433,27 +436,27 @@ void AudioPlayer::OnError(int code, void *data) {
 }
 
 void AudioPlayer::StartPositionUpdates() {
-  if (!timer_) {
+  if (timer_id_ == 0) {
     // The audioplayers app facing package expects position
     // update events to fire roughly every 200 milliseconds.
-    const double kTimeInterval = 0.2;
-    timer_ = ecore_timer_add(kTimeInterval, OnPositionUpdate, this);
-    if (!timer_) {
+    const guint kTimeInterval = 200;
+    timer_id_ = g_timeout_add(kTimeInterval, OnPositionUpdate, this);
+    if (timer_id_ == 0) {
       log_listener_(player_id_, "Failed to add a position update timer.");
     }
   }
 }
 
-Eina_Bool AudioPlayer::OnPositionUpdate(void *data) {
+gboolean AudioPlayer::OnPositionUpdate(gpointer data) {
   auto *player = reinterpret_cast<AudioPlayer *>(data);
   try {
     if (player->IsPlaying()) {
       player->duration_listener_(player->player_id_, player->GetDuration());
-      return ECORE_CALLBACK_RENEW;
+      return G_SOURCE_CONTINUE;
     }
   } catch (const AudioPlayerError &error) {
     player->log_listener_(player->player_id_, "Failed to update position.");
   }
-  player->timer_ = nullptr;
-  return ECORE_CALLBACK_CANCEL;
+  player->timer_id_ = 0;
+  return G_SOURCE_REMOVE;
 }

packages/audioplayers/tizen/src/audio_player.h

@@ -5,7 +5,7 @@
 #ifndef FLUTTER_PLUGIN_AUDIO_PLAYER_H_
 #define FLUTTER_PLUGIN_AUDIO_PLAYER_H_
 
-#include <Ecore.h>
+#include <glib.h>
 #include <player.h>
 
 #include <functional>
@@ -71,7 +71,7 @@ class AudioPlayer {
   static void OnPlayCompleted(void *data);
   static void OnInterrupted(player_interrupted_code_e code, void *data);
   static void OnError(int code, void *data);
-  static Eina_Bool OnPositionUpdate(void *data);
+  static gboolean OnPositionUpdate(gpointer data);
 
   player_h player_ = nullptr;
   const std::string player_id_;
@@ -84,7 +84,7 @@ class AudioPlayer {
   bool preparing_ = false;
   bool seeking_ = false;
   bool should_play_ = false;
-  Ecore_Timer *timer_ = nullptr;
+  guint timer_id_ = 0;
 
   PreparedListener prepared_listener_;
   DurationListener duration_listener_;