Closed Bug 1239008 Opened 4 years ago Closed 4 months ago
Remove unused Chrome
See Bug #1037992
This converts ChromeDebuggerActor and moves it to its own file. It also changes the signature because the first parameter (connection) wasn't used anywhere. Try push: https://treeherder.mozilla.org/#/jobs?repo=try&revision=e22637aab3e1
Attachment #8707063 - Flags: review?(jryans)
Attachment #8707063 - Flags: review?(jryans) → review+
backed out for test failures like https://treeherder.mozilla.org/logviewer.html#?job_id=6605434&repo=fx-team
seems also https://treeherder.mozilla.org/logviewer.html#?job_id=6605353&repo=fx-team is related to this failures
Comment on attachment 8707063 [details] [diff] [review] Bug1239008.patch Canceling out the r+ on this because the patch did actually make things fail.
Attachment #8707063 - Flags: review+
Severity: normal → enhancement
No longer blocks: 1289193
Summary: Change ChromeDebuggerActor to protocol.js → Remove unused ChromeDebuggerActor
Pushed by email@example.com: https://hg.mozilla.org/integration/autoland/rev/e5983cf83dce Remove unused ChromeDebuggerActor. r=jdescottes
You need to log in before you can comment on or make changes to this bug.