Notify NQE of TCP RTT values

net/base/socket_performance_watcher.cc

Index: net/base/socket_performance_watcher.cc
diff --git a/net/base/socket_performance_watcher.cc b/net/base/socket_performance_watcher.cc
index aa0dab3b03c9476a8d099bde13e15e9b65668850..de61df70fa8bce3a3d4ca568209b2c884a9d365f 100644
--- a/net/base/socket_performance_watcher.cc
+++ b/net/base/socket_performance_watcher.cc
@@ -12,6 +12,8 @@ SocketPerformanceWatcher::SocketPerformanceWatcher(
     const SocketPerformanceWatcherFactory::Protocol protocol,
     SocketPerformanceWatcherFactory* socket_performance_watcher_factory)
     : protocol_(protocol),
+      // Currently using a large value until crbug.com/590300 is fixed.
+      rtt_notification_interval_(base::TimeDelta::FromHours(10)),
       socket_performance_watcher_factory_(socket_performance_watcher_factory) {
   DCHECK(socket_performance_watcher_factory_);
 
@@ -27,8 +29,18 @@ SocketPerformanceWatcher::SocketPerformanceWatcher(
 SocketPerformanceWatcher::~SocketPerformanceWatcher() {}
 
 void SocketPerformanceWatcher::OnUpdatedRTTAvailable(
-    const base::TimeDelta& rtt) const {
+    const base::TimeDelta& rtt) {
+  last_rtt_notification_ = base::TimeTicks::Now();
   socket_performance_watcher_factory_->OnUpdatedRTTAvailable(protocol_, rtt);
 }
 
+bool SocketPerformanceWatcher::CanNotifyRTT() const {
+  return base::TimeTicks::Now() - last_rtt_notification_ >=
+         rtt_notification_interval_;
+}
+
+void SocketPerformanceWatcher::Reset() const {

[Ryan Sleevi, 2016/03/02 19:56:57]

Shouldn't you also reset last_rtt_notification_ to the default value of the
ctor?

[tbansal1, 2016/03/03 02:09:08]

I was going to replace last_rtt_notification_ with a counter that counts how
many RTT notifications have been received so far. But, it makes sense to reset
the last_rtt_notification_ to the default value here for consistency.

+  socket_performance_watcher_factory_->OnWatcherReset();
+}
+
 }  // namespace net