Dão Gottwald [:dao] Assignee
	Description • 7 years ago

This code is already self-contained and the extra indentation due to this object being embedded in the TabBrowser class is just annoying. This should also save some resources, since currently every object returned by mTabProgressListener is a brand-new one that doesn't share anything with the other "instances."

	Dão Gottwald [:dao] Assignee
	Comment 1 • 7 years ago

https://treeherder.mozilla.org/#/jobs?repo=try&revision=b6c49013ef30da68bb6c1fbc9f91b5e0f21fb4d7

	Dão Gottwald [:dao] Assignee
	Comment 3 • 7 years ago

Reviewboard is trying to be too smart, completely messing up the diff view. Please view the plain diff instead:

https://hg.mozilla.org/try/rev/b6c49013ef30da68bb6c1fbc9f91b5e0f21fb4d7

Here's basically what I did:

1. replace this.mTabBrowser with gBrowser
2. replace this.mTabProgressListener with new TabProgressListener
3. move the code
4. fix up indentation

	Jared Wein [:jaws] (please needinfo? me)
	Comment 4 • 7 years ago
mozreview-review

Comment on attachment 8954729 [details]
Bug 1441816 - Move mTabProgressListener into its own class.

https://reviewboard.mozilla.org/r/223866/#review230016

Thank you for linking to the diff on tryserver. I admit I had looked at the reviewboard diff and de-prioritized the review due to how crazy it looked on reviewboard.

	Mozilla Autoland
	Comment 5 • 7 years ago

We're sorry, Autoland could not rebase your commits for you automatically. Please manually rebase your commits and try again.

hg error in cmd: hg rebase -s fc5a5ffed43bec87b239139d7249e514a4d9b5e7 -d e03272cbedc0: rebasing 449533:fc5a5ffed43b "Bug 1441816 - Move mTabProgressListener into its own class. r=jaws" (tip)
merging browser/base/content/tabbrowser.js
warning: conflicts while merging browser/base/content/tabbrowser.js! (edit, then use 'hg resolve --mark')
unresolved conflicts (see hg resolve, then hg rebase --continue)

	Pulsebot
	Comment 7 • 7 years ago

Pushed by dgottwald@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/c884294ea2dd
Move mTabProgressListener into its own class. r=jaws

	Tiberius Oros[:tiberius_oros]
	Comment 8 • 7 years ago
bugherder

https://hg.mozilla.org/mozilla-central/rev/c884294ea2dd