55516 - Context menu listener implementation in webbrowser giving back wrong nodes

Status: VERIFIED FIXED

(Core Graveyard :: Embedding: APIs, defect, P3)

Description

[Adam Lock]

The implementation in web browser for listening and firing context menu events 
does not always pass the most useful node to the subscribed 
nsIContextMenuListener. Modify the code to pass back the most relevant node and 
document the new behaviour.

The firing mechanism also doesn't work if the chrome site does not implement 
nsIDocShellTreeOwner. Change this.

Comment 1

[Adam Lock]

Attachment: Patch to pass more useful node to context menu listener. Still need to fix docshelltreeowner issue

Comment 2

[Adam Lock]

Attachment: Updated context menu listener diff.

Comment 3

[Adam Lock]

a=blizzard. Fix checked into the trunk - embedding branch checkin pending

Comment 4

[Adam Lock]

Fixed checked into embedding branch

Comment 5

[Paul Wyskoczka]

Updating QA Contact

Comment 6

[Michael Dunn]

Correction: Changing QA contact for the Embed API bugs to David Epstein.

Comment 7

[David Epstein]

change qa contact to dsirnapalli

Comment 8

[dsirnapalli]

code checked in.verified.