We should start using OwnPtr internally for non-ref counted Ewk types.
Created attachment 168968 [details] Patch
Created attachment 169082 [details] Patch Rebase on master.
Comment on attachment 169082 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=169082&action=review > Source/WebKit2/UIProcess/API/efl/ewk_cookie_manager_private.h:51 > typedef struct _Ewk_Cookie_Manager Ewk_Cookie_Manager; Why do we do these C style things?
(In reply to comment #3) > (From update of attachment 169082 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=169082&action=review > > > Source/WebKit2/UIProcess/API/efl/ewk_cookie_manager_private.h:51 > > typedef struct _Ewk_Cookie_Manager Ewk_Cookie_Manager; > > Why do we do these C style things? Think we can do the same as it's done in WK C API: typedef struct WKPageLoaderClient WKPageLoaderClient; That would let us to remove typedef from private header at all.
(In reply to comment #4) > (In reply to comment #3) > > (From update of attachment 169082 [details] [details]) > > View in context: https://bugs.webkit.org/attachment.cgi?id=169082&action=review > > > > > Source/WebKit2/UIProcess/API/efl/ewk_cookie_manager_private.h:51 > > > typedef struct _Ewk_Cookie_Manager Ewk_Cookie_Manager; > > > > Why do we do these C style things? > > Think we can do the same as it's done in WK C API: > > typedef struct WKPageLoaderClient WKPageLoaderClient; > > That would let us to remove typedef from private header at all. I mean use typedef struct Ewk_Cookie_Manager Ewk_Cookie_Manager; in public header.
Comment on attachment 169082 [details] Patch Clearing flags on attachment: 169082 Committed r131558: <http://trac.webkit.org/changeset/131558>
All reviewed patches have been landed. Closing bug.