Ewk_Web_Error is just passed to application as a callback and it is released internally. So, we don't need to expose this as public API.
Agreed. Thanks.
Created attachment 167249 [details] Patch
Comment on attachment 167249 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=167249&action=review > Source/WebKit2/UIProcess/API/efl/ewk_web_error.h:-55 > - * Frees the given object. This doc should probably be moved to the cpp file with a @internal tag.
Created attachment 167253 [details] Patch
(In reply to comment #3) > (From update of attachment 167249 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=167249&action=review > > > Source/WebKit2/UIProcess/API/efl/ewk_web_error.h:-55 > > - * Frees the given object. > > This doc should probably be moved to the cpp file with a @internal tag. OK, moved.
Comment on attachment 167253 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=167253&action=review > Source/WebKit2/UIProcess/API/efl/ewk_web_error.cpp:72 > + * Frees the give object. "given"
Created attachment 167254 [details] Patch
(In reply to comment #6) > (From update of attachment 167253 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=167253&action=review > > > Source/WebKit2/UIProcess/API/efl/ewk_web_error.cpp:72 > > + * Frees the give object. > > "given" mistake. fixed. Thank you.
Comment on attachment 167254 [details] Patch LGTM. Thanks.
Comment on attachment 167254 [details] Patch Clearing flags on attachment: 167254 Committed r130470: <http://trac.webkit.org/changeset/130470>
All reviewed patches have been landed. Closing bug.