[webkit-changes] [89800] trunk/Source/WebKit/efl

Mon, 27 Jun 2011 01:01:57 -0700

Title: [89800] trunk/Source/WebKit/efl
Revision
89800
Author
[email protected]
Date
2011-06-27 01:01:50 -0700 (Mon, 27 Jun 2011)

Log Message

2011-06-27  KwangHyuk Kim  <[email protected]>

        Reviewed by Kenneth Rohde Christiansen.

        [EFL] Remove dead code related with Ewk_Tiled_Backing_Store_Item's update field.
        https://bugs.webkit.org/show_bug.cgi?id=62967

        As some code do not affect any logical flow of tiled backing store except condition check,
        I would like to clean dead code related with Ewk_Tiled_Backing_Store_Item's update field
        and Ewk_Tiled_Backing_Store_Data's queue field

        * ewk/ewk_tiled_backing_store.c:
        (_ewk_tiled_backing_store_item_process_idler_cb):
        (_ewk_tiled_backing_store_item_fill):
        (_ewk_tiled_backing_store_item_add):
        (_ewk_tiled_backing_store_item_del):

Modified Paths

Property Changed

Diff

Modified: trunk/Source/WebKit/efl/ChangeLog (89799 => 89800)


--- trunk/Source/WebKit/efl/ChangeLog	2011-06-27 07:59:48 UTC (rev 89799)
+++ trunk/Source/WebKit/efl/ChangeLog	2011-06-27 08:01:50 UTC (rev 89800)
@@ -1,3 +1,20 @@
+2011-06-27  KwangHyuk Kim  <[email protected]>
+
+        Reviewed by Kenneth Rohde Christiansen.
+
+        [EFL] Remove dead code related with Ewk_Tiled_Backing_Store_Item's update field.
+        https://bugs.webkit.org/show_bug.cgi?id=62967
+
+        As some code do not affect any logical flow of tiled backing store except condition check,
+        I would like to clean dead code related with Ewk_Tiled_Backing_Store_Item's update field
+        and Ewk_Tiled_Backing_Store_Data's queue field 
+
+        * ewk/ewk_tiled_backing_store.c:
+        (_ewk_tiled_backing_store_item_process_idler_cb):
+        (_ewk_tiled_backing_store_item_fill):
+        (_ewk_tiled_backing_store_item_add):
+        (_ewk_tiled_backing_store_item_del):
+
 2011-06-26  Ryuan Choi  <[email protected]>
 
         Reviewed by Antonio Gomes.

Modified: trunk/Source/WebKit/efl/ewk/ewk_tiled_backing_store.c (89799 => 89800)


--- trunk/Source/WebKit/efl/ewk/ewk_tiled_backing_store.c	2011-06-27 07:59:48 UTC (rev 89799)
+++ trunk/Source/WebKit/efl/ewk/ewk_tiled_backing_store.c	2011-06-27 08:01:50 UTC (rev 89800)
@@ -57,11 +57,6 @@
     struct {
         Evas_Coord x, y, w, h;
     } geometry;
-    struct {
-        Eina_List *process;
-        unsigned long row, col;
-        float zoom;
-    } update;
     Eina_Bool smooth_scale;
 };
 
@@ -104,7 +99,6 @@
     struct {
         Eina_Bool (*cb)(void *data, Ewk_Tile *t, const Eina_Rectangle *area);
         void *data;
-        Eina_List *queue;
         Eina_Bool process_entire_queue;
         Eina_Inlist *pre_render_requests;
         Ecore_Idler *idler;
@@ -377,50 +371,17 @@
     Ewk_Tiled_Backing_Store_Data *priv = data;
     Ewk_Tiled_Backing_Store_Item *it = NULL;
 
-    while (priv->render.queue) {
-        it = priv->render.queue->data;
-        if (it->tile->zoom == priv->view.tile.zoom) {
-            _ewk_tiled_backing_store_item_request_del(priv, it);
-            it = NULL;
-        } else {
-            unsigned long row, col;
-            float zoom;
-            Ewk_Tile *t;
-            if (it->tile) {
-                double last_used = ecore_loop_time_get();
-                _ewk_tiled_backing_store_tile_dissociate(priv, it, last_used);
-            }
-
-            row = it->update.row;
-            col = it->update.col;
-            zoom = it->update.zoom;
-            t = _ewk_tiled_backing_store_tile_new(priv, col, row, zoom);
-            if (!t) {
-                priv->render.idler = NULL;
-                return EINA_FALSE;
-            }
-
-            _ewk_tiled_backing_store_tile_associate(priv, t, it);
-            it->update.process = NULL;
-            priv->render.queue = eina_list_remove_list(priv->render.queue,
-                                                       priv->render.queue);
-            if (!priv->render.process_entire_queue)
-                break;
-        }
-    }
-
     if (priv->process.pre_cb)
         data = "" priv->self);
 
     ewk_tile_matrix_updates_process(priv->model.matrix);
 
