[Payments] Implement web payment app manifest

content/browser/service_worker/service_worker_context_wrapper.h

Index: content/browser/service_worker/service_worker_context_wrapper.h
diff --git a/content/browser/service_worker/service_worker_context_wrapper.h b/content/browser/service_worker/service_worker_context_wrapper.h
index a41d24539c4d0d08b3f70a0c9328b7b76858048a..8b49be0f9d9b85d405a0fb1f9b3637fe8dba227e 100644
--- a/content/browser/service_worker/service_worker_context_wrapper.h
+++ b/content/browser/service_worker/service_worker_context_wrapper.h
@@ -244,6 +244,11 @@ class CONTENT_EXPORT ServiceWorkerContextWrapper
       int service_worker_provider_id,
       mojom::ServiceWorkerWorkerClientAssociatedPtrInfo client_ptr_info);
 
+  // The core context is only for use on the IO thread.
+  // Can be null before/during init, during/after shutdown, and after
+  // DeleteAndStartOver fails.
+  ServiceWorkerContextCore* context();

[falken, 2017/06/30 01:14:36]

This represents a big shift in this class's role. This class is supposed to be a
barrier against using ContextCore directly since bad things happened when code
outside content/browser/serviceworker/ had direct access:
https://codereview.chromium.org/1079923002

I'm not totally sure whether the reasons given in that CL are still valid, but I
think for this patch we should try to keep the same model and add a helper
function to ContextWrapper for what you need instead of exposing ContextCore.

[gogerald1, 2017/06/30 02:55:22]

Done.

+
  private:
   friend class BackgroundSyncManagerTest;
   friend class base::RefCountedThreadSafe<ServiceWorkerContextWrapper>;
@@ -304,11 +309,6 @@ class CONTENT_EXPORT ServiceWorkerContextWrapper
       const StartServiceWorkerForNavigationHintCallback& callback,
       ServiceWorkerStatusCode code);
 
-  // The core context is only for use on the IO thread.
-  // Can be null before/during init, during/after shutdown, and after
-  // DeleteAndStartOver fails.
-  ServiceWorkerContextCore* context();
-
   // Observers of |context_core_| which live within content's implementation
   // boundary. Shared with |context_core_|.
   const scoped_refptr<