We should make the *_ref() functions return the object in Ewk APIs to make it more convenient to use: e.g. the following: ewk_download_job_ref(ewkDownload); ewkContext->downloadJobs.add(downloadId, ewkDownload); is replaced by: ewkContext->downloadJobs.add(downloadId, ewk_download_job_ref(ewkDownload)); This is consistent with eina_stringshare_ref() API in EFL and g_object_ref() in glib.
Created attachment 163787 [details] Patch
Of course, I'll update the EFL coding guidelines when this lands.
Comment on attachment 163787 [details] Patch Looks good refactoring.
Comment on attachment 163787 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=163787&action=review > Source/WebKit2/ChangeLog:3 > + [EFL] Make _ref() functions return the object Nit: missing [WK2] prefix.
Created attachment 163797 [details] Patch for landing Could someone please cq+?
Comment on attachment 163797 [details] Patch for landing Clearing flags on attachment: 163797 Committed r128413: <http://trac.webkit.org/changeset/128413>
All reviewed patches have been landed. Closing bug.