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

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

Issue 2379433002: [Sync] Refactoring of sync integration test checkers to remove boilerplate await methods. (Closed)
Patch Set: Rebase Created 4 years, 2 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.cc
diff --git a/chrome/browser/sync/test/integration/status_change_checker.cc b/chrome/browser/sync/test/integration/status_change_checker.cc
index f5314bbe2133f4eedc198c687e16315e031b07c9..0407390a6a1007322f4b95500922191b0679cbde 100644
--- a/chrome/browser/sync/test/integration/status_change_checker.cc
+++ b/chrome/browser/sync/test/integration/status_change_checker.cc
@@ -9,11 +9,20 @@
#include "base/run_loop.h"
#include "base/timer/timer.h"
-StatusChangeChecker::StatusChangeChecker() : timed_out_(false) {
-}
+StatusChangeChecker::StatusChangeChecker() : timed_out_(false) {}
StatusChangeChecker::~StatusChangeChecker() {}
+bool StatusChangeChecker::Wait() {
+ if (IsExitConditionSatisfied()) {
+ DVLOG(1) << "Already satisfied: " << GetDebugMessage();
+ } else {
+ DVLOG(1) << "Blocking: " << GetDebugMessage();
+ StartBlockingWait();
+ }
+ return !TimedOut();
+}
+
bool StatusChangeChecker::TimedOut() const {
return timed_out_;
}
« no previous file with comments | « chrome/browser/sync/test/integration/status_change_checker.h ('k') | chrome/browser/sync/test/integration/sync_auth_test.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698