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

Unified Diff: content/public/browser/background_sync_parameters.cc

Issue 1536023002: [BackgroundSync] Add max sync event duration to BackgroundSyncParameters (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@sync_time2
Patch Set: Created 5 years 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: content/public/browser/background_sync_parameters.cc
diff --git a/content/public/browser/background_sync_parameters.cc b/content/public/browser/background_sync_parameters.cc
index 2f05b41f95f3dcf5f7dfc1a6f7270c02d6b33555..79ddcbd8e49e9903a812d73095df0a005aac9d65 100644
--- a/content/public/browser/background_sync_parameters.cc
+++ b/content/public/browser/background_sync_parameters.cc
@@ -11,6 +11,7 @@ const int kMaxSyncAttempts = 5;
const int kInitialRetryDelayMins = 5;
const int kRetryDelayFactor = 3;
const int64_t kMinSyncRecoveryTimeMs = 1000 * 60 * 6; // 6 minutes
+const int kMaxSyncEventSec = 3 * 60; // 3 minutes
}
BackgroundSyncParameters::BackgroundSyncParameters()
@@ -19,7 +20,8 @@ BackgroundSyncParameters::BackgroundSyncParameters()
initial_retry_delay(base::TimeDelta::FromMinutes(kInitialRetryDelayMins)),
retry_delay_factor(kRetryDelayFactor),
min_sync_recovery_time(
- base::TimeDelta::FromMilliseconds(kMinSyncRecoveryTimeMs)) {}
+ base::TimeDelta::FromMilliseconds(kMinSyncRecoveryTimeMs)),
+ max_sync_event_duration(base::TimeDelta::FromSeconds(kMaxSyncEventSec)) {}
bool BackgroundSyncParameters::operator==(
const BackgroundSyncParameters& other) const {
@@ -27,7 +29,8 @@ bool BackgroundSyncParameters::operator==(
max_sync_attempts == other.max_sync_attempts &&
initial_retry_delay == other.initial_retry_delay &&
retry_delay_factor == other.retry_delay_factor &&
- min_sync_recovery_time == other.min_sync_recovery_time;
+ min_sync_recovery_time == other.min_sync_recovery_time &&
+ max_sync_event_duration == other.max_sync_event_duration;
}
} // namespace content

Powered by Google App Engine
This is Rietveld 408576698