1189645 - remove spdy specific telemetry and cwnd cookie

Status: RESOLVED FIXED

(Core :: Networking: HTTP, defect)

Description

[Patrick McManus [:mcmanus]]

as we start to transition to h2, remove telemetry that is spdy specific and remove the persistent cwnd settings that only google.com made use of (they are now on h2)

Comment 1

[Patrick McManus [:mcmanus]]

Attachment: patch 1

Comment 2

[u408661]

Comment on attachment 8641559 [details] [diff] [review]
patch 1

Review of attachment 8641559 [details] [diff] [review]:
-----------------------------------------------------------------

::: netwerk/protocol/http/SpdySession31.cpp
@@ +1430,5 @@
>      LOG3(("Settings ID %d, Flags %X, Value %d", id, flags, value));
>  
>      switch (id)
>      {
>      case SETTINGS_TYPE_UPLOAD_BW:

Get rid of this (and the rest of the now-empty cases)

::: toolkit/components/telemetry/Histograms.json
@@ +1508,5 @@
>      "kind": "exponential",
>      "high": "1000",
>      "n_buckets": 50,
>      "extended_statistics_ok": true,
> +    "description": "h2: Settings Initial Window (rounded to KB)"

nit: caps H2 like you did above (or lower the other, or not :P)

Comment 3

[Patrick McManus [:mcmanus]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/e293bbf8223c

Comment 4

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/e293bbf8223c