Remove unused gBrowser.tabContextMenu property

RESOLVED FIXED in Firefox 60

Status

()

enhancement
P3
normal
RESOLVED FIXED
a year ago
a year ago

People

(Reporter: dao, Assigned: boghdady5, Mentored)

Tracking

({good-first-bug})

Trunk
Firefox 60
Points:
---

Firefox Tracking Flags

(firefox60 fixed)

Details

(Whiteboard: [lang=js])

Attachments

(1 attachment, 1 obsolete attachment)

Assignee

Comment 1

a year ago
Hi, I'm new here, can I get assigned this bug?? Thanks
Reporter

Comment 2

a year ago
(In reply to Omar from comment #1)
> Hi, I'm new here, can I get assigned this bug?? Thanks

Sure, and please let me know if you have questions.
Assignee: nobody → boghdady5
Status: NEW → ASSIGNED
Assignee

Comment 3

a year ago
Is it enough to just delete the property? (Because I already have a patch ready for that) or do I need to delete the attribute context menu from tab container too?
Reporter

Comment 4

a year ago
(In reply to Omar from comment #3)
> Is it enough to just delete the property? (Because I already have a patch
> ready for that) or do I need to delete the attribute context menu from tab
> container too?

Just the gBrowser property.
Assignee

Comment 5

a year ago
Alright I removed it I just need to figure out the patch submitting process I'll have it posted up in the next couple of hours
Assignee

Comment 6

a year ago
Posted patch A patch for bug #1443833 (obsolete) — Splinter Review
Attachment #8957376 - Flags: review?(dao+bmo)
Reporter

Comment 7

a year ago
Comment on attachment 8957376 [details] [diff] [review]
A patch for bug #1443833

>   get visibleTabs() {
>     if (!this._visibleTabs)
>       this._visibleTabs = Array.filter(this.tabs,
>         tab => !tab.hidden && !tab.closing);
>     return this._visibleTabs;
>-  },

Looks like you accidentally removed this line. Can you please submit a new patch without this? Thanks!
Attachment #8957376 - Flags: review?(dao+bmo)
Assignee

Comment 8

a year ago
I'm terribly sorry for this mistake I'll fix it right away
Assignee

Comment 9

a year ago
Attachment #8957376 - Attachment is obsolete: true
Attachment #8957558 - Flags: review?(dao+bmo)
Reporter

Comment 10

a year ago
Comment on attachment 8957558 [details] [diff] [review]
removed getContextMenu without the extra bracket removed

Looks good, thanks!
Attachment #8957558 - Flags: review?(dao+bmo) → review+

Comment 11

a year ago
Pushed by dgottwald@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/696b0f84b342
Remove unused gBrowser.tabContextMenu property. r=dao

Comment 12

a year ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/696b0f84b342
Status: ASSIGNED → RESOLVED
Last Resolved: a year ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 60
You need to log in before you can comment on or make changes to this bug.