1447956 - AsyncTabSwitcher.updateDisplay should use this.tabbrowser.tabContainer instead of this.tabbrowser.tabbox.tabs

Status: RESOLVED FIXED

(Firefox :: Tabbed Browser, enhancement, P3)

Description

[Dão Gottwald [:dao]]

We should get rid of the tabs variable (it's only used once) and use this.tabbrowser.tabContainer instead:

https://searchfox.org/mozilla-central/rev/c217fbde244344fedfd07b57a740c694a456dbca/browser/modules/AsyncTabSwitcher.jsm#414,416

While we're at it, we should also rename the tabPanel variable to tabpanels.

Comment 1

[Kiran]

I just build firefox now.
I wanted to work on this bug as my first bug.


Following changes i will do in AsynchTabSwtcher.jsm file

      let tabs = this.tabbrowser.tabbox.tabs; //remove this line
      let tabPanel = this.tabbrowser.tabpanels;  //rename tabPanel to tabPanels
     let showPanel = tabs.getRelatedElement(showTab) ;  //this line will be modified to
     let showPanel = this.tabbrowser.tabbox.tabs.getRelatedElement(showTab)

Can I proceed?

Comment 2

[Dão Gottwald [:dao]]

(In reply to Kiran from comment #1)
> I just build firefox now.
> I wanted to work on this bug as my first bug.

Hi. :)

> Following changes i will do in AsynchTabSwtcher.jsm file
> 
>       let tabs = this.tabbrowser.tabbox.tabs; //remove this line
>       let tabPanel = this.tabbrowser.tabpanels;  //rename tabPanel to
> tabPanels

tabpanels (no camel case) rather than tabPanels, and make sure to also update the references to tabPanel.

>      let showPanel = tabs.getRelatedElement(showTab) ;  //this line will be
> modified to
>      let showPanel = this.tabbrowser.tabbox.tabs.getRelatedElement(showTab)

Should be this.tabbrowser.tabContainer rather than this.tabbrowser.tabbox.tabs

Comment 3

[Kiran]

Ok I will fix this

Comment 4

[Kiran]

Hey how can i generate patch for same? and send it to review

Comment 5

[Dão Gottwald [:dao]]

(In reply to Kiran from comment #4)
> Hey how can i generate patch for same? and send it to review

The simplest way is:

hg diff > ~/mypatch.diff

... and then upload the diff file on this page ("Attach File" link).

Comment 6

[Kiran]

Attachment: mypatch.diff

-Fixed Bug 1447956.

Comment 7

[Dão Gottwald [:dao]]

Comment on attachment 8962178 [details] [diff] [review]
mypatch.diff

Looks good, thanks!

Comment 8

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/76eccbfed981
Use this.tabbrowser.tabContainer instead of this.tabbrowser.tabbox.tabs. r=dao

Comment 9

[Kiran]

Thank you for all your help to make my first contribution easier..!!
Cheers.

Comment 10

[Cristian Brindusan [:cbrindusan]]

https://hg.mozilla.org/mozilla-central/rev/76eccbfed981

Comment 11

[Dão Gottwald [:dao]]

(In reply to Kiran from comment #9)
> Thank you for all your help to make my first contribution easier..!!
> Cheers.

You're welcome!