Closed Bug 1297569 Opened 3 years ago Closed 3 years ago

Disable XUL popup contextmenu handler for all remote targets

Categories

(Core :: XUL, defect, P1)

defect

Tracking

()

RESOLVED FIXED
mozilla51
Iteration:
51.2 - Aug 29
Tracking Status
firefox51 --- fixed

People

(Reporter: jryans, Assigned: jryans)

References

Details

(Whiteboard: [multiviewport][reserve-rdm])

Attachments

(1 file)

In bug 516753, nsXULPopupListener.cpp was modified to abort contextmenu event handling if the event target is content inside a remote xul:browser.

On the DevTools team, we are working on a new Responsive Design Mode tool which uses an <iframe mozbrowser remote> element to display content.

Currently, this XUL popup listener is grabbing contextmenu events before the remote content can process them.
Attachment #8784201 - Flags: review?(felipc)
Comment on attachment 8784201 [details]
Bug 1297569 - Disable XUL popup contextmenu handler for all remote targets.

https://reviewboard.mozilla.org/r/73742/#review72234
Attachment #8784201 - Flags: review?(felipc) → review+
Blocks: 1292133
Pushed by jryans@gmail.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/b1c1291f1f67
Disable XUL popup contextmenu handler for all remote targets. r=felipe
Iteration: --- → 51.2 - Aug 29
Flags: qe-verify-
Priority: -- → P1
Whiteboard: [multiviewport][reserve-rdm]
https://hg.mozilla.org/mozilla-central/rev/b1c1291f1f67
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla51
You need to log in before you can comment on or make changes to this bug.