Expose RTT and downlink bandwidth using experimental Javascript API

third_party/WebKit/Source/modules/netinfo/NetworkInformation.h

Index: third_party/WebKit/Source/modules/netinfo/NetworkInformation.h
diff --git a/third_party/WebKit/Source/modules/netinfo/NetworkInformation.h b/third_party/WebKit/Source/modules/netinfo/NetworkInformation.h
index 5ac714c2bed778ec5f2fd7dbd085b0ccbca6ff6f..846ac501b990f18804a526700702331e13777256 100644
--- a/third_party/WebKit/Source/modules/netinfo/NetworkInformation.h
+++ b/third_party/WebKit/Source/modules/netinfo/NetworkInformation.h
@@ -29,9 +29,15 @@ class NetworkInformation final
 
   String type() const;
   double downlinkMax() const;
+  double rtt() const;
+  double downlink() const;
 
   // NetworkStateObserver overrides.
-  void ConnectionChange(WebConnectionType, double downlink_max_mbps) override;
+  void ConnectionChange(WebConnectionType,
+                        double downlink_max_mbps,
+                        int http_rtt_msec,
+                        int transport_rtt_msec,
+                        double downlink_throughput_mbps) override;

[bengr, 2017/05/12 19:09:55]

Names are weird. Either do s/downlink_max_mbps/downlink_throughput_max_mbps/ or
s/downlink_throughput_mbps/downlink_mbps/.

[tbansal1, 2017/05/12 22:18:17]

Done.

 
   // EventTarget overrides.
   const AtomicString& InterfaceName() const override;
@@ -61,11 +67,11 @@ class NetworkInformation final
   void StartObserving();
   void StopObserving();
 
-  // Touched only on context thread.
+  // Variables below are touched only on context thread.
   WebConnectionType type_;
-
-  // Touched only on context thread.
   double downlink_max_mbps_;
+  int transport_rtt_msec_;
+  double downlink_throughput_mbps_;

[bengr, 2017/05/12 19:09:55]

Names are similarly weird.

[tbansal1, 2017/05/12 22:18:17]

Done.

 
   // Whether this object is listening for events from NetworkStateNotifier.
   bool observing_;