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

Side by Side Diff: google_apis/gcm/engine/heartbeat_manager.cc

Issue 641943002: components: Introduce AlarmTimer class and use it for GCM heartbeat (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix gyp files Created 6 years, 2 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 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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 "google_apis/gcm/engine/heartbeat_manager.h" 5 #include "google_apis/gcm/engine/heartbeat_manager.h"
6 6
7 #include "base/timer/timer.h"
7 #include "google_apis/gcm/protocol/mcs.pb.h" 8 #include "google_apis/gcm/protocol/mcs.pb.h"
8 #include "net/base/network_change_notifier.h" 9 #include "net/base/network_change_notifier.h"
9 10
10 namespace gcm { 11 namespace gcm {
11 12
12 namespace { 13 namespace {
13 // The default heartbeat when on a mobile or unknown network . 14 // The default heartbeat when on a mobile or unknown network .
14 const int64 kCellHeartbeatDefaultMs = 1000 * 60 * 28; // 28 minutes. 15 const int64 kCellHeartbeatDefaultMs = 1000 * 60 * 28; // 28 minutes.
15 // The default heartbeat when on WiFi (also used for ethernet). 16 // The default heartbeat when on WiFi (also used for ethernet).
16 const int64 kWifiHeartbeatDefaultMs = 1000 * 60 * 15; // 15 minutes. 17 const int64 kWifiHeartbeatDefaultMs = 1000 * 60 * 15; // 15 minutes.
17 // The default heartbeat ack interval. 18 // The default heartbeat ack interval.
18 const int64 kHeartbeatAckDefaultMs = 1000 * 60 * 1; // 1 minute. 19 const int64 kHeartbeatAckDefaultMs = 1000 * 60 * 1; // 1 minute.
19 } // namespace 20 } // namespace
20 21
21 HeartbeatManager::HeartbeatManager() 22 HeartbeatManager::HeartbeatManager()
22 : waiting_for_ack_(false), 23 : started_(false),
24 waiting_for_ack_(false),
23 heartbeat_interval_ms_(0), 25 heartbeat_interval_ms_(0),
24 server_interval_ms_(0), 26 server_interval_ms_(0),
25 heartbeat_timer_(true /* retain user task */, 27 heartbeat_timer_(new base::Timer(true /* retain user task */,
26 false /* not repeating */), 28 false /* not repeating */)),
27 weak_ptr_factory_(this) {} 29 weak_ptr_factory_(this) {}
28 30
29 HeartbeatManager::~HeartbeatManager() {} 31 HeartbeatManager::~HeartbeatManager() {}
30 32
31 void HeartbeatManager::Start( 33 void HeartbeatManager::Start(
32 const base::Closure& send_heartbeat_callback, 34 const base::Closure& send_heartbeat_callback,
33 const base::Closure& trigger_reconnect_callback) { 35 const base::Closure& trigger_reconnect_callback) {
34 DCHECK(!send_heartbeat_callback.is_null()); 36 DCHECK(!send_heartbeat_callback.is_null());
35 DCHECK(!trigger_reconnect_callback.is_null()); 37 DCHECK(!trigger_reconnect_callback.is_null());
36 send_heartbeat_callback_ = send_heartbeat_callback; 38 send_heartbeat_callback_ = send_heartbeat_callback;
37 trigger_reconnect_callback_ = trigger_reconnect_callback; 39 trigger_reconnect_callback_ = trigger_reconnect_callback;
38 40
39 // Kicks off the timer. 41 // Kicks off the timer.
42 started_ = true;
40 waiting_for_ack_ = false; 43 waiting_for_ack_ = false;
41 RestartTimer(); 44 RestartTimer();
42 } 45 }
43 46
44 void HeartbeatManager::Stop() { 47 void HeartbeatManager::Stop() {
45 heartbeat_timer_.Stop(); 48 heartbeat_timer_->Stop();
46 waiting_for_ack_ = false; 49 waiting_for_ack_ = false;
47 } 50 }
48 51
49 void HeartbeatManager::OnHeartbeatAcked() { 52 void HeartbeatManager::OnHeartbeatAcked() {
50 if (!heartbeat_timer_.IsRunning()) 53 if (!heartbeat_timer_->IsRunning())
51 return; 54 return;
52 55
53 DCHECK(!send_heartbeat_callback_.is_null()); 56 DCHECK(!send_heartbeat_callback_.is_null());
54 DCHECK(!trigger_reconnect_callback_.is_null()); 57 DCHECK(!trigger_reconnect_callback_.is_null());
55 waiting_for_ack_ = false; 58 waiting_for_ack_ = false;
56 RestartTimer(); 59 RestartTimer();
57 } 60 }
58 61
59 void HeartbeatManager::UpdateHeartbeatConfig( 62 void HeartbeatManager::UpdateHeartbeatConfig(
60 const mcs_proto::HeartbeatConfig& config) { 63 const mcs_proto::HeartbeatConfig& config) {
61 if (!config.IsInitialized() || 64 if (!config.IsInitialized() ||
62 !config.has_interval_ms() || 65 !config.has_interval_ms() ||
63 config.interval_ms() <= 0) { 66 config.interval_ms() <= 0) {
64 return; 67 return;
65 } 68 }
66 DVLOG(1) << "Updating heartbeat interval to " << config.interval_ms(); 69 DVLOG(1) << "Updating heartbeat interval to " << config.interval_ms();
67 server_interval_ms_ = config.interval_ms(); 70 server_interval_ms_ = config.interval_ms();
68 } 71 }
69 72
70 base::TimeTicks HeartbeatManager::GetNextHeartbeatTime() const { 73 base::TimeTicks HeartbeatManager::GetNextHeartbeatTime() const {
71 if (heartbeat_timer_.IsRunning()) 74 if (heartbeat_timer_->IsRunning())
72 return heartbeat_timer_.desired_run_time(); 75 return heartbeat_timer_->desired_run_time();
73 else 76 else
74 return base::TimeTicks(); 77 return base::TimeTicks();
75 } 78 }
76 79
80 void HeartbeatManager::SetHeartbeatTimer(scoped_ptr<base::Timer> timer) {
81 DCHECK(!started_) << "Attempting to set heartbeat timer after Start() has "
82 << "been called";
83 heartbeat_timer_ = timer.Pass();
84 }
85
77 void HeartbeatManager::OnHeartbeatTriggered() { 86 void HeartbeatManager::OnHeartbeatTriggered() {
78 if (waiting_for_ack_) { 87 if (waiting_for_ack_) {
79 LOG(WARNING) << "Lost connection to MCS, reconnecting."; 88 LOG(WARNING) << "Lost connection to MCS, reconnecting.";
80 Stop(); 89 Stop();
81 trigger_reconnect_callback_.Run(); 90 trigger_reconnect_callback_.Run();
82 return; 91 return;
83 } 92 }
84 93
85 waiting_for_ack_ = true; 94 waiting_for_ack_ = true;
86 RestartTimer(); 95 RestartTimer();
(...skipping 15 matching lines...) Expand all
102 // For unknown connections, use the longer cellular heartbeat interval. 111 // For unknown connections, use the longer cellular heartbeat interval.
103 heartbeat_interval_ms_ = kCellHeartbeatDefaultMs; 112 heartbeat_interval_ms_ = kCellHeartbeatDefaultMs;
104 } 113 }
105 DVLOG(1) << "Sending next heartbeat in " 114 DVLOG(1) << "Sending next heartbeat in "
106 << heartbeat_interval_ms_ << " ms."; 115 << heartbeat_interval_ms_ << " ms.";
107 } else { 116 } else {
108 heartbeat_interval_ms_ = kHeartbeatAckDefaultMs; 117 heartbeat_interval_ms_ = kHeartbeatAckDefaultMs;
109 DVLOG(1) << "Resetting timer for ack with " 118 DVLOG(1) << "Resetting timer for ack with "
110 << heartbeat_interval_ms_ << " ms interval."; 119 << heartbeat_interval_ms_ << " ms interval.";
111 } 120 }
112 heartbeat_timer_.Start(FROM_HERE, 121 heartbeat_timer_->Start(FROM_HERE,
113 base::TimeDelta::FromMilliseconds( 122 base::TimeDelta::FromMilliseconds(
114 heartbeat_interval_ms_), 123 heartbeat_interval_ms_),
115 base::Bind(&HeartbeatManager::OnHeartbeatTriggered, 124 base::Bind(&HeartbeatManager::OnHeartbeatTriggered,
116 weak_ptr_factory_.GetWeakPtr())); 125 weak_ptr_factory_.GetWeakPtr()));
117 } 126 }
118 127
119 } // namespace gcm 128 } // namespace gcm
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698