[Webkit-unassigned] [Bug 22675] wxWidgets bindings don't have context menu support

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Sat Dec 13 15:01:11 PST 2008


------- Comment #3 from mr.diggilin at gmail.com  2008-12-13 15:01 PDT -------
(In reply to comment #2)
> When I try this patch, I get an assert in
> ContextMenuItem::ContextMenuItem(ContextMenuItemType type, ContextMenuAction
> action, const String& title, ContextMenu* subMenu). 
> Here's the assert message:
> [Debug] 09:57:24: /BUILD/wxPython-src-
> assert "parentMenu != NULL" failed in wxMenuItemBase(): menuitem should have a
> menu
> It's possible that you built in release mode and thus didn't see this assert.
> To build debug, you can run `WebKitTools/Scripts/set-webkit-configuration --wx
> --debug`.
> In any case, I had to give an r- to this patch because an assert like that
> usually indicates that on at least one platform, the approach you're trying
> won't work. I think what we need to do is mimic the GTK and QT ports here, and
> have a PlatformMenuItemDescription structure that includes a native structure,
> but the actual wxMenuItem(s) will be created in the ContextMenu::appendItem
> call. The ContextMenuItem constructors themselves will just store and return
> the values passed to them inside the PlatformMenuItemDescription struct.
> Thanks for all your work on this!

The default constructor for a menu item is NULL, apparently this is a wx bug
that has been fixed in later versions:

Request that this be disregarded and patch reconsidered?

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

More information about the webkit-unassigned mailing list