Comment on attachment 8812476 [details] [diff] [review] Remove the NetworkStats API and related code Review of attachment 8812476 [details] [diff] [review]: ----------------------------------------------------------------- oh... I really like this API. Wondering if it makes sense to keep it for some other purpose, addons? webExtensions? a network about page? But, a part of this, the patch seems fine to me.

At least part of the implementation needs to change if we want to keep this around since this mostly accounts for per-app usage. I'm not aware of any concrete use cases, but at any rate the code is in the history if someone needs to dig it up again.

Comment on attachment 8812476 [details] [diff] [review] Remove the NetworkStats API and related code Review of attachment 8812476 [details] [diff] [review]: ----------------------------------------------------------------- I kind of like this code too but appreciate that it isn't right anymore. If you want to toss a comment into somewhere like nsHttpTransaction::WriteRequestSegment() with this bug # in case someone needs to recreate the byte counters that might be helpful.

Sure, will do!

Pushed by eakhgari@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/9ef303d80d12 Remove the NetworkStats API and related code; r=baku,mcmanus