test cases for server directed error handling code.

chrome/browser/sync/profile_sync_service_harness.h

Index: chrome/browser/sync/profile_sync_service_harness.h
diff --git a/chrome/browser/sync/profile_sync_service_harness.h b/chrome/browser/sync/profile_sync_service_harness.h
index fcaa603d9d2b463ee2598e706e993b312148ff46..eab11a944a0b3acfccacb2f912cd47ab7b71700a 100644
--- a/chrome/browser/sync/profile_sync_service_harness.h
+++ b/chrome/browser/sync/profile_sync_service_harness.h
@@ -87,6 +87,9 @@ class ProfileSyncServiceHarness
   // Blocks the caller until exponential backoff has been verified to happen.
   bool AwaitExponentialBackoffVerification();
 
+  // Blocks the caller until the syncer receives a actionable error.

[Raghu Simha, 2011/09/16 04:01:02]

nit: s/a actionable/an actionable/

[Raghu Simha, 2011/09/16 04:01:02]

Add "Returns true when so-and-so happens" to this comment.

[lipalani1, 2011/09/19 18:59:13]

Done.

[lipalani1, 2011/09/19 18:59:13]

Done.

+  bool AwaitActionableError();
+
   // Blocks until the given set of data types are migrated.
   bool AwaitMigration(const syncable::ModelTypeSet& expected_migrated_types);
 
@@ -220,6 +223,9 @@ class ProfileSyncServiceHarness
     // The sync client is waiting for migration to finish.
     WAITING_FOR_MIGRATION_TO_FINISH,
 
+    // The sync client is waiting for an actionable error from server.

[Raghu Simha, 2011/09/16 04:01:02]

nit: s/from server/from the server/

[lipalani1, 2011/09/19 18:59:13]

Done.

+    WAITING_FOR_ACTIONABLE_ERROR,
+
     // The client verification is complete. We don't care about the state of
     // the syncer any more.
     WAITING_FOR_NOTHING,