Title: [92190] trunk/Source/WebKit/efl
- Revision
- 92190
- Author
- [email protected]
- Date
- 2011-08-02 08:18:31 -0700 (Tue, 02 Aug 2011)
Log Message
[EFL] Remove overlapping recursive layout function
https://bugs.webkit.org/show_bug.cgi?id=63241
First, _ewk_view_layout_if_needed_recursive is called twice in _ewk_view_smart_calculate
and _ewk_view_tiled_updates_process_pre,so it is no problem removing one of them.
Second, In tiled backingstore configuration, _ewk_view_layout_if_needed_recursive called
in _ewk_view_smart_calculate will result in generating repaints for dirty areas
located outside of viewport since m_paintEntireContents is set.
But, the areas will not be actually painted. Thus, the layout is unnecessary except
when there is something to paint inside of viewport, which will consume CPU unnecessarily.
So,the layout func was removed from _ewk_view_smart_calculate
and added to _ewk_view_single_smart_repaints_process for single backingstore.
Patch by Eunsol Park <[email protected]> on 2011-08-02
Reviewed by Antonio Gomes.
* ewk/ewk_view.cpp:
(_ewk_view_smart_calculate):
* ewk/ewk_view_single.c:
(_ewk_view_single_smart_repaints_process):
Modified Paths
Diff
Modified: trunk/Source/WebKit/efl/ChangeLog (92189 => 92190)
--- trunk/Source/WebKit/efl/ChangeLog 2011-08-02 15:08:45 UTC (rev 92189)
+++ trunk/Source/WebKit/efl/ChangeLog 2011-08-02 15:18:31 UTC (rev 92190)
@@ -1,3 +1,25 @@
+2011-08-02 Eunsol Park <[email protected]>
+
+ [EFL] Remove overlapping recursive layout function
+ https://bugs.webkit.org/show_bug.cgi?id=63241
+
+ First, _ewk_view_layout_if_needed_recursive is called twice in _ewk_view_smart_calculate
+ and _ewk_view_tiled_updates_process_pre,so it is no problem removing one of them.
+ Second, In tiled backingstore configuration, _ewk_view_layout_if_needed_recursive called
+ in _ewk_view_smart_calculate will result in generating repaints for dirty areas
+ located outside of viewport since m_paintEntireContents is set.
+ But, the areas will not be actually painted. Thus, the layout is unnecessary except
+ when there is something to paint inside of viewport, which will consume CPU unnecessarily.
+ So,the layout func was removed from _ewk_view_smart_calculate
+ and added to _ewk_view_single_smart_repaints_process for single backingstore.
+
+ Reviewed by Antonio Gomes.
+
+ * ewk/ewk_view.cpp:
+ (_ewk_view_smart_calculate):
+ * ewk/ewk_view_single.c:
+ (_ewk_view_single_smart_repaints_process):
+
2011-08-02 Gyuyoung Kim <[email protected]>
[EFL] Add fullscreen button to media control UI.
Modified: trunk/Source/WebKit/efl/ewk/ewk_view.cpp (92189 => 92190)
--- trunk/Source/WebKit/efl/ewk/ewk_view.cpp 2011-08-02 15:08:45 UTC (rev 92189)
+++ trunk/Source/WebKit/efl/ewk/ewk_view.cpp 2011-08-02 15:18:31 UTC (rev 92190)
@@ -853,8 +853,6 @@
}
sd->changed.position = EINA_FALSE;
- ewk_view_layout_if_needed_recursive(sd->_priv);
-
if (!sd->api->scrolls_process(sd))
ERR("failed to process scrolls.");
_ewk_view_scrolls_flush(priv);
Modified: trunk/Source/WebKit/efl/ewk/ewk_view_single.c (92189 => 92190)
--- trunk/Source/WebKit/efl/ewk/ewk_view_single.c 2011-08-02 15:08:45 UTC (rev 92189)
+++ trunk/Source/WebKit/efl/ewk/ewk_view_single.c 2011-08-02 15:18:31 UTC (rev 92190)
@@ -437,6 +437,8 @@
goto error_iterator;
}
+ ewk_view_layout_if_needed_recursive(sd->_priv);
+
int sx, sy;
ewk_frame_scroll_pos_get(sd->main_frame, &sx, &sy);
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes
