Closed Bug 1371065 Opened 7 years ago Closed 7 years ago

Remove the addon SDK from Firefox

Categories

(Add-on SDK Graveyard :: General, enhancement)

enhancement
Not set
normal

Tracking

(firefox57 fixed)

RESOLVED FIXED
mozilla57
Tracking Status
firefox57 --- fixed

People

(Reporter: andy+bugzilla, Assigned: kmag)

References

(Depends on 2 open bugs)

Details

(Keywords: meta)

Attachments

(3 files)

This bug is to track removing the SDK from Firefox. Once we've landed no SDK add-ons on release for Firefox as per:

https://wiki.mozilla.org/Add-ons/Firefox57

... we'll remove the SDK from Firefox. How we'd do this and impact on other platforms (eg. Thunderbird) will need to be discussed.
Depends on: 1375700
Blocks: 1377587
Blocks: 1308332
Depends on: 1386694
Depends on: 1389469
No longer blocks: 1377587
Assignee: nobody → kmaglione+bmo
Comment on attachment 8906200 [details]
Bug 1371065: Part 0 - Fix unit tests that use the SDK loader :/

https://reviewboard.mozilla.org/r/177964/#review183398
Attachment #8906200 - Flags: review?(dtownsend) → review+
Comment on attachment 8906202 [details]
Bug 1371065: Part 2 - rm -rf addon-sdk

https://reviewboard.mozilla.org/r/177968/#review183400
Attachment #8906202 - Flags: review?(dtownsend) → review+
Comment on attachment 8906201 [details]
Bug 1371065: Part 1 - Remove external references to the SDK.

https://reviewboard.mozilla.org/r/177966/#review183430
Attachment #8906201 - Flags: review?(dtownsend) → review+
The bug title got me confused, as I was thinking "but we removed the Firefox SDK in bug 1333826"!
Summary: Remove the SDK from Firefox → Remove the addon SDK from Firefox
Comment on attachment 8906201 [details]
Bug 1371065: Part 1 - Remove external references to the SDK.

https://reviewboard.mozilla.org/r/177966/#review183628

::: toolkit/modules/moz.build:157
(Diff revision 2)
>  XPCSHELL_TESTS_MANIFESTS += ['tests/xpcshell/xpcshell.ini']
>  BROWSER_CHROME_MANIFESTS += ['tests/browser/browser.ini']
>  MOCHITEST_CHROME_MANIFESTS += ['tests/chrome/chrome.ini']
>  
>  TESTING_JS_MODULES += [
> +    'HiddenFrame.jsm',

The HiddenFrame change seems like it would be more appropriate in a separate patch with a proper commit message (or even a followup bug).
Attachment #8906201 - Flags: review?(mh+mozilla) → review+
(In reply to Mike Hommey [:glandium] from comment #9)
> The bug title got me confused, as I was thinking "but we removed the Firefox
> SDK in bug 1333826"!

Heh. I'll admit, it confused me too. I actually couldn't even find the bug when I looked for it because there was no add-on/addon/jetpack in the title...
Comment on attachment 8906201 [details]
Bug 1371065: Part 1 - Remove external references to the SDK.

https://reviewboard.mozilla.org/r/177966/#review183628

> The HiddenFrame change seems like it would be more appropriate in a separate patch with a proper commit message (or even a followup bug).

Jetpack was the last non-test consumer of it, so without this change, browser_all_files_referenced failed. I thought about adding an exception instead, but not shipping unused code seemed better than shipping it. I guess I'll add an exception for now and move this change to a follow-up, though.
Blocks: 1399216
https://hg.mozilla.org/integration/mozilla-inbound/rev/2d0d78a70953c589f5c62f527f7f95881492d3a6
Bug 1371065: Follow-up: Fix emitter sandbox validation bustage. r=me CLOSED TREE
Blocks: 1399562
Depends on: 1400181
This observer still exists:
https://dxr.mozilla.org/mozilla-central/search?q=content-contextmenu&redirect=false

It was used by the addon SDK.
Flags: needinfo?(kmaglione+bmo)
(In reply to kernp25 from comment #16)
> This observer still exists:
> https://dxr.mozilla.org/mozilla-central/search?q=content-
> contextmenu&redirect=false
> 
> It was used by the addon SDK.

Filed bug 1401028 to remove it.
Flags: needinfo?(kmaglione+bmo)
Commit pushed to master at https://github.com/mozilla/normandy

https://github.com/mozilla/normandy/commit/b59561d4085489d7efb21391c8b30b3bb737a5b2
Sync changes from mozilla-central.

- Bug 1371065: Part 0 - Fix unit tests that use the SDK loader :/ r=Mossop
  https://hg.mozilla.org/mozilla-central/rev/46b8854f1f8e
- Bug 1398050 - Polish preferences page to match visual spec. r=mconley
  https://hg.mozilla.org/mozilla-central/rev/8918870e7a20
You need to log in before you can comment on or make changes to this bug.