1318883 - Remove the NetworkStats API and related code

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, defect)

Comment 1

[(no longer active)]

Attachment: Remove the NetworkStats API and related code

Comment 2

[Andrea Marchesini [:baku]]

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.

Comment 3

[(no longer active)]

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 4

[Patrick McManus [:mcmanus]]

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.

Comment 5

[(no longer active)]

Sure, will do!

Comment 6

[Pulsebot]

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

Comment 7

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/9ef303d80d12