If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

Merge firefox-specific context-menu items "back" to tabbrowser.xml

RESOLVED FIXED in Firefox 3 beta1

Status

()

Firefox
General
P2
normal
RESOLVED FIXED
10 years ago
10 years ago

People

(Reporter: mano, Assigned: mano)

Tracking

({perf})

Trunk
Firefox 3 beta1
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

Merge firefox-specific context-menu items "back" to tabbrowser.xml.

Comment 1

10 years ago
This is about addBookmarkMenuitems and _addUndoCloseTabContextMenu in browser.js, right? Are you working on this? I just came across this and almost filed a bug, would be willing to post a patch.
Version: unspecified → Trunk
Created attachment 283226 [details] [diff] [review]
patch
Attachment #283226 - Flags: review?(mconnor)
Attachment #283226 - Flags: approval1.9?
Status: NEW → ASSIGNED
Keywords: perf
Priority: -- → P2
Target Milestone: --- → Firefox 3 M9
Comment on attachment 283226 [details] [diff] [review]
patch

yay to stupid cruft removal
Attachment #283226 - Flags: review?(mconnor) → review+

Updated

10 years ago
Attachment #283226 - Flags: approval1.9? → approval1.9+
mozilla/browser/base/content/browser.js 1.859
mozilla/browser/base/content/tabbrowser.xml 1.245
mozilla/browser/locales/en-US/chrome/browser/browser.properties 1.49
mozilla/browser/locales/en-US/chrome/browser/tabbrowser.dtd 1.5
Status: ASSIGNED → RESOLVED
Last Resolved: 10 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.