Fixing the super-long delay when GCMAccountTracker posts a task

chrome/browser/services/gcm/gcm_account_tracker_unittest.cc

 // Copyright 2014 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 "chrome/browser/services/gcm/gcm_account_tracker.h"
 
 #include <map>
 #include <string>
 
 #include "base/memory/scoped_ptr.h"
@@ skipping 458 matching lines @@
   // Make sure that if the token was expired it is still needed.
   EXPECT_TRUE(IsFetchingRequired());
 }
 
 // Tests what is the expected time to the next token fetching.
 TEST_F(GCMAccountTrackerTest, GetTimeToNextTokenReporting) {
   tracker()->Start();
   // At this point the last token fetch time is never.
   EXPECT_EQ(base::TimeDelta(), GetTimeToNextTokenReporting());
 
+  // Regular case. The tokens have been just reported.
   driver()->SetLastTokenFetchTime(base::Time::Now());
   EXPECT_TRUE(GetTimeToNextTokenReporting() <=
                   base::TimeDelta::FromSeconds(12 * 60 * 60));
+
+  // A case when gcm driver is not yet initialized.
+  driver()->SetLastTokenFetchTime(base::Time::Max());
+  EXPECT_EQ(base::TimeDelta::FromSeconds(12 * 60 * 60),
+            GetTimeToNextTokenReporting());
+
+  // A case when token reporting calculation is expected to result in more than
+  // 12 hours, in which case we expect exactly 12 hours.
+  driver()->SetLastTokenFetchTime(base::Time::Now() +
+      base::TimeDelta::FromDays(2));
+  EXPECT_EQ(base::TimeDelta::FromSeconds(12 * 60 * 60),
+            GetTimeToNextTokenReporting());
 }
 
 // Tests conditions when token reporting is required.
 TEST_F(GCMAccountTrackerTest, IsTokenReportingRequired) {
   tracker()->Start();
   // Required because it is overdue.
   EXPECT_TRUE(IsTokenReportingRequired());
 
   // Not required because it just happened.
   driver()->SetLastTokenFetchTime(base::Time::Now());
   EXPECT_FALSE(IsTokenReportingRequired());
 
   SignInAccount(kAccountId1);
   IssueAccessToken(kAccountId1);
   driver()->ResetResults();
   // Reporting was triggered, which means testing for required will give false,
   // but we have the update call.
   SignOutAccount(kAccountId1);
   EXPECT_TRUE(driver()->update_accounts_called());
   EXPECT_FALSE(IsTokenReportingRequired());
 }
 
 // TODO(fgorski): Add test for adding account after removal >> make sure it does
 // not mark removal.
 
 }  // namespace gcm