[webkit-reviews] review canceled: [Bug 108794] [EFL][WK2] Stop using WebString in ewk_cookie_manager, ewk_form_submission_request and ewk_text_checker : [Attachment 186383] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Mon Feb 4 08:39:31 PST 2013


Christophe Dumez <dchris at gmail.com> has canceled Christophe Dumez
<dchris at gmail.com>'s request for review:
Bug 108794: [EFL][WK2] Stop using WebString in ewk_cookie_manager,
ewk_form_submission_request and ewk_text_checker
https://bugs.webkit.org/show_bug.cgi?id=108794

Attachment 186383: Patch
https://bugs.webkit.org/attachment.cgi?id=186383&action=review

------- Additional Comments from Christophe Dumez <dchris at gmail.com>
View in context: https://bugs.webkit.org/attachment.cgi?id=186383&action=review


>> Source/WebKit2/UIProcess/API/cpp/efl/WKEinaSharedString.h:49
>> +	const char* leakString();
> 
> think it should return Eina_Stringshare*

Yes, it is clearer. I'll update. BTW, we use const char* when we mean
Eina_Stringshare* in a lot of places. We should probably fix that.


More information about the webkit-reviews mailing list