1022214 - Remove the obsolete mTabstripCloseButton property

Status: VERIFIED FIXED

(Firefox :: Tabbed Browser, defect)

Description

[Luís Miguel [:quicksaver] (not reading bugmail)]

Property .mTabstripCloseButton in #tabbrowser-tabs always returns null now. http://mxr.mozilla.org/mozilla-central/source/browser/base/content/tabbrowser.xml#41541

Comment 1

[Shashank VRSN Sabniveesu]

(In reply to Luís Miguel [:Quicksaver] from comment #0)
> Property .mTabstripCloseButton in #tabbrowser-tabs always returns null now.
> http://mxr.mozilla.org/mozilla-central/source/browser/base/content/
> tabbrowser.xml#41541

Are you referring to http://mxr.mozilla.org/mozilla-central/source/browser/base/content/tabbrowser.xml#4160 ?

Is it just enough to delete this property or the next one too?

Comment 2

[Luís Miguel [:quicksaver] (not reading bugmail)]

Yes it's that, I'm sorry about that, must have been a copy-paste mishap, didn't even notice it.

mTabStripClosebutton returns a node with id "tabs-closebutton" which was removed by bug 865826. A search in central for "tabs-closebutton" and "mTabStripClosebutton" return only that single line. So that property will return null at all times when called, which it never is apparently.

Perhaps I'm missing a specific case as to why this variable was kept (needinfo'ing Dao on this as he was assigned that bug and will probably know the reason if there is any), but I don't see how this helps/hurts with any backwards compatibility issues, whether it is in-house or for add-ons.

I don't know exactly what the use of mAllTabsPopup is/was, so I can't answer that. But I'm sure that is not the same case as mTabStripCloseButton, as the node it refers to still exists at least (from the name I think it's the popup menu that opens when you click the all tabs button, haven't investigated further than that though).

Comment 3

[Dão Gottwald [:dao]]

(In reply to Shashank VRSN Sabniveesu from comment #1)
> (In reply to Luís Miguel [:Quicksaver] from comment #0)
> > Property .mTabstripCloseButton in #tabbrowser-tabs always returns null now.
> > http://mxr.mozilla.org/mozilla-central/source/browser/base/content/
> > tabbrowser.xml#41541
> 
> Are you referring to
> http://mxr.mozilla.org/mozilla-central/source/browser/base/content/
> tabbrowser.xml#4160 ?
> 
> Is it just enough to delete this property or the next one too?

Only the mTabstripClosebutton property.

Comment 4

[Shashank VRSN Sabniveesu]

Attachment: BUG1022214.patch Removed the relevant 'property' element

Comment 5

[Dão Gottwald [:dao]]

Comment on attachment 8437983 [details] [diff] [review]
BUG1022214.patch Removed the relevant 'property' element

Thanks!

Comment 6

[Dão Gottwald [:dao]]

https://hg.mozilla.org/integration/fx-team/rev/20f9b816ebcd

Comment 7

[Ed Morley [:emorley]]

https://hg.mozilla.org/mozilla-central/rev/20f9b816ebcd