-    if (!it)
-        _ewk_tiled_backing_store_pre_render_request_process_single(priv);
+    _ewk_tiled_backing_store_pre_render_request_process_single(priv);
 
     if (priv->process.post_cb)
         priv->process.post_cb(priv->process.post_data, data, priv->self);
 
-    if (!priv->render.queue && !priv->render.pre_render_requests) {
+    if (!priv->render.pre_render_requests) {
         priv->render.idler = NULL;
         return EINA_FALSE;
     }
@@ -445,29 +406,6 @@
         _ewk_tiled_backing_store_item_process_idler_cb, priv);
 }
 
-static inline void _ewk_tiled_backing_store_item_request_del(Ewk_Tiled_Backing_Store_Data *priv, Ewk_Tiled_Backing_Store_Item *it)
-{
-    priv->render.queue = eina_list_remove_list(priv->render.queue,
-                                               it->update.process);
-    it->update.process = NULL;
-}
-
-static inline void _ewk_tiled_backing_store_item_request_add(Ewk_Tiled_Backing_Store_Data *priv, Ewk_Tiled_Backing_Store_Item *it, int m_col, int m_row, float zoom)
-{
-    if (it->update.process)
-        return;
-
-    it->update.col = m_col;
-    it->update.row = m_row;
-    it->update.zoom = zoom;
-
-    priv->render.queue = eina_list_append(priv->render.queue, it);
-    it->update.process = eina_list_last(priv->render.queue);
-
-    if (!priv->render.suspend)
-        _ewk_tiled_backing_store_item_process_idler_start(priv);
-}
-
 static Eina_Bool _ewk_tiled_backing_store_disable_render(Ewk_Tiled_Backing_Store_Data *priv)
 {
     if (priv->render.suspend)
@@ -501,24 +439,12 @@
         || (unsigned long)(m_col) >= priv->model.cur.cols
         || (unsigned long)(m_row) >= priv->model.cur.rows) {
 
-        if (it->tile) {
+        if (it->tile)
             _ewk_tiled_backing_store_tile_dissociate(priv, it, last_used);
-            if (it->update.process)
-                _ewk_tiled_backing_store_item_request_del(priv, it);
-        }
     } else {
         Ewk_Tile *t;
         const float zoom = priv->view.tile.zoom;
 
-        if (it->update.process) {
-            if (it->update.row == (unsigned long)(m_row)
-                && it->update.col == (unsigned long)(m_col)
-                && it->update.zoom == zoom)
-                return EINA_TRUE;
-
-            _ewk_tiled_backing_store_item_request_del(priv, it);
-        }
-
         if (it->tile) {
             Ewk_Tile *old = it->tile;
             if (old->row != (unsigned long)(m_row)
@@ -526,8 +452,7 @@
                 || old->zoom != zoom) {
                 _ewk_tiled_backing_store_tile_dissociate(priv, it,
                                                          last_used);
-                if (it->update.process)
-                    _ewk_tiled_backing_store_item_request_del(priv, it);
+
             } else if (old->row == (unsigned long)(m_row)
                        && old->col == (unsigned long)(m_col)
                        && old->zoom == zoom)
@@ -552,12 +477,10 @@
                 _ewk_tiled_backing_store_tile_associate(priv, t, it);
             }
         } else if (t != it->tile) {
-            if (!it->update.process) {
-                if (it->tile)
-                    _ewk_tiled_backing_store_tile_dissociate(priv,
+            if (it->tile)
+                _ewk_tiled_backing_store_tile_dissociate(priv,
                                                              it, last_used);
-                _ewk_tiled_backing_store_tile_associate(priv, t, it);
-            }
+            _ewk_tiled_backing_store_tile_associate(priv, t, it);
         }
 
       end:
@@ -583,7 +506,7 @@
     y = priv->view.offset.base.y + priv->view.y + th  *row;
 
     it->tile = NULL;
-    it->update.process = NULL;
+
     it->smooth_scale = priv->view.tile.zoom_weak_smooth_scale;
     _ewk_tiled_backing_store_item_move(it, x, y);
     _ewk_tiled_backing_store_item_resize(it, tw, th);
@@ -601,8 +524,7 @@
         double last_used = ecore_loop_time_get();
         _ewk_tiled_backing_store_tile_dissociate(priv, it, last_used);
     }
-    if (it->update.process)
-        _ewk_tiled_backing_store_item_request_del(priv, it);
+
     free(it);
 }
 
Property changes on: trunk/Source/WebKit/efl/ewk/ewk_tiled_backing_store.c
___________________________________________________________________

Added: svn:executable

_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to