Summary: | [EFL] [WK2] Memory leak in ewk_view_resource_load_client.cpp | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Sudarsana Nagineni (babu) <naginenis> | ||||||
Component: | WebKit EFL | Assignee: | Sudarsana Nagineni (babu) <naginenis> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | cdumez, gyuyoung.kim, kenneth, lucas.de.marchi, mrobinson, webkit.review.bot | ||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Attachments: |
|
Description
Sudarsana Nagineni (babu)
2012-07-24 04:40:33 PDT
Created attachment 154028 [details]
Patch
Comment on attachment 154028 [details]
Patch
LGTM. Thanks for fixing this.
Comment on attachment 154028 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=154028&action=review > Source/WebKit2/UIProcess/API/efl/ewk_view.cpp:297 > + for ( ; it != end; ++it) If possible, why don't you modify this one as below ? for (LoadingResourcesMap::iterator it = loadingResourcesMap.begin(); it != end; ++it) In addition, we can save a line. Comment on attachment 154028 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=154028&action=review >> Source/WebKit2/UIProcess/API/efl/ewk_view.cpp:297 >> + for ( ; it != end; ++it) > > If possible, why don't you modify this one as below ? > > for (LoadingResourcesMap::iterator it = loadingResourcesMap.begin(); it != end; ++it) > > In addition, we can save a line. This is really a matter of preference. There is no coding style rule that says we should do it either way and AFAIK there is no performance impact. (In reply to comment #4) > This is really a matter of preference. There is no coding style rule that says we should do it either way and AFAIK there is no performance impact. This is just my suggestion because it seems to me above code is more used in WebKit inside. Of course, there is no performance gain. (In reply to comment #5) > This is just my suggestion because it seems to me above code is more used in WebKit inside. Of course, there is no performance gain. Can I leave this as it is since there is no any gain in changing it? (In reply to comment #6) > (In reply to comment #5) > > This is just my suggestion because it seems to me above code is more used in WebKit inside. Of course, there is no performance gain. > > Can I leave this as it is since there is no any gain in changing it? Ok, I don't wanna disturb this patch is landed. LGTM as well. Created attachment 154581 [details]
Patch
Rebased
Comment on attachment 154581 [details]
Patch
Looks OK
Comment on attachment 154581 [details] Patch Clearing flags on attachment: 154581 Committed r123727: <http://trac.webkit.org/changeset/123727> All reviewed patches have been landed. Closing bug. |