[Webkit-unassigned] [Bug 102853] [EFL][WK2] Add ewk_application_cache_manager APIs

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Dec 4 19:37:32 PST 2012


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





--- Comment #12 from Ryuan Choi <ryuan.choi at samsung.com>  2012-12-04 19:39:56 PST ---
(From update of attachment 177636)
View in context: https://bugs.webkit.org/attachment.cgi?id=177636&action=review

> Source/WebKit2/UIProcess/API/efl/tests/UnitTestUtils/EWK2UnitTestServer.h:34
> +    unsigned getPort() const;

strange, is port() correct for coding style?

> Source/WebKit2/UIProcess/API/efl/tests/test_ewk2_application_cache_manager.cpp:31
> +#include "ewk_security_origin_private.h"

private.h is required?

> Source/WebKit2/UIProcess/API/efl/tests/test_ewk2_application_cache_manager.cpp:33
> +#include <Ecore.h>

I am not sure whether we should add EWebKit2.h and Ecore.h
In other test cases, it was omitted because EWK2UnitTestBase.h include.

> Source/WebKit2/UIProcess/API/efl/tests/test_ewk2_application_cache_manager.cpp:62
> +const char* applicationCacheHtml =

Should it be globa?
If then, please add static.

And const char XXX[] = is prefered.

> Source/WebKit2/UIProcess/API/efl/tests/test_ewk2_application_cache_manager.cpp:126
> +        eina_strbuf_append_printf(body, applicationCacheHtml, 0);
> +        const size_t bodyLength = eina_strbuf_length_get(body);
> +        soup_message_body_append(message->response_body, SOUP_MEMORY_TAKE, eina_strbuf_string_steal(body), bodyLength);
> +    } else if (!strcmp(path, applicationCacheManifestFile)) {
> +        eina_strbuf_append_printf(body, applicationCacheManifest, 0);
> +        const size_t bodyLength = eina_strbuf_length_get(body);
> +        soup_message_body_append(message->response_body, SOUP_MEMORY_TAKE, eina_strbuf_string_steal(body), bodyLength);
> +    } else if (!strcmp(path, applicationCacheCssFile)) {
> +        eina_strbuf_append_printf(body, applicationCacheCss, 0);
> +        const size_t bodyLength = eina_strbuf_length_get(body);
> +        soup_message_body_append(message->response_body, SOUP_MEMORY_TAKE, eina_strbuf_string_steal(body), bodyLength);
> +    } else
> +        soup_message_set_status(message, SOUP_STATUS_NOT_FOUND);

How about reducing the duplication?

-- 
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