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

Unified Diff: sync/notifier/chrome_invalidation_client.cc

Issue 10916131: [Invalidations] Add GetInvalidatorState() to Invalidator{,Frontend} (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix compile error Created 8 years, 3 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 side-by-side diff with in-line comments
Download patch
Index: sync/notifier/chrome_invalidation_client.cc
diff --git a/sync/notifier/chrome_invalidation_client.cc b/sync/notifier/chrome_invalidation_client.cc
index 938f7e679bb922c711acc1b4957eb4ca14a5182f..8ac7b60ce43eb8d77957ce19e8eb6e08405aadee 100644
--- a/sync/notifier/chrome_invalidation_client.cc
+++ b/sync/notifier/chrome_invalidation_client.cc
@@ -34,8 +34,8 @@ ChromeInvalidationClient::ChromeInvalidationClient(
chrome_system_resources_(push_client.Pass(),
ALLOW_THIS_IN_INITIALIZER_LIST(this)),
listener_(NULL),
- ticl_state_(DEFAULT_NOTIFICATION_ERROR),
- push_client_state_(DEFAULT_NOTIFICATION_ERROR) {
+ ticl_state_(DEFAULT_INVALIDATION_ERROR),
+ push_client_state_(DEFAULT_INVALIDATION_ERROR) {
DCHECK(CalledOnValidThread());
push_client_->AddObserver(this);
}
@@ -105,10 +105,10 @@ void ChromeInvalidationClient::UpdateCredentials(
void ChromeInvalidationClient::UpdateRegisteredIds(const ObjectIdSet& ids) {
DCHECK(CalledOnValidThread());
registered_ids_ = ids;
- // |ticl_state_| can go to NO_NOTIFICATION_ERROR even without a
+ // |ticl_state_| can go to INVALIDATIONS_ENABLED even without a
// working XMPP connection (as observed by us), so check it instead
// of GetState() (see http://crbug.com/139424).
- if (ticl_state_ == NO_NOTIFICATION_ERROR && registration_manager_.get()) {
+ if (ticl_state_ == INVALIDATIONS_ENABLED && registration_manager_.get()) {
DoRegistrationUpdate();
}
}
@@ -117,7 +117,7 @@ void ChromeInvalidationClient::Ready(
invalidation::InvalidationClient* client) {
DCHECK(CalledOnValidThread());
DCHECK_EQ(client, invalidation_client_.get());
- ticl_state_ = NO_NOTIFICATION_ERROR;
+ ticl_state_ = INVALIDATIONS_ENABLED;
EmitStateChange();
DoRegistrationUpdate();
}
@@ -274,9 +274,9 @@ void ChromeInvalidationClient::InformError(
<< error_info.error_message()
<< " (transient = " << error_info.is_transient() << ")";
if (error_info.error_reason() == invalidation::ErrorReason::AUTH_FAILURE) {
- ticl_state_ = NOTIFICATION_CREDENTIALS_REJECTED;
+ ticl_state_ = INVALIDATION_CREDENTIALS_REJECTED;
} else {
- ticl_state_ = TRANSIENT_NOTIFICATION_ERROR;
+ ticl_state_ = TRANSIENT_INVALIDATION_ERROR;
}
EmitStateChange();
}
@@ -316,40 +316,36 @@ void ChromeInvalidationClient::Stop() {
invalidation_state_tracker_.Reset();
max_invalidation_versions_.clear();
- ticl_state_ = DEFAULT_NOTIFICATION_ERROR;
- push_client_state_ = DEFAULT_NOTIFICATION_ERROR;
+ ticl_state_ = DEFAULT_INVALIDATION_ERROR;
+ push_client_state_ = DEFAULT_INVALIDATION_ERROR;
}
-NotificationsDisabledReason ChromeInvalidationClient::GetState() const {
+InvalidatorState ChromeInvalidationClient::GetState() const {
DCHECK(CalledOnValidThread());
- if (ticl_state_ == NOTIFICATION_CREDENTIALS_REJECTED ||
- push_client_state_ == NOTIFICATION_CREDENTIALS_REJECTED) {
+ if (ticl_state_ == INVALIDATION_CREDENTIALS_REJECTED ||
+ push_client_state_ == INVALIDATION_CREDENTIALS_REJECTED) {
// If either the ticl or the push client rejected our credentials,
- // return NOTIFICATION_CREDENTIALS_REJECTED.
- return NOTIFICATION_CREDENTIALS_REJECTED;
+ // return INVALIDATION_CREDENTIALS_REJECTED.
+ return INVALIDATION_CREDENTIALS_REJECTED;
}
- if (ticl_state_ == NO_NOTIFICATION_ERROR &&
- push_client_state_ == NO_NOTIFICATION_ERROR) {
+ if (ticl_state_ == INVALIDATIONS_ENABLED &&
+ push_client_state_ == INVALIDATIONS_ENABLED) {
// If the ticl is ready and the push client notifications are
- // enabled, return NO_NOTIFICATION_ERROR.
- return NO_NOTIFICATION_ERROR;
+ // enabled, return INVALIDATIONS_ENABLED.
+ return INVALIDATIONS_ENABLED;
}
// Otherwise, we have a transient error.
- return TRANSIENT_NOTIFICATION_ERROR;
+ return TRANSIENT_INVALIDATION_ERROR;
}
void ChromeInvalidationClient::EmitStateChange() {
DCHECK(CalledOnValidThread());
- if (GetState() == NO_NOTIFICATION_ERROR) {
- listener_->OnNotificationsEnabled();
- } else {
- listener_->OnNotificationsDisabled(GetState());
- }
+ listener_->OnInvalidatorStateChange(GetState());
}
void ChromeInvalidationClient::OnNotificationsEnabled() {
DCHECK(CalledOnValidThread());
- push_client_state_ = NO_NOTIFICATION_ERROR;
+ push_client_state_ = INVALIDATIONS_ENABLED;
EmitStateChange();
}

Powered by Google App Engine
This is Rietveld 408576698