Expose changes in the network quality to the renderers

content/renderer/render_thread_impl.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/renderer/render_thread_impl.h"
 
 #include <algorithm>
 #include <limits>
 #include <map>
 #include <utility>
@@ skipping 2111 matching lines @@
     net::NetworkChangeNotifier::ConnectionType type,
     double max_bandwidth_mbps) {
   bool online = type != net::NetworkChangeNotifier::CONNECTION_NONE;
   WebNetworkStateNotifier::SetOnLine(online);
   for (auto& observer : observers_)
     observer.NetworkStateChanged(online);
   WebNetworkStateNotifier::SetWebConnection(
       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) {
     renderer_scheduler_->SuspendTimerQueue();
   } else {
     renderer_scheduler_->ResumeTimerQueue();
   }
   webkit_shared_timer_suspended_ = suspend;
 #else
   NOTREACHED();
@@ skipping 299 matching lines @@
   }
 }
 
 void RenderThreadImpl::OnRendererInterfaceRequest(
     mojom::RendererAssociatedRequest request) {
   DCHECK(!renderer_binding_.is_bound());
   renderer_binding_.Bind(std::move(request));
 }
 
 }  // namespace content