1190452 - Remove dbg-server.jsm and use commonjs require to import DebuggerServer

Status: RESOLVED FIXED

(DevTools :: General, defect)

Description

[Alexandre Poirot [:ochameau]]

Let's make a dedicated bug to get rid of dbg-server.jsm.
This jsm is useless as we should import this module via
   const { DebuggerServer } = require("devtools/server/main");
(Instead of Cu.import("resource://gre/modules/devtools/dbg-server.jsm", {}))

All usages of dbg-server.jsm prevent correctly reloading server files when using local source directory.

Comment 1

[Alexandre Poirot [:ochameau]]

Attachment: patch v1

Let's see what try says:
  https://treeherder.mozilla.org/#/jobs?repo=try&revision=8843ced5ad39

Comment 2

[Alexandre Poirot [:ochameau]]

Attachment: patch v2

https://treeherder.mozilla.org/#/jobs?repo=try&revision=b7c7b7b7dd2c
This time, I'm not removing the jsm as it is used in too many addons.

Comment 3

[Alexandre Poirot [:ochameau]]

... but I've dispatched some previous try run with dbg-server.jsm removed to verify there was no more usages. Some new one may slit through, we would have to verify during reviews and may be a bit latter, check m-c codebase for possible new usages of it.

Comment 4

[J. Ryan Stinnett [:jryans] (Use needinfo, replies may be slow)]

Comment on attachment 8644911 [details] [diff] [review]
patch v2

Review of attachment 8644911 [details] [diff] [review]:
-----------------------------------------------------------------

Should dbg-server.jsm log a deprecation notice when it's used?

Comment 5

[Pulsebot]

https://hg.mozilla.org/integration/fx-team/rev/2fcd73fa8a7e

Comment 6

[Alexandre Poirot [:ochameau]]

(In reply to J. Ryan Stinnett [:jryans] (use ni?) from comment #4)
> Comment on attachment 8644911 [details] [diff] [review]
> patch v2
> 
> Review of attachment 8644911 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> Should dbg-server.jsm log a deprecation notice when it's used?

It's not a big deal if they keep using it or if we keep this alias.
But yes, we could do that. I would do that a bit later once file are moved, loader paths are cleaned up and everything is more stable as things might change again.

Comment 7

[Alexandre Poirot [:ochameau]]

Btw, I've pushed two try just before landing.
One removing dbg-server.jsm:
  https://treeherder.mozilla.org/#/jobs?repo=try&revision=9cc2dd728d33
And another without removing it (what actually landed):
  https://treeherder.mozilla.org/#/jobs?repo=try&revision=585e58baa292

Everything was green (enough)!

Comment 8

[[github robot]]

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

https://github.com/mozilla/addon-sdk/commit/a50a0c78b557af3994fa78bbc8d830a3c77b6620
Bug 1190452 - Always import toolkit/devtools/server/main.js as CommonJS module. r=jryans

Comment 9

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/2fcd73fa8a7e