Expose RTT and downlink bandwidth using experimental Javascript API

third_party/WebKit/Source/platform/exported/WebNetworkStateNotifier.cpp

Index: third_party/WebKit/Source/platform/exported/WebNetworkStateNotifier.cpp
diff --git a/third_party/WebKit/Source/platform/exported/WebNetworkStateNotifier.cpp b/third_party/WebKit/Source/platform/exported/WebNetworkStateNotifier.cpp
index c979c0f3d48ebb45a192b12e3c1a585897d1bc24..d910883fbf1bd2eb68354b05ad3de935c26e1b5f 100644
--- a/third_party/WebKit/Source/platform/exported/WebNetworkStateNotifier.cpp
+++ b/third_party/WebKit/Source/platform/exported/WebNetworkStateNotifier.cpp
@@ -43,4 +43,11 @@ void WebNetworkStateNotifier::SetWebConnection(WebConnectionType type,
   GetNetworkStateNotifier().SetWebConnection(type, max_bandwidth_mbps);
 }
 
+void WebNetworkStateNotifier::SetNetworkQuality(TimeDelta http_rtt,

[dcheng, 2017/05/19 19:19:18]

The header declares this as base::TimeDelta while this is WTF::TimeDelta. While
this happens to work today because it's the same type, doing this is depending
on internal implementation details of WTF:TimeDelta.

I guess it's OK in this case since we plan on allowing subsets of base in Blink
soon though?

[tbansal1, 2017/05/19 19:28:59]

I am not sure. The alternative is to pass the base::TimeDelta as int32_t, and
convert to WTF::TimeDelta. Please let me know.

[dcheng, 2017/05/19 19:40:14]

Sorry to be clear, this question is aimed for haraken

+                                                TimeDelta transport_rtt,
+                                                int downlink_throughput_kbps) {
+  GetNetworkStateNotifier().SetNetworkQuality(http_rtt, transport_rtt,
+                                              downlink_throughput_kbps);
+}
+
 }  // namespace blink