Always use milliseconds in the Budget API implementation

chrome/browser/budget_service/budget_database.cc

 // Copyright 2016 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/budget_service/budget_database.h"
 
 #include "base/memory/ptr_util.h"
 #include "base/metrics/histogram_macros.h"
 #include "base/task_scheduler/post_task.h"
 #include "base/time/clock.h"
@@ skipping 138 matching lines @@
 
   // Now, build up the BudgetExpection. This is different from the format
   // in which the cache stores the data. The cache stores chunks of budget and
   // when that budget expires. The mojo array describes a set of times
   // and the budget at those times.
   double total = GetBudget(origin);
 
   // Always add one entry at the front of the list for the total budget now.
   blink::mojom::BudgetStatePtr prediction(blink::mojom::BudgetState::New());
   prediction->budget_at = total;
-  prediction->time = clock_->Now().ToDoubleT();
+  prediction->time =
+      clock_->Now().ToDoubleT() * base::Time::kMillisecondsPerSecond;

[Miguel Garcia, 2017/06/05 18:47:10]

Why not simply  clock_->Now().InMilliseconds() (or
clock_->Now().InMilliseconds() if you need a double)


https://cs.corp.google.com/clankium/src/base/time/time.h?sq=package:%5Eclanki...

[Peter Beverloo, 2017/06/05 19:01:24]

Used ToJsTime() so that we get the epoch - thanks!

   predictions.push_back(std::move(prediction));
 
   // Starting with the soonest expiring chunks, add entries for the
   // expiration times going forward.
   const BudgetChunks& chunks = budget_map_[origin].chunks;
   for (const auto& chunk : chunks) {
     blink::mojom::BudgetStatePtr prediction(blink::mojom::BudgetState::New());
     total -= chunk.amount;
     prediction->budget_at = total;
-    prediction->time = chunk.expiration.ToDoubleT();
+    prediction->time =
+        chunk.expiration.ToDoubleT() * base::Time::kMillisecondsPerSecond;
     predictions.push_back(std::move(prediction));
   }
 
   callback.Run(blink::mojom::BudgetServiceErrorType::NONE,
                std::move(predictions));
 }
 
 void BudgetDatabase::SpendBudgetAfterSync(const url::Origin& origin,
                                           double amount,
                                           const SpendBudgetCallback& callback,
@@ skipping 192 matching lines @@
   return false;
 }
 
 double BudgetDatabase::GetSiteEngagementScoreForOrigin(
     const url::Origin& origin) const {
   if (profile_->IsOffTheRecord())
     return 0;
 
   return SiteEngagementService::Get(profile_)->GetScore(origin.GetURL());
 }