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

Unified Diff: content/browser/payments/payment_app_context_impl.cc

Issue 2575583003: PaymentApp: Add a interface to query all payment app manifests. (Closed)
Patch Set: PaymentApp: Add a interface to query all payment app manifests. Created 4 years 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 | « content/browser/payments/payment_app_context_impl.h ('k') | content/browser/payments/payment_app_manager.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/browser/payments/payment_app_context_impl.cc
diff --git a/content/browser/payments/payment_app_context.cc b/content/browser/payments/payment_app_context_impl.cc
similarity index 63%
rename from content/browser/payments/payment_app_context.cc
rename to content/browser/payments/payment_app_context_impl.cc
index 8fc961a5243d3821e1b51dc9484f882fa653e4f7..2565567a5e68e025fe168a7b55fda0731f75f66e 100644
--- a/content/browser/payments/payment_app_context.cc
+++ b/content/browser/payments/payment_app_context_impl.cc
@@ -2,7 +2,7 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
-#include "content/browser/payments/payment_app_context.h"
+#include "content/browser/payments/payment_app_context_impl.h"
#include <utility>
@@ -15,52 +15,60 @@
namespace content {
-PaymentAppContext::PaymentAppContext(
+PaymentAppContextImpl::PaymentAppContextImpl(
scoped_refptr<ServiceWorkerContextWrapper> service_worker_context)
: service_worker_context_(std::move(service_worker_context)) {
DCHECK_CURRENTLY_ON(BrowserThread::UI);
}
-PaymentAppContext::~PaymentAppContext() {
- DCHECK(services_.empty());
-}
-
-void PaymentAppContext::Shutdown() {
+void PaymentAppContextImpl::Shutdown() {
DCHECK_CURRENTLY_ON(BrowserThread::UI);
- BrowserThread::PostTask(BrowserThread::IO, FROM_HERE,
- base::Bind(&PaymentAppContext::ShutdownOnIO, this));
+ BrowserThread::PostTask(
+ BrowserThread::IO, FROM_HERE,
+ base::Bind(&PaymentAppContextImpl::ShutdownOnIO, this));
}
-void PaymentAppContext::CreateService(
+void PaymentAppContextImpl::CreateService(
mojo::InterfaceRequest<payments::mojom::PaymentAppManager> request) {
DCHECK_CURRENTLY_ON(BrowserThread::UI);
BrowserThread::PostTask(
BrowserThread::IO, FROM_HERE,
- base::Bind(&PaymentAppContext::CreateServiceOnIOThread, this,
+ base::Bind(&PaymentAppContextImpl::CreateServiceOnIOThread, this,
base::Passed(&request)));
}
-void PaymentAppContext::ServiceHadConnectionError(PaymentAppManager* service) {
+void PaymentAppContextImpl::ServiceHadConnectionError(
+ PaymentAppManager* service) {
DCHECK_CURRENTLY_ON(BrowserThread::IO);
DCHECK(base::ContainsKey(services_, service));
services_.erase(service);
}
-ServiceWorkerContextWrapper* PaymentAppContext::service_worker_context() const {
+ServiceWorkerContextWrapper* PaymentAppContextImpl::service_worker_context()
+ const {
return service_worker_context_.get();
}
-void PaymentAppContext::CreateServiceOnIOThread(
+void PaymentAppContextImpl::GetAllManifests(
+ const GetAllManifestsCallback& callback) {
+ NOTIMPLEMENTED();
+}
+
+PaymentAppContextImpl::~PaymentAppContextImpl() {
+ DCHECK(services_.empty());
+}
+
+void PaymentAppContextImpl::CreateServiceOnIOThread(
mojo::InterfaceRequest<payments::mojom::PaymentAppManager> request) {
DCHECK_CURRENTLY_ON(BrowserThread::IO);
PaymentAppManager* service = new PaymentAppManager(this, std::move(request));
services_[service] = base::WrapUnique(service);
}
-void PaymentAppContext::ShutdownOnIO() {
+void PaymentAppContextImpl::ShutdownOnIO() {
DCHECK_CURRENTLY_ON(BrowserThread::IO);
services_.clear();
« no previous file with comments | « content/browser/payments/payment_app_context_impl.h ('k') | content/browser/payments/payment_app_manager.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698