Refactor budget computation to be more tuneable.

third_party/WebKit/Source/modules/budget/BudgetService.cpp

 // 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 "modules/budget/BudgetService.h"
 
 #include "bindings/core/v8/ScriptPromise.h"
 #include "bindings/core/v8/ScriptPromiseResolver.h"
 #include "bindings/core/v8/ScriptState.h"
 #include "core/dom/DOMException.h"
@@ skipping 94 matching lines @@
 
 void BudgetService::gotBudget(
     ScriptPromiseResolver* resolver,
     mojom::blink::BudgetServiceErrorType error,
     const WTF::Vector<mojom::blink::BudgetStatePtr> expectations) const {
   if (error != mojom::blink::BudgetServiceErrorType::NONE) {
     resolver->reject(errorTypeToException(error));
     return;
   }
 
   // Copy the chunks into the budget array.

[Peter Beverloo, 2017/01/12 14:40:04]

nit: Please document why we're flooring.

[harkness, 2017/01/12 17:44:15]

Done.

   HeapVector<Member<BudgetState>> budget(expectations.size());
-  for (size_t i = 0; i < expectations.size(); i++)
-    budget[i] =
-        new BudgetState(expectations[i]->budget_at, expectations[i]->time);
+  for (size_t i = 0; i < expectations.size(); i++) {
+    budget[i] = new BudgetState(floor(expectations[i]->budget_at),
+                                expectations[i]->time);
+  }
 
   resolver->resolve(budget);
 }
 
 ScriptPromise BudgetService::reserve(ScriptState* scriptState,
                                      const AtomicString& operation) {
   DCHECK(m_service);
 
   mojom::blink::BudgetOperationType type = stringToOperationType(operation);
   DCHECK_NE(type, mojom::blink::BudgetOperationType::INVALID_OPERATION);
@@ skipping 26 matching lines @@
 
   resolver->resolve(success);
 }
 
 void BudgetService::onConnectionError() {
   LOG(ERROR) << "Unable to connect to the Mojo BudgetService.";
   // TODO(harkness): Reject in flight promises.
 }
 
 }  // namespace blink