1341420 - Stop loading devtools/client/webconsole/net/main

Status: RESOLVED DUPLICATE of bug 1326937

(DevTools :: Console, defect)

Description

[Tim Nguyen :ntim]

Only used by old console.

https://dxr.mozilla.org/mozilla-central/source/devtools/client/webconsole/webconsole.xul#32

Comment 1

[Tom Tromey :tromey]

I'm not 100% sure this is worth doing, considering that we hope to remove the old console
and that console.time/timeEnd logging shows that this file's load takes ~50ms.
However, I did it anyway to try to learn something about the console.

Comment 2

[Tom Tromey :tromey]

Attachment: Bug 1341420 - don't load main.js from webconsole.xul;

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

Comment 3

[Nicolas Chevobbe [:nchevobbe]]

Comment on attachment 8851046 [details]
Bug 1341420 - don't load main.js from webconsole.xul;

https://reviewboard.mozilla.org/r/123490/#review126206

This looks good for me. Let's see what TRY thinks about it

Comment 4

[Nicolas Chevobbe [:nchevobbe]]

Looks like TRY isn't happy with the patch : https://treeherder.mozilla.org/#/jobs?repo=try&revision=94d83cd6eb5131bca45bc756e1669da2528ca0ca
Mainly, it seems to causes issues with network messages.

Comment 5

[Nicolas Chevobbe [:nchevobbe]]

Comment on attachment 8851046 [details]
Bug 1341420 - don't load main.js from webconsole.xul;

https://reviewboard.mozilla.org/r/123492/#review130822

r- since TRY is failing

Comment 6

[Brian Grinstead [:bgrins]]

When loading in an HTML frame for the new console (Bug 1326937) we won't need to include this script since the new frontend doesn't use xhrspy (Bug 1362036 is opened to use some sort of shared component for requests with the netmonitor).  So going to dupe to Bug 1326937.