[Webkit-unassigned] [Bug 84572] [EFL] Split ewk_private.h file to multiple private files.
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Tue May 15 00:47:08 PDT 2012
https://bugs.webkit.org/show_bug.cgi?id=84572
--- Comment #18 from Gyuyoung Kim <gyuyoung.kim at samsung.com> 2012-05-15 00:46:12 PST ---
(From update of attachment 141877)
View in context: https://bugs.webkit.org/attachment.cgi?id=141877&action=review
There are many style nits in this patch. This style nits weren't fixed before. Can you fix style nits within no big changes?
http://trac.webkit.org/wiki/EFLWebKitCodingStyle
> Source/WebKit/efl/ewk/ewk_auth_soup_private.h:36
> + GObject parent_instance;
Do not use _ in variable name.
> Source/WebKit/efl/ewk/ewk_auth_soup_private.h:43
> +GType ewk_auth_soup_dialog_get_type(void);
WebKit EFL adheres webkit coding style and C++ coding style except for public APIs. So, I think you need to remove void keyword in parameter.
> Source/WebKit/efl/ewk/ewk_auth_soup_private.h:50
> +void ewk_auth_soup_credentials_set(const char *username, const char *password, void *data);
Move '*' to data type side.
> Source/WebKit/efl/ewk/ewk_frame_private.h:82
> +WebCore::Frame *coreFrame(const Evas_Object *ewkFrame);
ditto.
> Source/WebKit/efl/ewk/ewk_history_private.h:31
> +Ewk_History_Item *ewk_history_item_new_from_core(WebCore::HistoryItem *core);
ditto.
> Source/WebKit/efl/ewk/ewk_history_private.h:36
> +WebCore::HistoryItem *coreHistoryItem(const Ewk_History_Item *ewkHistoryItem);
ditto.
> Source/WebKit/efl/ewk/ewk_tiled_backing_store_private.h:63
> + Eina_Tiler *updates; /**< updated/dirty areas */
ditto.
> Source/WebKit/efl/ewk/ewk_tiled_backing_store_private.h:77
> + Evas_Object *image; /**< Evas Image, the tile to be rendered */
ditto.
--
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