[Webkit-unassigned] [Bug 129740] [EFL] Add WK2 API to control TLS error policy on WebContext.

[bugzilla-daemon at webkit.org]

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


Gyuyoung Kim <gyuyoung.kim at samsung.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #225884|review?                     |review-
               Flag|                            |




--- Comment #4 from Gyuyoung Kim <gyuyoung.kim at samsung.com>  2014-03-05 16:52:51 PST ---
(From update of attachment 225884)
View in context: https://bugs.webkit.org/attachment.cgi?id=225884&action=review

> Source/WebKit2/UIProcess/API/efl/ewk_context.h:361
> +EAPI void ewk_context_set_ignore_tls(Ewk_Context* ewkContext, Eina_Bool ignoreTLSErrors);

1. Wrong * place and wrong efl function style. Ewk_Context* ewkContext => Ewk_Context *ewk_context, ignoreTLSErrors => ignore_tls_errors.

2. Generally EFL function naming has placed *verb* at the end of function. For instance, ewk_context_ignore_tls_set(...)

> Source/WebKit2/UIProcess/API/efl/tests/UnitTestUtils/EWK2UnitTestServer.cpp:24
> +EWK2UnitTestServer::EWK2UnitTestServer(GTlsCertificate* tls_cert)

We only use EFL coding style in public EFL APIs. So, tls_cert => tlsCert

> Source/WebKit2/UIProcess/API/efl/tests/UnitTestUtils/EWK2UnitTestServer.h:29
> +    EWK2UnitTestServer(GTlsCertificate* tls_cert = 0);

ditto

> Source/WebKit2/UIProcess/API/efl/tests/test_ewk2_ssl.cpp:41
> +static bool finish_test = false;

ditto.

> Source/WebKit2/UIProcess/API/efl/tests/test_ewk2_ssl.cpp:152
> +    Ewk_Error* error = 0;

0 => nullptr ?

> Source/WebKit2/UIProcess/API/efl/tests/test_ewk2_ssl.cpp:178
> +    Ewk_Error* error = 0;

ditto.

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