Bug 70900
| Summary: | [EFL] Invalidate request for the single view can be discarded without rendering. | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | KwangHyuk <hyuki.kim> |
| Component: | WebKit EFL | Assignee: | Nobody <webkit-unassigned> |
| Status: | RESOLVED DUPLICATE | ||
| Severity: | Normal | CC: | eunsol47.park, gyuyoung.kim, lucas.de.marchi, rakuco, ryuan.choi |
| Priority: | P2 | ||
| Version: | 528+ (Nightly build) | ||
| Hardware: | Other | ||
| OS: | Linux | ||
KwangHyuk
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
(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
(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
> There's no patch attached.
Let me attach it soon. :)
KwangHyuk
Eunsol will take care of this. :)
Ryuan Choi
I found that changseok raised same issue and created patch.
So, I close.
*** This bug has been marked as a duplicate of bug 70170 ***