RESOLVED DUPLICATE of bug 70170 70900
[EFL] Invalidate request for the single view can be discarded without rendering. 
https://bugs.webkit.org/show_bug.cgi?id=70900
Summary [EFL] Invalidate request for the single view can be discarded without rendering.
KwangHyuk
Reported 2011-10-26 03:20:40 PDT
As ewk_view_layout_if_needed_recursive api causes invalidate request via ChromeClient, it can be discarded without rendering as long as it would be called after ewk_view_repaints_get api. So, It is better to be move into other position, but I couldn't make sure whether it's really good when it must be handled whenever ewk_view_layout_if_needed_recursive api is called.
Attachments
Add attachment proposed patch, testcase, etc.
Ryuan Choi
Comment 1 2011-10-26 16:29:37 PDT
(In reply to comment #0) > As ewk_view_layout_if_needed_recursive api causes invalidate request via ChromeClient, it can be discarded without rendering as long as it would be called after ewk_view_repaints_get api. Wow, Good point. It solved many rendering issues in single view. > > So, It is better to be move into other position, but I couldn't make sure whether it's really good when it must be handled whenever ewk_view_layout_if_needed_recursive api is called. I moved related codes to below of ewk_view_layout_if_needed_recursive. It looks quite better.
Lucas De Marchi
Comment 2 2011-10-26 16:40:57 PDT
(In reply to comment #1) > (In reply to comment #0) > > As ewk_view_layout_if_needed_recursive api causes invalidate request via ChromeClient, it can be discarded without rendering as long as it would be called after ewk_view_repaints_get api. > > Wow, Good point. > It solved many rendering issues in single view. > > > > > So, It is better to be move into other position, but I couldn't make sure whether it's really good when it must be handled whenever ewk_view_layout_if_needed_recursive api is called. > > I moved related codes to below of ewk_view_layout_if_needed_recursive. > It looks quite better. There's no patch attached.
KwangHyuk
Comment 3 2011-10-26 19:08:43 PDT
> There's no patch attached. Let me attach it soon. :)
KwangHyuk
Comment 4 2011-10-26 22:25:44 PDT
Eunsol will take care of this. :)
Ryuan Choi
Comment 5 2011-10-27 02:29:26 PDT
I found that changseok raised same issue and created patch. So, I close. *** This bug has been marked as a duplicate of bug 70170 ***
Note You need to log in before you can comment on or make changes to this bug.