[Sync] Notify invalidation handlers when sync is disabled/enabled

chrome/browser/sync/profile_sync_service.h

Index: chrome/browser/sync/profile_sync_service.h
diff --git a/chrome/browser/sync/profile_sync_service.h b/chrome/browser/sync/profile_sync_service.h
index c1b9076a24c30e16efea4191a9231a84ee5130ef..238d582b1e77cb69d701aa5602b10444a82bf50a 100644
--- a/chrome/browser/sync/profile_sync_service.h
+++ b/chrome/browser/sync/profile_sync_service.h
@@ -715,6 +715,10 @@ class ProfileSyncService : public ProfileSyncServiceBase,
                                     bool delete_sync_database,
                                     UnrecoverableErrorReason reason);
 
+  // Must be called every time |backend_initialized_| or |state_| is

[Nicolas Zea, 2012/10/04 00:21:00]

state_ -> invalidator_state_?

[akalin, 2012/10/05 21:35:16]

Done.

+  // changed (but only if |invalidator_registrar_| is not NULL).
+  void UpdateInvalidatorState();

[Nicolas Zea, 2012/10/04 00:21:00]

This isn't actually updating the local state, perhaps rename to
NotifyInvalidatorStateChanged?

[akalin, 2012/10/05 21:35:16]

Renamed to UpdateInvalidatorRegistrarState().  Does that work better?

+
   // Destroys / recreates an instance of ProfileSyncService. Used exclusively by
   // the sync integration tests so they can restart sync from scratch without
   // tearing down and recreating the browser process. Needed because simply
@@ -843,6 +847,12 @@ class ProfileSyncService : public ProfileSyncServiceBase,
   // Factory the backend will use to build the SyncManager.
   syncer::SyncManagerFactory sync_manager_factory_;
 
+  // Holds the current invalidator state as updated by
+  // OnInvalidatorStateChange().  Note that this is different from the
+  // state known by |invalidator_registrar_| (See
+  // UpdateInvalidatorState()).
+  syncer::InvalidatorState invalidator_state_;
+
   // Dispatches invalidations to handlers.  Set in Initialize() and
   // unset in Shutdown().
   scoped_ptr<syncer::InvalidatorRegistrar> invalidator_registrar_;