Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(531)

Unified Diff: chrome/browser/sync/test/integration/status_change_checker.h

Issue 167873002: sync: Improve StatusChangeChecker debug strings (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/sync/test/integration/status_change_checker.h
diff --git a/chrome/browser/sync/test/integration/status_change_checker.h b/chrome/browser/sync/test/integration/status_change_checker.h
index 5d34c0b480d4df5c6b89bac369e59b22dc90fd96..839a26f48ff07a1ac77ce4d9e970b63f23bea6ba 100644
--- a/chrome/browser/sync/test/integration/status_change_checker.h
+++ b/chrome/browser/sync/test/integration/status_change_checker.h
@@ -16,22 +16,19 @@
// ProfileSyncServiceHarness::AwaitStatusChange().
class StatusChangeChecker {
public:
- explicit StatusChangeChecker(const std::string& source);
+ explicit StatusChangeChecker();
// Called every time ProfileSyncServiceHarness is notified of a change in the
// state of the sync engine. Returns true if the desired change has occurred.
virtual bool IsExitConditionSatisfied() = 0;
- std::string source() const { return source_; }
+ // Returns a string representing this current StatusChangeChecker, and
+ // possibly some small part of its state. For example: "AwaitPassphraseError"
+ // or "AwaitMigrationDone(BOOKMARKS)".
+ virtual std::string GetDebugMessage() const = 0;
protected:
virtual ~StatusChangeChecker();
-
- private:
- // Used for logging / debugging. Can be used to hold the name of the internal
- // function called by IsExitConditionSatisfied. Logged along with select info
- // when ProfileSyncServiceHarness observes a change in ProfileSyncService.
- std::string source_;
};
#endif // CHROME_BROWSER_SYNC_TEST_INTEGRATION_STATUS_CHANGE_CHECKER_H_

Powered by Google App Engine
This is Rietveld 408576698