[BackgroundSync] Add max sync event duration to BackgroundSyncParameters

chrome/browser/background_sync/background_sync_controller_impl_unittest.cc

Index: chrome/browser/background_sync/background_sync_controller_impl_unittest.cc
diff --git a/chrome/browser/background_sync/background_sync_controller_impl_unittest.cc b/chrome/browser/background_sync/background_sync_controller_impl_unittest.cc
index 3467c73b21360989ca76e5aa3ad73b8aff4d591a..78ecf49b53f82f27189769cbf948f6969737212d 100644
--- a/chrome/browser/background_sync/background_sync_controller_impl_unittest.cc
+++ b/chrome/browser/background_sync/background_sync_controller_impl_unittest.cc
@@ -111,6 +111,8 @@ TEST_F(BackgroundSyncControllerImplTest, SomeParamsSet) {
   EXPECT_EQ(original.retry_delay_factor, sync_parameters.retry_delay_factor);
   EXPECT_EQ(original.min_sync_recovery_time,
             sync_parameters.min_sync_recovery_time);
+  EXPECT_EQ(original.max_sync_event_duration,
+            sync_parameters.max_sync_event_duration);
 }
 
 TEST_F(BackgroundSyncControllerImplTest, AllParamsSet) {
@@ -124,7 +126,8 @@ TEST_F(BackgroundSyncControllerImplTest, AllParamsSet) {
   field_parameters
       [BackgroundSyncControllerImpl::kRetryDelayFactorParameterName] = "300";
   field_parameters[BackgroundSyncControllerImpl::kMinSyncRecoveryTimeName] =
-      "8000000000000";  // something larger than int32.

[davidben, 2015/12/19 00:28:25]

Did you mean to lose that part of the test?

[jkarlin, 2015/12/28 15:51:29]

Yeah, there is no need for it to be an int64. Also changed to int in the field
trial parsing code.

+      "400";
+  field_parameters[BackgroundSyncControllerImpl::kMaxSyncEventDuration] = "500";
   ASSERT_TRUE(variations::AssociateVariationParams(
       BackgroundSyncControllerImpl::kFieldTrialName, kFieldTrialGroup,
       field_parameters));
@@ -137,8 +140,10 @@ TEST_F(BackgroundSyncControllerImplTest, AllParamsSet) {
             sync_parameters.initial_retry_delay);
   EXPECT_EQ(200, sync_parameters.max_sync_attempts);
   EXPECT_EQ(300, sync_parameters.retry_delay_factor);
-  EXPECT_EQ(base::TimeDelta::FromMilliseconds(8000000000000),
+  EXPECT_EQ(base::TimeDelta::FromMilliseconds(400),
             sync_parameters.min_sync_recovery_time);
+  EXPECT_EQ(base::TimeDelta::FromSeconds(500),
+            sync_parameters.max_sync_event_duration);
 }
 
 }  // namespace