1543679 - Ensure all files under remote/ are referenced, or removed

Status: RESOLVED FIXED

(Remote Protocol :: Agent, defect, P1)

Description

[Andreas Tolfsen ❲:ato❳]

There are three files under remote/ which are never referenced,
causing browser/base/content/test/static/browser_all_files_referenced.js
to fail:

[task 2019-04-11T12:20:58.872Z] 12:20:58     INFO - TEST-UNEXPECTED-FAIL | browser/base/content/test/static/browser_all_files_referenced.js | there should be no unreferenced files - Got 3, expected 0
[task 2019-04-11T12:20:58.873Z] 12:20:58     INFO - Stack trace:
[task 2019-04-11T12:20:58.874Z] 12:20:58     INFO - chrome://mochikit/content/browser-test.js:test_is:1325
[task 2019-04-11T12:20:58.874Z] 12:20:58     INFO - chrome://mochitests/content/browser/browser/base/content/test/static/browser_all_files_referenced.js:checkAllTheFiles:790
[task 2019-04-11T12:20:58.875Z] 12:20:58     INFO - chrome://mochikit/content/browser-test.js:Tester_execTest/<:1116
[task 2019-04-11T12:20:58.875Z] 12:20:58     INFO - chrome://mochikit/content/browser-test.js:Tester_execTest:1144
[task 2019-04-11T12:20:58.876Z] 12:20:58     INFO - chrome://mochikit/content/browser-test.js:nextTest/<:1005
[task 2019-04-11T12:20:58.876Z] 12:20:58     INFO - chrome://mochikit/content/tests/SimpleTest/SimpleTest.js:SimpleTest.waitForFocus/waitForFocusInner/focusedOrLoaded/<:803
[task 2019-04-11T12:20:58.878Z] 12:20:58     INFO - Not taking screenshot here: see the one that was previously logged
[task 2019-04-11T12:20:58.878Z] 12:20:58     INFO - TEST-UNEXPECTED-FAIL | browser/base/content/test/static/browser_all_files_referenced.js | file only referenced from unreferenced files: chrome://remote/content/server/Packet.jsm referenced from chrome://remote/content/server/Transport.jsm - 
[task 2019-04-11T12:20:58.879Z] 12:20:58     INFO - Stack trace:
[task 2019-04-11T12:20:58.879Z] 12:20:58     INFO - chrome://mochikit/content/browser-test.js:test_ok:1314
[task 2019-04-11T12:20:58.879Z] 12:20:58     INFO - chrome://mochitests/content/browser/browser/base/content/test/static/browser_all_files_referenced.js:checkAllTheFiles:800
[task 2019-04-11T12:20:58.880Z] 12:20:58     INFO - chrome://mochikit/content/browser-test.js:Tester_execTest/<:1116
[task 2019-04-11T12:20:58.880Z] 12:20:58     INFO - chrome://mochikit/content/browser-test.js:Tester_execTest:1144
[task 2019-04-11T12:20:58.880Z] 12:20:58     INFO - chrome://mochikit/content/browser-test.js:nextTest/<:1005
[task 2019-04-11T12:20:58.880Z] 12:20:58     INFO - chrome://mochikit/content/tests/SimpleTest/SimpleTest.js:SimpleTest.waitForFocus/waitForFocusInner/focusedOrLoaded/<:803
[task 2019-04-11T12:20:58.883Z] 12:20:58     INFO - Not taking screenshot here: see the one that was previously logged
[task 2019-04-11T12:20:58.884Z] 12:20:58     INFO - TEST-UNEXPECTED-FAIL | browser/base/content/test/static/browser_all_files_referenced.js | unreferenced file: chrome://remote/content/server/Socket.jsm - 
[task 2019-04-11T12:20:58.885Z] 12:20:58     INFO - Stack trace:
[task 2019-04-11T12:20:58.886Z] 12:20:58     INFO - chrome://mochikit/content/browser-test.js:test_ok:1314
[task 2019-04-11T12:20:58.887Z] 12:20:58     INFO - chrome://mochitests/content/browser/browser/base/content/test/static/browser_all_files_referenced.js:checkAllTheFiles:794
[task 2019-04-11T12:20:58.887Z] 12:20:58     INFO - chrome://mochikit/content/browser-test.js:Tester_execTest/<:1116
[task 2019-04-11T12:20:58.888Z] 12:20:58     INFO - chrome://mochikit/content/browser-test.js:Tester_execTest:1144
[task 2019-04-11T12:20:58.889Z] 12:20:58     INFO - chrome://mochikit/content/browser-test.js:nextTest/<:1005
[task 2019-04-11T12:20:58.890Z] 12:20:58     INFO - chrome://mochikit/content/tests/SimpleTest/SimpleTest.js:SimpleTest.waitForFocus/waitForFocusInner/focusedOrLoaded/<:803
[task 2019-04-11T12:20:58.891Z] 12:20:58     INFO - Not taking screenshot here: see the one that was previously logged
[task 2019-04-11T12:20:58.892Z] 12:20:58     INFO - TEST-UNEXPECTED-FAIL | browser/base/content/test/static/browser_all_files_referenced.js | file only referenced from unreferenced files: chrome://remote/content/server/Transport.jsm referenced from chrome://remote/content/server/Socket.jsm - 

Comment 1

[Andreas Tolfsen ❲:ato❳]

Attachment: bug 1543679: remote: remove unused server files;

These files became redundant after we started serving WebSocket
connections from the HTTPD.

This fixes the test
browser/base/content/test/static/browser_all_files_referenced.js,
which checks for unreferenced JSMs.

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/04185cfeb1ae
remote: remove unused server files; r=ochameau,yulia

Comment 3

[Cristina Coroiu [:ccoroiu]]

https://hg.mozilla.org/mozilla-central/rev/04185cfeb1ae