Plumb reserve method of the BudgetAPI

chrome/browser/budget_service/budget_manager.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_manager.h"
 
 #include <stdint.h>
 
 #include "base/callback.h"
 #include "base/memory/ptr_util.h"
@@ skipping 89 matching lines @@
 void BudgetManager::RegisterProfilePrefs(
     user_prefs::PrefRegistrySyncable* registry) {
   registry->RegisterDictionaryPref(prefs::kBackgroundBudgetMap);
 }
 
 // static
 double BudgetManager::GetCost(blink::mojom::BudgetOperationType type) {
   switch (type) {
     case blink::mojom::BudgetOperationType::SILENT_PUSH:
       return 2.0;
+    case blink::mojom::BudgetOperationType::INVALID_OPERATION:
+      return SiteEngagementScore::kMaxPoints + 1;
       // No default case.

[Peter Beverloo, 2016/09/05 17:06:16]

micro nit: it feels to me like this is indented two spaces too much

[harkness, 2016/09/06 15:45:00]

Done.

[Peter Beverloo, 2016/09/06 15:54:10]

No you didn't?

[harkness, 2016/09/06 16:17:25]

Alas, git cl format changed it back after I changed it, and this directory
requires git cl format.

   }
   NOTREACHED();
   return SiteEngagementScore::kMaxPoints + 1.0;
 }
 
 void BudgetManager::GetBudget(const GURL& origin,
                               const GetBudgetCallback& callback) {
   DCHECK_EQ(origin, origin.GetOrigin());
 
   // Get the current SES score, which we'll use to set a new budget.
@@ skipping 66 matching lines @@
   base::Time time = clock_->Now();
   SetBudgetDataInPrefs(profile_, origin, time.ToDoubleT(), budget, ses_score);
 
   BrowserThread::PostTask(BrowserThread::UI, FROM_HERE, base::Bind(closure));
 }
 
 // Override the default clock with the specified clock. Only used for testing.
 void BudgetManager::SetClockForTesting(std::unique_ptr<base::Clock> clock) {
   clock_ = std::move(clock);
 }