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

Unified Diff: chrome/browser/budget_service/budget_service_impl.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
Index: chrome/browser/budget_service/budget_service_impl.cc
diff --git a/chrome/browser/budget_service/budget_service_impl.cc b/chrome/browser/budget_service/budget_service_impl.cc
index 5a6b8b857681a27b94605ddd76559a3f0e74569a..84378291bc1d3fa10fc693ac789ab672f6a5e397 100644
--- a/chrome/browser/budget_service/budget_service_impl.cc
+++ b/chrome/browser/budget_service/budget_service_impl.cc
@@ -41,5 +41,12 @@ void BudgetServiceImpl::GetBudget(const url::Origin& origin,
const GetBudgetCallback& callback) {
// TODO(harkness) Call the BudgetManager once it supports detailed GetBudget
// calls.
- callback.Run(0);
+ callback.Run(blink::mojom::BudgetServiceErrorType::NONE, 0);
+}
+
+void BudgetServiceImpl::Reserve(const url::Origin& origin,
+ blink::mojom::BudgetOperationType operation,
+ const ReserveCallback& callback) {
+ // TODO(harkness): Call the BudgetManager::Reserve when it is available.
+ callback.Run(blink::mojom::BudgetServiceErrorType::NONE, false);
}

Powered by Google App Engine
This is Rietveld 408576698