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

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

Issue 2389063002: [Sync] Fixing easy lint violations. (Closed)
Patch Set: Updated for Max's comments. 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/retry_verifier.cc
diff --git a/chrome/browser/sync/test/integration/retry_verifier.cc b/chrome/browser/sync/test/integration/retry_verifier.cc
index 7b832d69e14b2b63fcf3935d767713bb588f333e..72b3d98e5c05d122757b468baee87c382c6151e9 100644
--- a/chrome/browser/sync/test/integration/retry_verifier.cc
+++ b/chrome/browser/sync/test/integration/retry_verifier.cc
@@ -39,7 +39,7 @@ DelayInfo CalculateDelay(int64_t current_delay) {
// Fills the table with the maximum and minimum values for each retry, upto
// |count| number of retries.
void FillDelayTable(DelayInfo* delay_table, int count) {
- DCHECK(count > 1);
+ DCHECK_GT(count, 1);
// We start off with the minimum value of 2 seconds.
delay_table[0].min_delay = static_cast<int64_t>(2);
@@ -104,7 +104,7 @@ void RetryVerifier::VerifyRetryInterval(const syncer::SyncCycleSnapshot& snap) {
// has taken place.
if (snap.sync_start_time() != last_sync_time_) {
base::TimeDelta delta = snap.sync_start_time() - last_sync_time_;
- success_ = IsRetryOnTime(delay_table_,retry_count_ -1, delta);
+ success_ = IsRetryOnTime(delay_table_, retry_count_ - 1, delta);
last_sync_time_ = snap.sync_start_time();
++retry_count_;
done_ = (retry_count_ >= kMaxRetry);

Powered by Google App Engine
This is Rietveld 408576698