Bug 834033 landed a WebSocket library to support SimplePush. Snorp says this is never going to be used for Push support, so let's remove it. Of the four cset's landed in bug 834033: https://hg.mozilla.org/mozilla-central/rev/0bf3505c02b2 https://hg.mozilla.org/mozilla-central/rev/05a6210472f1 https://hg.mozilla.org/mozilla-central/rev/c1779b8d2bf1 https://hg.mozilla.org/mozilla-central/rev/a597da48b937 I think the only one we need to back out for WebSockets is: https://hg.mozilla.org/mozilla-central/rev/c1779b8d2bf1 This cset does package more in our APK, but I don't know if we want the SystemMessage stuff or not. Maybe just remove the Push.js stuff, but DO NOT remove the necko_websocket.xpt stuff: https://hg.mozilla.org/mozilla-central/rev/05a6210472f1

I will probably end up doing this.

Per Bug 834033 Comment 75.

Assuming that this isn't used by anything else. It's a DOM API, and I don't think we care.

Same. I took a look at this code, and it doesn't seem at all relevant to Fennec.

Because why specify dependencies in one place when two will do?

Try is happy.

Comment on attachment 8486492 [details] [diff] [review] Part 2: remove Push.js. v1 Review of attachment 8486492 [details] [diff] [review]: ----------------------------------------------------------------- We'll need this (or something similar) eventually, I think. Can take it out for now.

Comment on attachment 8486494 [details] [diff] [review] Part 3: remove SystemMessage stuff. v1 Review of attachment 8486494 [details] [diff] [review]: ----------------------------------------------------------------- I don't even know what this is, but looks B2G related?

https://hg.mozilla.org/integration/fx-team/rev/cbcf1d354938 https://hg.mozilla.org/integration/fx-team/rev/2c4ae93e8784 https://hg.mozilla.org/integration/fx-team/rev/341ae3a743a5

https://hg.mozilla.org/mozilla-central/rev/cbcf1d354938 https://hg.mozilla.org/mozilla-central/rev/2c4ae93e8784 https://hg.mozilla.org/mozilla-central/rev/341ae3a743a5