Closed Bug 409415 Opened 17 years ago Closed 6 years ago

Common gtkmozembed event context interface

Categories

(Core Graveyard :: Embedding: GTK Widget, defect)

x86
Linux
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: romaxa, Unassigned)

References

Details

Attachments

(1 file)

I guess it would be better to add Embed context info interface into gtkmozembed API.

Probably it should be merged from such projects like Maemo, Epiphany, ... in one and will provide more less full info about event context.
Attached patch Proposed patch (deleted) — Splinter Review
Combination from old EmbedContextInfo and Epiphany EmbedEvent...
Blocks: 408238
This duplicates a lot of the nsIContextMenuListener/nsIContextMenuListener2 functionality. I know this interfaces are somewhat limited and the implementation is not complete. But can you explain the rationale to reimplement this at the gtkmozembed level? It doesn't seem to be gtk specific at all.
Oh, yes...
Then we have 2 ways to extend default nsIContextMenuListener*:

A) Attach to the nsIContextMenuListener, and implement additional features in OnShowContextMenu handler.
B) Implement additional features directly inside nsIContextMenuListener.


Product: Core → Core Graveyard
Embedding: GTK Widget isn't a thing, closing.
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: