Adding a method to the Sync test harness that disables sync for a client.

The sync integration test harness does not provide tests with a way of
disabling a sync client, thereby blocking the automation of some tests.

This patch adds a method called DisableSyncForAllDatatypes() to the
ProfileSyncServiceTestHarness class. It also contains a refactor of some
of the inner methods, to make them support the SYNC_DISABLED state.

BUG=56297
TEST=Disable sync, make a local change, make a change in a partner, re-enable sync, etc.

Review URL: http://codereview.chromium.org/3647006

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62441 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 4 insertions, 1 deletions

diff --git a/chrome/browser/sync/profile_sync_service.h b/chrome/browser/sync/profile_sync_service.h
index 1e9dc4d..d5514040 100644
--- a/chrome/browser/sync/profile_sync_service.h
+++ b/chrome/browser/sync/profile_sync_service.h
@@ -275,6 +275,9 @@ class ProfileSyncService : public browser_sync::SyncFrontend,
   virtual void AddObserver(Observer* observer);
   virtual void RemoveObserver(Observer* observer);
 
+  // Returns true if |observer| has already been added as an observer.
+  bool HasObserver(Observer* observer) const;
+
   // Record stats on various events.
   static void SyncEvent(SyncEventCodes code);
 
@@ -283,7 +286,7 @@ class ProfileSyncService : public browser_sync::SyncFrontend,
   // command-line switches).
   static bool IsSyncEnabled();
 
-  // Retuns whether sync is managed, i.e. controlled by configuration
+  // Returns whether sync is managed, i.e. controlled by configuration
   // management. If so, the user is not allowed to configure sync.
   bool IsManaged();