[Payments] Implement web payment app manifest

content/browser/service_worker/service_worker_context_wrapper.h

 // Copyright 2013 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.
 
 #ifndef CONTENT_BROWSER_SERVICE_WORKER_SERVICE_WORKER_CONTEXT_WRAPPER_H_
 #define CONTENT_BROWSER_SERVICE_WORKER_SERVICE_WORKER_CONTEXT_WRAPPER_H_
 
 #include <stdint.h>
 
 #include <memory>
@@ skipping 226 matching lines @@
   bool OriginHasForeignFetchRegistrations(const GURL& origin);
 
   // Binds the ServiceWorkerWorkerClient of a dedicated (or shared) worker to
   // the parent frame's ServiceWorkerProviderHost. (This is used only when
   // off-main-thread-fetch is enabled.)
   void BindWorkerFetchContext(
       int render_process_id,
       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>;
   friend class EmbeddedWorkerTestHelper;
   friend class EmbeddedWorkerBrowserTest;
   friend class ForeignFetchRequestHandler;
   friend class ServiceWorkerDispatcherHost;
   friend class ServiceWorkerInternalsUI;
   friend class ServiceWorkerNavigationHandleCore;
   friend class ServiceWorkerProcessManager;
@@ skipping 40 matching lines @@
   void DidFindRegistrationForNavigationHint(
       const StartServiceWorkerForNavigationHintCallback& callback,
       ServiceWorkerStatusCode status,
       scoped_refptr<ServiceWorkerRegistration> registration);
 
   void DidStartServiceWorkerForNavigationHint(
       const GURL& pattern,
       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<
       base::ObserverListThreadSafe<ServiceWorkerContextCoreObserver>>
       core_observer_list_;
 
   // Observers which live outside content's implementation boundary. Observer
   // methods will always be dispatched on the UI thread.
   base::ObserverList<ServiceWorkerContextObserver> observer_list_;
 
   const std::unique_ptr<ServiceWorkerProcessManager> process_manager_;
   // Cleared in ShutdownOnIO():
   std::unique_ptr<ServiceWorkerContextCore> context_core_;
 
   // Initialized in Init(); true if the user data directory is empty.
   bool is_incognito_;
 
   // Raw pointer to the StoragePartitionImpl owning |this|.
   StoragePartitionImpl* storage_partition_;
 
   // The ResourceContext associated with this context.
   ResourceContext* resource_context_;
 
   DISALLOW_COPY_AND_ASSIGN(ServiceWorkerContextWrapper);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_SERVICE_WORKER_SERVICE_WORKER_CONTEXT_WRAPPER_H_