[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 23:03:25 PST 2012
https://bugs.webkit.org/show_bug.cgi?id=102853
--- Comment #15 from Jiyeon Kim <jiyeon0402.kim at samsung.com> 2012-12-04 23:05:49 PST ---
(In reply to comment #12)
> (From update of attachment 177636 [details])
> 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?
>
Yes, I'll fix.
> > Source/WebKit2/UIProcess/API/efl/tests/test_ewk2_application_cache_manager.cpp:31
> > +#include "ewk_security_origin_private.h"
>
> private.h is required?
>
I'll change ewk_security_origin.h to ewk_security_origin_private.h
> > 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.
>
I'll remove.
> > 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.
>
I'll fix.
> > 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?
I'll reduce 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