Expose changes in the network quality to the renderers

content/renderer/render_thread_impl.cc

Index: content/renderer/render_thread_impl.cc
diff --git a/content/renderer/render_thread_impl.cc b/content/renderer/render_thread_impl.cc
index 9dad0206afcd8f05445f0486fe1a066391a0249b..4257028ba1206bc8f08dc7287126dcc7d489db04 100644
--- a/content/renderer/render_thread_impl.cc
+++ b/content/renderer/render_thread_impl.cc
@@ -2129,6 +2129,15 @@ void RenderThreadImpl::OnNetworkConnectionChanged(
       NetConnectionTypeToWebConnectionType(type), max_bandwidth_mbps);
 }
 
+void RenderThreadImpl::OnNetworkQualityChanged(
+    double http_rtt_msec,
+    double transport_rtt_msec,
+    double downlink_throughput_kbps) {
+  UMA_HISTOGRAM_EXACT_LINEAR("NQE.RenderThreadNotified", 1, 2);
+  // TODO(tbansal): https://crbug.com/719108. Notify WebNetworkStateNotifier of
+  // the change in the network quality.

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

Is there a reason why this code is not included in this CL? In general, we
(security team) want to see the endpoint implementation of Mojo
interfaces/methods in the same CL as the one introducing the interface/method.

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

I can include that but I thought it would make the CL
too long and a bit unwieldy. Here is the complete
code path: https://codereview.chromium.org/2863973003/
(please disregard try failures. It needs to be rebased).

+}
+
 void RenderThreadImpl::SetWebKitSharedTimersSuspended(bool suspend) {
 #if defined(OS_ANDROID)
   if (suspend) {