[Webkit-unassigned] [Bug 54352] [GTK] Match more various WebKit API enum values with WebCore enum values

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Feb 15 08:53:03 PST 2011


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





--- Comment #5 from Martin Robinson <mrobinson at webkit.org>  2011-02-15 08:53:03 PST ---
(In reply to comment #4)
> If we keep the ABI, the patch should be updated as follows,
> But, I'm not sure if this is neat.
> 
> COMPILE_ASSERT_MATCHING_ENUM(WEBKIT_WEB_VIEW_VIEW_MODE_WINDOWED+1, Page::ViewModeWindowed);
> COMPILE_ASSERT_MATCHING_ENUM(WEBKIT_WEB_VIEW_VIEW_MODE_FLOATING+1, Page::ViewModeFloating);
> COMPILE_ASSERT_MATCHING_ENUM(WEBKIT_WEB_VIEW_VIEW_MODE_FULLSCREEN+1, Page::ViewModeFullscreen);
> COMPILE_ASSERT_MATCHING_ENUM(WEBKIT_WEB_VIEW_VIEW_MODE_MAXIMIZED+1, Page::ViewModeMaximized);
> COMPILE_ASSERT_MATCHING_ENUM(WEBKIT_WEB_VIEW_VIEW_MODE_MINIMIZED+1, Page::ViewModeMinimized);

Essentially if there is no place where you cast directly between the two enum types, they do not need to be in AssertMatchingEnums.cpp. AssertMatchingEnums.cpp just ensures that a cast between the two types is valid.

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