Expose changes in the network quality to the renderers

content/common/renderer.mojom

Index: content/common/renderer.mojom
diff --git a/content/common/renderer.mojom b/content/common/renderer.mojom
index 6c7fb6abb946757b3f8bc5cdb246c0a774c4e9c0..2a6c1f097ea46761a860ca62499da624954a9aa0 100644
--- a/content/common/renderer.mojom
+++ b/content/common/renderer.mojom
@@ -157,6 +157,15 @@ interface Renderer {
   OnNetworkConnectionChanged(NetworkConnectionType connection_type,
                              double max_bandwidth_mbps);
 
+  // Tells the renderer that the network quality has changed. |http_rtt|,

[nasko, 2017/05/08 18:27:45]

nit: s/renderer/renderer process/

[tbansal1, 2017/05/08 19:50:19]

Done.

+  // |transport_rtt| and |downstream_throughput_kbps| are the computed estimates
+  // of the HTTP RTT, transport RTT and downstream throughput (in kilobits per

[nasko, 2017/05/08 18:27:45]

nit: You can omit the actual names, as the variables are very much
self-describing. Do leave the units of those variables, as that is very
important bit.

[tbansal1, 2017/05/08 19:50:19]

Done.

+  // second), respectively. If an estimate of the HTTP or transport RTT is
+  // unavailable, it will be set to net::nqe::internal::InvalidRTT(). If the

[RyanSturm, 2017/05/08 17:37:20]

nit: Instead of/In addition to net::nqe::internal::InvalidRTT(), you might want
to say -1.

[nasko, 2017/05/08 18:27:45]

I'd rather not use magic values without defining it through a symbolic constant.
There is also base::Optional, though I'm not sure what the overhead would be for
a double parameter.

[tbansal1, 2017/05/08 19:50:19]

is it possible to IPC/mojoify base::Optional?

[tbansal1, 2017/05/08 19:50:19]

I do not think I want to hardcode the number here.

+  // throughput estimate is unavailable, it will be set to
+  // net::nqe::internal::kInvalidThroughput.
+  OnNetworkQualityChanged(double http_rtt_msec, double transport_rtt_msec, double bandwidth_kbps);
+
   // Tells the renderer to suspend/resume the webkit timers. Only for use on
   // Android.
   SetWebKitSharedTimersSuspended(bool suspend);