Notify NQE of TCP RTT values

net/base/socket_performance_watcher.cc

 // Copyright 2015 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 "net/base/socket_performance_watcher.h"
 
 #include "base/logging.h"
 
 namespace net {
 
 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_);
 
   switch (protocol) {
     case SocketPerformanceWatcherFactory::PROTOCOL_TCP:
     case SocketPerformanceWatcherFactory::PROTOCOL_QUIC:
       return;
     default:
       NOTREACHED();
   }
 }
 
 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