1406061 - remove devtools/client/shared/Jsbeautify.jsm

Status: RESOLVED FIXED

(DevTools :: Framework, enhancement, P3)

Description

[Tom Tromey :tromey]

devtools/client/shared/Jsbeautify.jsm seems to be a lingering shim,
not used from devtools any more.  I think it can be removed.

Comment 1

[Tom Tromey :tromey]

The css part of jsbeautify doesn't seem to be used.
And, since we have pretty-fast, maybe all of jsbeautify could be removed.

Comment 2

[Patrick Brosset <:pbro>]

Attachment: Bug 1406061 - Remove the unused Jsbeautify.jsm module;

Review commit: https://reviewboard.mozilla.org/r/191036/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/191036/

Comment 3

[Patrick Brosset <:pbro>]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=7601a2c6f9e2a5934bcf748d1ecec4a04cfbb887&group_state=expanded

Comment 4

[Julian Descottes [:jdescottes]]

Comment on attachment 8920045 [details]
Bug 1406061 - Remove the unused Jsbeautify.jsm module;

https://reviewboard.mozilla.org/r/191036/#review196250

LGTM, but I think this should be deleted too:
http://searchfox.org/mozilla-central/rev/d0448c970093f94bd986a21d3a9e8c366b316eb4/tools/lint/eslint/modules.json#113

Comment 5

[Patrick Brosset <:pbro>]

Comment on attachment 8920045 [details]
Bug 1406061 - Remove the unused Jsbeautify.jsm module;

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/191036/diff/1-2/

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/fc6cfc87908b
Remove the unused Jsbeautify.jsm module; r=jdescottes

Comment 7

[Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)]

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