[Doodle] Move ttl clamping from fetcher to service

components/doodle/doodle_service_unittest.cc

 // Copyright 2017 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "components/doodle/doodle_service.h"
 
 #include <memory>
 #include <utility>
 #include <vector>
 
@@ skipping 339 matching lines @@
   // Now load an expired config again. The cached one should go away.
   service()->Refresh();
   EXPECT_CALL(observer, OnDoodleConfigUpdated(Eq(base::nullopt)));
   fetcher()->ServeAllCallbacks(DoodleState::AVAILABLE,
                                base::TimeDelta::FromSeconds(0), config);
 
   // Remove the observer before the service gets destroyed.
   service()->RemoveObserver(&observer);
 }
 
+TEST_F(DoodleServiceTest, ClampsTimeToLive) {
+  // Load a config with an excessive time-to-live.
+  service()->Refresh();
+  DoodleConfig config = CreateConfig(DoodleType::SIMPLE);
+  fetcher()->ServeAllCallbacks(DoodleState::AVAILABLE,
+                               base::TimeDelta::FromDays(100), config);
+  ASSERT_THAT(service()->config(), Eq(config));
+
+  // The time-to-live should have been clamped to a reasonable maximum.
+  ASSERT_THAT(task_runner()->GetPendingTaskCount(), Eq(1u));
+  EXPECT_THAT(task_runner()->NextPendingTaskDelay(),
+              Eq(base::TimeDelta::FromDays(30)));
+}
+
 TEST_F(DoodleServiceTest, RecordsMetricsForSuccessfulDownload) {
   base::HistogramTester histograms;
 
   // Load a doodle config as usual, but let it take some time.
   service()->Refresh();
   task_runner()->FastForwardBy(base::TimeDelta::FromSeconds(5));
   DoodleConfig config = CreateConfig(DoodleType::SIMPLE);
   fetcher()->ServeAllCallbacks(DoodleState::AVAILABLE,
                                base::TimeDelta::FromHours(1), config);
   ASSERT_THAT(service()->config(), Eq(config));
@@ skipping 83 matching lines @@
   // Fast-forward time so that the config expires.
   task_runner()->FastForwardBy(base::TimeDelta::FromHours(1));
   ASSERT_THAT(service()->config(), Eq(base::nullopt));
 
   // This should not have resulted in any metrics being emitted.
   histograms.ExpectTotalCount("Doodle.ConfigDownloadOutcome", 0);
   histograms.ExpectTotalCount("Doodle.ConfigDownloadTime", 0);
 }
 
 }  // namespace doodle