Closed Bug 1397452 Opened 3 years ago Closed 3 years ago

Remove all SDK compatibility code from devtools

Categories

(DevTools :: General, enhancement, P3)

enhancement

Tracking

(firefox58 fixed)

RESOLVED FIXED
Firefox 58
Tracking Status
firefox58 --- fixed

People

(Reporter: ochameau, Assigned: ochameau)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

This isn't directly related to SDK, but to bootstrapped add-on which are going to be rejected as SDK add-ons.
We could also remove such code:
http://searchfox.org/mozilla-central/source/devtools/shared/protocol.js#1129-1170
I'm wondering if we have some other code like that, kept just for add-ons (not only sdk ones)??
About comment 1 and protocol.js compatiblity code, there is more context in bug 1295171.

Honza, What is the status of RDP monitor?
Is it a WebExtension? If not, does it still work? Does it still uses ActorClass and ActorClassWithSpec?

Otherwise you may have some input on dropping SDK addon supports mentioned in comment 0?
Should we wait FF58? Or just wait for complete SDK removal first?
Flags: needinfo?(odvarko)
(In reply to Alexandre Poirot [:ochameau] from comment #2)
> About comment 1 and protocol.js compatiblity code, there is more context in
> bug 1295171.
> 
> Honza, What is the status of RDP monitor?
> Is it a WebExtension? If not, does it still work? Does it still uses
It still implemented as old-school Add-on SDK based extension, so
not compatible with the latest Firefox

> ActorClass and ActorClassWithSpec?
Yes, it's using actor and dynamic actor registration API.

> Otherwise you may have some input on dropping SDK addon supports mentioned
> in comment 0?
> Should we wait FF58? Or just wait for complete SDK removal first?
The extension should be transformed into WebExtension and in the
current state it should not block the NO-SDK effort.

Also the importance of the RDPi decreases as we consider supporting CDP.

Honza
Flags: needinfo?(odvarko)
Severity: normal → enhancement
Priority: -- → P3
Assignee: nobody → poirot.alex
Comment on attachment 8911809 [details]
Bug 1397452 - Remove all SDK compatiblity code from DevTools.

https://reviewboard.mozilla.org/r/183236/#review188398

Thanks for cleaning up my code :)

::: devtools/shim/DevToolsShim.jsm:36
(Diff revision 2)
>   * existing mozilla-central code that still needs to interact with DevTools (such as
>   * web-extensions).
>   *
>   * DevToolsShim is a singleton that provides a set of helpers to interact with DevTools,
>   * that work whether the DevTools addon is installed or not. It can be used to start
> - * listening to events, register tools, themes. As soon as a DevTools addon is installed
> + * listening to events. As soon as a DevTools addon is installed the DevToolsShim willi

s/willi/will

::: devtools/shim/DevToolsShim.jsm:110
(Diff revision 2)
>     * - registerTheme
>     * - unregisterTheme

can be removed too
Attachment #8911809 - Flags: review?(jdescottes) → review+
Pushed by apoirot@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/0dfa545d2f02
Remove all SDK compatiblity code from DevTools. r=jdescottes
https://hg.mozilla.org/mozilla-central/rev/0dfa545d2f02
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 58
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.