From: MinJeong Kim <minjjj.kim@samsung.com>
Date: Thu, 23 Jul 2015 10:08:10 +0000 (+0900)
Subject: Use resource id and Fix wrong use of Eina_List
X-Git-Tag: submit/tizen/20150728.101124~1
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=a3a921d67ff439de32b829529a42a9b7a57e15bc;p=platform%2Fcore%2Fuifw%2Fe-tizen-testcase.git

Use resource id and Fix wrong use of Eina_List

Change-Id: Iae45d30c6a2ff30c098d07fad7fe9b0bd76380e6
Signed-off-by: MinJeong Kim <minjjj.kim@samsung.com>
---

diff --git a/configure.ac b/configure.ac
index 9d25352..4390dd5 100644
--- a/configure.ac
+++ b/configure.ac
@@ -49,7 +49,8 @@ requirements="\
 if test "x$WITH_WAYLAND" = x"yes"; then
 requirements="$requirements \
               wayland-client \
-              ecore-wayland"
+              ecore-wayland \
+              tizen-extension-client"
 fi
 
 PKG_CHECK_MODULES(E_TEST_RUNNER, [${requirements}])
diff --git a/packaging/e-tizen-testcase.spec b/packaging/e-tizen-testcase.spec
index 2f27cf5..2409c85 100644
--- a/packaging/e-tizen-testcase.spec
+++ b/packaging/e-tizen-testcase.spec
@@ -12,7 +12,11 @@ License: BSD-2-Clause
 BuildRequires: pkgconfig(enlightenment)
 BuildRequires: pkgconfig(eldbus)
 BuildRequires: pkgconfig(capi-ui-efl-util)
-BuildRequires:  gettext
+BuildRequires: gettext
+%if %{with wayland}
+BuildRequires: pkgconfig(tizen-extension-client)
+%endif
+
 
 %description
 This package is a test case runner for enlightenment.
diff --git a/src/e_test_runner.c b/src/e_test_runner.c
index 8059282..e81e1d8 100644
--- a/src/e_test_runner.c
+++ b/src/e_test_runner.c
@@ -2,6 +2,79 @@
 
 int _log_dom = -1;
 
+#if HAVE_WAYLAND
+static void
+_e_test_runner_cb_resource_id(void *data,
+                              struct tizen_resource *tizen_resource EINA_UNUSED,
+                              uint32_t id)
+{
+   Ecore_Window *res_id = data;
+
+   *res_id = id;
+
+   elm_exit();
+}
+
+static const struct tizen_resource_listener _ecore_tizen_resource_listener =
+{
+   _e_test_runner_cb_resource_id,
+};
+#endif
+
+static Ecore_Window
+_e_test_runner_window_id_get(Evas_Object *elm_win)
+{
+#if HAVE_WAYLAND
+   Ecore_Wl_Window *wlwin;
+   struct wl_surface *surf;
+   Ecore_Window id = 0;
+
+   Eina_Inlist *globals;
+   Ecore_Wl_Global *global;
+   struct wl_registry *registry;
+
+   struct tizen_surface *tizen_surface = NULL;
+   struct tizen_resource *tizen_resource = NULL;
+
+   wlwin = elm_win_wl_window_get(elm_win);
+   if (!wlwin) return 0;
+
+   surf = ecore_wl_window_surface_get(wlwin);
+   if (!surf) return 0;
+
+   registry = ecore_wl_registry_get();
+   globals = ecore_wl_globals_get();
+
+   if (!registry || !globals) return 0;
+
+   EINA_INLIST_FOREACH(globals, global)
+     {
+        if (!strcmp(global->interface, "tizen_surface"))
+          {
+             tizen_surface =
+                wl_registry_bind(registry, global->id,
+                                 &tizen_surface_interface, 1);
+             break;
+          }
+     }
+
+   if (!tizen_surface) return 0;
+
+   tizen_resource = tizen_surface_get_tizen_resource(tizen_surface, surf);
+   if (!tizen_resource) return 0;
+   tizen_resource_add_listener(tizen_resource,
+                               &_ecore_tizen_resource_listener, &id);
+
+   elm_run();
+
+   if (tizen_resource) tizen_resource_destroy(tizen_resource);
+   if (tizen_surface) tizen_surface_destroy(tizen_surface);
+
+   return id;
+#endif
+   return elm_win_window_id_get(elm_win);
+}
+
 static void
 _cb_method_win_info_list_get(void *data,
                              const Eldbus_Message *msg,
@@ -19,8 +92,6 @@ _cb_method_win_info_list_get(void *data,
    res = eldbus_message_arguments_get(msg, "ua(usiiiiibb)", &target_win, &array);
    EINA_SAFETY_ON_FALSE_GOTO(res, finish);
 
-   list = eina_list_free(list);
-
    while (eldbus_message_iter_get_and_next(array, 'r', &ec))
      {
         const char *win_name;
@@ -265,6 +336,7 @@ e_test_runner_req_win_info_list_get(E_Test_Runner *runner)
    EINA_SAFETY_ON_NULL_RETURN_VAL(p, NULL);
 
    elm_run();
+   list = eina_list_remove(list, NULL);
 
    return list;
 }
@@ -367,7 +439,7 @@ e_tc_win_add(E_TC_Win *parent,
    EINA_SAFETY_ON_NULL_RETURN_VAL(tw, NULL);
 
    tw->elm_win = elm_win;
-   tw->native_win = elm_win_window_id_get(elm_win);
+   tw->native_win = _e_test_runner_window_id_get(elm_win);
    tw->name = eina_stringshare_add(name);
    tw->x = x;
    tw->y = y;
@@ -376,10 +448,6 @@ e_tc_win_add(E_TC_Win *parent,
    tw->layer = layer;
    tw->alpha = alpha;
 
-#if HAVE_WAYLAND
-   tw->native_win = ((uint64_t)tw->native_win << 32) + getpid();
-#endif
-
    return tw;
 
 err:
diff --git a/src/e_test_runner.h b/src/e_test_runner.h
index c113a23..3460f15 100644
--- a/src/e_test_runner.h
+++ b/src/e_test_runner.h
@@ -5,6 +5,12 @@
 #include <Eldbus.h>
 #include <Elementary.h>
 
+#if HAVE_WAYLAND
+#include <Ecore_Wayland.h>
+#include <wayland-client.h>
+#include <tizen-extension-client-protocol.h>
+#endif
+
 extern int _log_dom;
 
 #ifdef ERR