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 variabl
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:
nasko
2017/05/08 18:27:45
I'd rather not use magic values without defining i
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); |