[Webkit-unassigned] [Bug 90954] [EFL] Add const to the parameter of getters in ewk_security_origin

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Jul 11 04:04:22 PDT 2012


https://bugs.webkit.org/show_bug.cgi?id=90954





--- Comment #5 from Gyuyoung Kim <gyuyoung.kim at samsung.com>  2012-07-11 04:04:22 PST ---
(From update of attachment 151652)
View in context: https://bugs.webkit.org/attachment.cgi?id=151652&action=review

>>> Source/WebKit/efl/ewk/ewk_security_origin.cpp:139
>>> +    origin->protocol = eina_stringshare_add(coreOrigin->protocol().utf8().data());
>> 
>> I wonder if coreOrigin always has protocol.
> 
> If there is not protocol or host in coreOrigin, eina_stringshare_add returns NULL.

If so, don't you need to mention it in API description ? I think you need to mention this to ewk_security_origin.h.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.



More information about the webkit-unassigned mailing list