Closed Bug 1401028 Opened 7 years ago Closed 7 years ago

Remove the content-contextmenu observer notification

Categories

(Firefox :: General, enhancement)

enhancement
Not set
normal

Tracking

()

RESOLVED FIXED
Firefox 57
Tracking Status
firefox57 --- fixed

People

(Reporter: ehsan.akhgari, Assigned: ehsan.akhgari)

Details

Attachments

(1 file)

This was used by the add-on SDK.
Attachment #8909524 - Flags: review?(kmaglione+bmo)
Comment on attachment 8909524 [details] [diff] [review]
Remove the content-contextmenu observer notification

Review of attachment 8909524 [details] [diff] [review]:
-----------------------------------------------------------------

::: browser/modules/ContextMenu.jsm
@@ -509,5 @@
>      if (defaultPrevented) {
>        return;
>      }
>  
>      let addonInfo = Object.create(null);

This isn't actually used anymore, either. We should probably remove it from here and nsContextMenu.js
Attachment #8909524 - Flags: review?(kmaglione+bmo) → review+
Pushed by eakhgari@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/a67df8254927
Remove the content-contextmenu observer notification; r=kmag
https://hg.mozilla.org/mozilla-central/rev/a67df8254927
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 57
Assignee: nobody → ehsan
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: