847593 - Remove tab-browser

Status: RESOLVED FIXED

(Add-on SDK Graveyard :: General, defect, P2)

Description

[Alexandre Poirot [:ochameau]]

tab-browser now ends up being a "medium level API" between various low level tabs helper and the high-level tabs API.
Keeping it, introduce complexity in document which API should be used for what,
and we should avoid having to maintain too much code in parallel.

Since the new layout, the module moved to deprecated folder, but thanks to the mapping files, we can still require it with require(tab-browser) so that it doesn't appear as being deprecated, except in documentation.

We should start adding a runtime deprecation message for addons using it.
But we have to modify tests that are still using it, and addon builder helper.

Comment 1

[Erik Vold [:erikvold] (please needinfo? me)]

can we just remove this module?

Comment 2

[Irakli Gozalishvili [:irakli] [:gozala] [@gozala]]

Yes let's remove it!

Comment 3

[Erik Vold [:erikvold] (please needinfo? me)]

Attachment: Pointer to Github pull request: https://github.com/mozilla/addon-sdk/pull/1328

Pointer to Github pull-request

Comment 4

[Irakli Gozalishvili [:irakli] [:gozala] [@gozala]]

Comment on attachment 8349650 [details]
Pointer to Github pull request: https://github.com/mozilla/addon-sdk/pull/1328

Oh yeah!!!

Comment 5

[[github robot]]

Commits pushed to master at https://github.com/mozilla/addon-sdk

https://github.com/mozilla/addon-sdk/commit/110622f01e75b65598e0e511dce061b37d60e645
Bug 847593 - Remove tab-browser

https://github.com/mozilla/addon-sdk/commit/b891c6e9bbf6862525b183b4f087576e368014e4
Merge pull request #1328 from erikvold/847593

Bug 847593 - Remove tab-browser r=@gozala