Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(825)

Unified Diff: chrome/browser/budget_service/budget_manager.cc

Issue 2309863002: Plumb reserve method of the BudgetAPI (Closed)
Patch Set: Rename BudgetServiceErrorType::NO_ERROR to ::NONE to avoid Windows constant name clash. Created 4 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | chrome/browser/budget_service/budget_service_impl.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/budget_service/budget_manager.cc
diff --git a/chrome/browser/budget_service/budget_manager.cc b/chrome/browser/budget_service/budget_manager.cc
index d11b409ae03ee723d3c1673df91cd8353b9fa694..7e08dad0411935032df443bed8e8ee89ede19c1c 100644
--- a/chrome/browser/budget_service/budget_manager.cc
+++ b/chrome/browser/budget_service/budget_manager.cc
@@ -107,6 +107,8 @@ 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.
}
NOTREACHED();
« no previous file with comments | « no previous file | chrome/browser/budget_service/budget_service_impl.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698