Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(102)

Side by Side Diff: net/base/socket_performance_watcher.cc

Issue 1376473003: Notify NQE of TCP RTT values (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Addressed rsleevi comments Created 4 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "net/base/socket_performance_watcher.h" 5 #include "net/base/socket_performance_watcher.h"
6 6
7 #include "base/logging.h" 7 #include "base/logging.h"
8 8
9 namespace net { 9 namespace net {
10 10
11 SocketPerformanceWatcher::SocketPerformanceWatcher( 11 SocketPerformanceWatcher::SocketPerformanceWatcher(
12 const SocketPerformanceWatcherFactory::Protocol protocol, 12 const SocketPerformanceWatcherFactory::Protocol protocol,
13 SocketPerformanceWatcherFactory* socket_performance_watcher_factory) 13 SocketPerformanceWatcherFactory* socket_performance_watcher_factory)
14 : protocol_(protocol), 14 : protocol_(protocol),
15 rtt_notification_interval_(base::TimeDelta::FromMilliseconds(10000)),
Ryan Sleevi 2016/02/25 22:48:46 Why not ::FromSeconds(10) ?
tbansal1 2016/02/26 23:43:34 Done. Just changed it to a very large value until
15 socket_performance_watcher_factory_(socket_performance_watcher_factory) { 16 socket_performance_watcher_factory_(socket_performance_watcher_factory) {
16 DCHECK(socket_performance_watcher_factory_); 17 DCHECK(socket_performance_watcher_factory_);
17 18
18 switch (protocol) { 19 switch (protocol) {
19 case SocketPerformanceWatcherFactory::PROTOCOL_TCP: 20 case SocketPerformanceWatcherFactory::PROTOCOL_TCP:
20 case SocketPerformanceWatcherFactory::PROTOCOL_QUIC: 21 case SocketPerformanceWatcherFactory::PROTOCOL_QUIC:
21 return; 22 return;
22 default: 23 default:
23 NOTREACHED(); 24 NOTREACHED();
24 } 25 }
25 } 26 }
26 27
27 SocketPerformanceWatcher::~SocketPerformanceWatcher() {} 28 SocketPerformanceWatcher::~SocketPerformanceWatcher() {}
28 29
29 void SocketPerformanceWatcher::OnUpdatedRTTAvailable( 30 void SocketPerformanceWatcher::OnUpdatedRTTAvailable(
30 const base::TimeDelta& rtt) const { 31 const base::TimeDelta& rtt) const {
31 socket_performance_watcher_factory_->OnUpdatedRTTAvailable(protocol_, rtt); 32 socket_performance_watcher_factory_->OnUpdatedRTTAvailable(protocol_, rtt);
32 } 33 }
33 34
35 void SocketPerformanceWatcher::Reset() const {
36 socket_performance_watcher_factory_->WatcherReset();
37 }
38
34 } // namespace net 39 } // namespace net
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698