Closed
Bug 1391563
Opened 7 years ago
Closed 7 years ago
Check if exporting EventEmitter.emit from devtools/shared/protocol is still necessary
Categories
(DevTools :: General, enhancement, P1)
DevTools
General
Tracking
(firefox57 fixed)
RESOLVED
FIXED
Firefox 57
Tracking | Status | |
---|---|---|
firefox57 | --- | fixed |
People
(Reporter: jdescottes, Assigned: jdescottes)
References
Details
(Whiteboard: [reserve-nosdk])
Attachments
(1 file)
Follow up to Bug 1137935. We should check if exporting EventEmitter.emit is still necessary after migrating out of the sdk event emitter: http://searchfox.org/mozilla-central/rev/e8c36327cd8c9432c69e5e1383156a74330f11f2/devtools/shared/protocol.js#15
Comment hidden (mozreview-request) |
Assignee | ||
Comment 2•7 years ago
|
||
Simply removed the export and tested locally, seems to be fine. Try: https://treeherder.mozilla.org/#/jobs?repo=try&revision=96b19b96c5acd47ac2fb3aa4f98c4a37a7e63f67
Updated•7 years ago
|
Whiteboard: [nosdk]
Updated•7 years ago
|
Flags: qe-verify-
Priority: P3 → P1
Target Milestone: --- → Firefox 57
Updated•7 years ago
|
Whiteboard: [nosdk] → [reserve-nosdk]
Comment hidden (mozreview-request) |
Assignee | ||
Comment 4•7 years ago
|
||
Try seems to be ok with it. Just sent another push rebased on the latest central: https://treeherder.mozilla.org/#/jobs?repo=try&revision=8ff7f541d86bb3e980f62b6dd35a4d2b3eeb69e9
Comment 5•7 years ago
|
||
mozreview-review |
Comment on attachment 8898823 [details] Bug 1391563 - stop exporting event-emitter emit method from devtools protocol.js; https://reviewboard.mozilla.org/r/170198/#review176940
Attachment #8898823 -
Flags: review?(poirot.alex) → review+
Pushed by jdescottes@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/e2a683f3be4f stop exporting event-emitter emit method from devtools protocol.js;r=ochameau
Comment 7•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/e2a683f3be4f
Updated•6 years ago
|
Product: Firefox → DevTools
You need to log in
before you can comment on or make changes to this bug.
Description
•