From 5221d005e398f9cff36f1d93029f839d960c61b2 Mon Sep 17 00:00:00 2001
From: Gwanglim Lee <gl77.lee@samsung.com>
Date: Wed, 6 Dec 2017 20:30:56 +0900
Subject: [PATCH] e_main: removed e_first_frame feature.

It was intended for measuring initialization time of the enlightenment.
But it used insecure getenv to enable feature. We will try to change this
feature to use e_config later.

Change-Id: Ieef5ce450e8ec2f0dfa26b7c66c3ad80cf00eb2b
---
 src/bin/e.h             |  3 ---
 src/bin/e_comp_canvas.c | 19 -------------------
 src/bin/e_main.c        |  8 --------
 3 files changed, 30 deletions(-)

diff --git a/src/bin/e.h b/src/bin/e.h
index 18d5f94081..23a5a0ae2d 100644
--- a/src/bin/e.h
+++ b/src/bin/e.h
@@ -337,9 +337,6 @@ extern E_API Eina_Bool e_nopause;
 
 extern E_API Eina_Bool e_precache_end;
 
-extern EINTERN const char *e_first_frame;
-extern EINTERN double e_first_frame_start_time;
-
 //#define SMARTERR(args...) abort()
 #define SMARTERRNR() return
 #define SMARTERR(x)  return x
diff --git a/src/bin/e_comp_canvas.c b/src/bin/e_comp_canvas.c
index d7e1bf3e31..acdfd6071f 100644
--- a/src/bin/e_comp_canvas.c
+++ b/src/bin/e_comp_canvas.c
@@ -17,23 +17,6 @@ _e_comp_canvas_event_compositor_resize_free(void *data EINA_UNUSED, void *event
 
 ///////////////////////////////////
 
-static void
-_e_comp_canvas_cb_first_frame(void *data EINA_UNUSED, Evas *e, void *event_info EINA_UNUSED)
-{
-   double now = ecore_time_get();
-
-   switch (e_first_frame[0])
-     {
-      case 'A': abort();
-      case 'E':
-      case 'D': exit(-1);
-      case 'T': fprintf(stderr, "Startup time: '%f' - '%f' = '%f'\n", now, e_first_frame_start_time, now - e_first_frame_start_time);
-         break;
-     }
-
-   evas_event_callback_del_full(e, EVAS_CALLBACK_RENDER_POST, _e_comp_canvas_cb_first_frame, NULL);
-}
-
 static void
 _e_comp_canvas_render_post(void *data EINA_UNUSED, Evas *e EINA_UNUSED, void *event_info EINA_UNUSED)
 {
@@ -170,8 +153,6 @@ e_comp_canvas_init(int w, int h)
         opmode = e_config->comp_canvas_bg.opmode;
      }
 
-   if (e_first_frame)
-     evas_event_callback_add(e_comp->evas, EVAS_CALLBACK_RENDER_POST, _e_comp_canvas_cb_first_frame, NULL);
    o = evas_object_rectangle_add(e_comp->evas);
    e_comp->bg_blank_object = o;
    evas_object_layer_set(o, E_LAYER_BOTTOM);
diff --git a/src/bin/e_main.c b/src/bin/e_main.c
index d005fd362c..3c9d6b4b19 100644
--- a/src/bin/e_main.c
+++ b/src/bin/e_main.c
@@ -140,8 +140,6 @@ E_API Eina_Bool starting = EINA_TRUE;
 E_API Eina_Bool stopping = EINA_FALSE;
 E_API Eina_Bool restart = EINA_FALSE;
 E_API Eina_Bool e_nopause = EINA_FALSE;
-EINTERN const char *e_first_frame = NULL;
-EINTERN double e_first_frame_start_time = -1;
 
 static Eina_Bool
 _xdg_check_str(const char *env, const char *str)
@@ -465,12 +463,6 @@ main(int argc, char **argv)
    TSE("Ecore Init Done");
    _e_main_shutdown_push(ecore_shutdown);
 
-   e_first_frame = getenv("E_FIRST_FRAME");
-   if (e_first_frame && e_first_frame[0])
-     e_first_frame_start_time = ecore_time_get();
-   else
-     e_first_frame = NULL;
-
    TSB("EIO Init");
    if (!eio_init())
      {
-- 
2.34.1