[Webkit-unassigned] [Bug 55026] [EFL] Doxygen documentation for ewk_window_features and ewk_context_menu

[bugzilla-daemon at webkit.org]

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





--- Comment #2 from Gyuyoung Kim <gyuyoung.kim at samsung.com>  2011-02-23 15:21:49 PST ---
(From update of attachment 83461)
View in context: https://bugs.webkit.org/attachment.cgi?id=83461&action=review

> Source/WebKit/efl/ChangeLog:5
> +        [EF] Doxygen documentation for ewk_window_features and ewk_context_menu

[EF] => [EFL]

> Source/WebKit/efl/ewk/ewk_contextmenu.cpp:57
> +    Ewk_Context_Menu* submenu; /** contains the pointer to the submenu of the item */

Please add "<" to "/**"

> Source/WebKit/efl/ewk/ewk_contextmenu.h:79
> +    EWK_CONTEXT_MENU_ITEM_TAG_SPELLING_MENU, /**< spelling or spelling/grammar sub-menu */

This is not sync with ewk_cookie.cpp. Which one is better?  "/**<" or "///"

> Source/WebKit/efl/ewk/ewk_contextmenu.h:84
> +    EWK_CONTEXT_MENU_ITEM_TAG_FONT_MENU, /**< font sub-menu */

This is not sync with ewk_cookie.cpp. Which one is better?  "/**<" or "///"

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