PaymentApp: Introduce PaymentAppDatabase class.

content/browser/payments/payment_app_context_impl.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 "content/browser/payments/payment_app_context_impl.h"
 
 #include <utility>
 
 #include "base/bind.h"
 #include "base/memory/ptr_util.h"
 #include "base/stl_util.h"
+#include "content/browser/payments/payment_app_database.h"
 #include "content/browser/payments/payment_app_manager.h"
-#include "content/browser/service_worker/service_worker_context_wrapper.h"
 #include "content/public/browser/browser_thread.h"
 
 namespace content {
 
-PaymentAppContextImpl::PaymentAppContextImpl(
-    scoped_refptr<ServiceWorkerContextWrapper> service_worker_context)
-    : service_worker_context_(std::move(service_worker_context)) {
+PaymentAppContextImpl::PaymentAppContextImpl() {
   DCHECK_CURRENTLY_ON(BrowserThread::UI);
 }
 
+void PaymentAppContextImpl::Init(
+    scoped_refptr<ServiceWorkerContextWrapper> service_worker_context) {
+  DCHECK_CURRENTLY_ON(BrowserThread::UI);
+
+  BrowserThread::PostTask(
+      BrowserThread::IO, FROM_HERE,
+      base::Bind(&PaymentAppContextImpl::CreatePaymentAppDatabaseOnIO, this,

[please use gerrit instead, 2016/12/14 19:44:00]

Either call this InitOnIO() or rename the Init() method into
CreatePaymentAppDatabase(). The two methods on the different threads should have
similar enough names that it's clear who's calling who.

[zino, 2016/12/15 17:56:52]

The Init() function is paired with Shutdown() and only called in
StorageParitionImpl on UI thread. This currently only creates
a PaymentAppDatabase, but something can be added later.
Instead I'll leave a comment in header file.

+                 service_worker_context));
+}
+
 void PaymentAppContextImpl::Shutdown() {
   DCHECK_CURRENTLY_ON(BrowserThread::UI);
 
   BrowserThread::PostTask(
       BrowserThread::IO, FROM_HERE,
       base::Bind(&PaymentAppContextImpl::ShutdownOnIO, this));
 }
 
 void PaymentAppContextImpl::CreateService(
     mojo::InterfaceRequest<payments::mojom::PaymentAppManager> request) {
   DCHECK_CURRENTLY_ON(BrowserThread::UI);
 
-  BrowserThread::PostTask(
-      BrowserThread::IO, FROM_HERE,
-      base::Bind(&PaymentAppContextImpl::CreateServiceOnIOThread, this,
-                 base::Passed(&request)));
+  BrowserThread::PostTask(BrowserThread::IO, FROM_HERE,
+                          base::Bind(&PaymentAppContextImpl::CreateServiceOnIO,
+                                     this, base::Passed(&request)));
 }
 
 void PaymentAppContextImpl::ServiceHadConnectionError(
     PaymentAppManager* service) {
   DCHECK_CURRENTLY_ON(BrowserThread::IO);
   DCHECK(base::ContainsKey(services_, service));
 
   services_.erase(service);
 }
 
-ServiceWorkerContextWrapper* PaymentAppContextImpl::service_worker_context()
-    const {
-  return service_worker_context_.get();
+PaymentAppDatabase* PaymentAppContextImpl::payment_app_database() const {

[please use gerrit instead, 2016/12/14 19:44:00]

DCHECK IO thread.

[zino, 2016/12/15 17:56:52]

Done.

+  return payment_app_database_.get();
 }
 
 void PaymentAppContextImpl::GetAllManifests(
     const GetAllManifestsCallback& callback) {
   NOTIMPLEMENTED();
 }
 
 PaymentAppContextImpl::~PaymentAppContextImpl() {

[please use gerrit instead, 2016/12/14 19:44:00]

DCHECK you're on UI thread, same as constructor.

Don't access services_ and payment_app_database_ on the UI thread.

Have a UI-thread-only boolean variable is_shutdown_ instead. Change Shutdown()
to use BrowserThread::PostTaskAndReply() instead of BrowserThread::PostTask().
When the reply comes back to the UI thread, set is_shutdown_=true. Then you can
DCHECK(is_shutdown_); in this destructor.

[zino, 2016/12/15 17:56:52]

Done.

   DCHECK(services_.empty());
+  DCHECK(!payment_app_database_);
 }
 
-void PaymentAppContextImpl::CreateServiceOnIOThread(
+void PaymentAppContextImpl::CreatePaymentAppDatabaseOnIO(
+    scoped_refptr<ServiceWorkerContextWrapper> service_worker_context) {
+  DCHECK_CURRENTLY_ON(BrowserThread::IO);
+  payment_app_database_ =
+      base::MakeUnique<PaymentAppDatabase>(service_worker_context);
+}
+
+void PaymentAppContextImpl::CreateServiceOnIO(
     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 PaymentAppContextImpl::ShutdownOnIO() {
   DCHECK_CURRENTLY_ON(BrowserThread::IO);
 
   services_.clear();
+  payment_app_database_.reset();
 }
 
 }  // namespace content