sync: Change progress marker checking in tests

chrome/browser/sync/test/integration/single_client_status_change_checker.h

Index: chrome/browser/sync/test/integration/single_client_status_change_checker.h
diff --git a/chrome/browser/sync/test/integration/single_client_status_change_checker.h b/chrome/browser/sync/test/integration/single_client_status_change_checker.h
index be5a1ba50ac7566f7e3498f50e04abc74ca93040..c5f9a80808e72e519b70e45d8afdcf245ada1ef7 100644
--- a/chrome/browser/sync/test/integration/single_client_status_change_checker.h
+++ b/chrome/browser/sync/test/integration/single_client_status_change_checker.h
@@ -9,7 +9,7 @@
 #include "chrome/browser/sync/test/integration/status_change_checker.h"
 
 class ProfileSyncService;
-class ProfileSyncServiceObserver;
+class ProfileSyncServiceHarness;
 
 // This class provides some common functionality for StatusChangeCheckers that
 // observe only one ProfileSyncService.  This class is abstract.  Its
@@ -22,8 +22,8 @@ class SingleClientStatusChangeChecker : public StatusChangeChecker {
   // StatusChangeChecker implementations and stubs.
   virtual bool IsExitConditionSatisfied() = 0;
   virtual std::string GetDebugMessage() const = 0;
-  virtual void InitObserver(ProfileSyncServiceObserver*) OVERRIDE;
-  virtual void UninitObserver(ProfileSyncServiceObserver*) OVERRIDE;
+  virtual void InitObserver(ProfileSyncServiceHarness* obs) OVERRIDE;

[rlarocque, 2014/03/04 01:56:18]

This change is pretty ugly.  It was needed so we could have the
QuiesceStatusChangeChecker poke the harness so it would call back into the
QuiesceStatusChangeChecker::IsExitConditionSatisfied().

The next CL I write should remove large parts of ProfileSyncServiceHarness and
this function.

+  virtual void UninitObserver(ProfileSyncServiceHarness* obs) OVERRIDE;
 
  protected:
   ProfileSyncService* service() { return service_; }