| 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();
|
| }
|
|
|
|
|