PaymentApp: Implement GetAllManifests() in PaymentAppContext.

content/browser/payments/payment_app_context_impl.h

 // 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.
 
 #ifndef CONTENT_BROWSER_PAYMENTS_PAYMENT_APP_CONTEXT_IMPL_H_
 #define CONTENT_BROWSER_PAYMENTS_PAYMENT_APP_CONTEXT_IMPL_H_
 
 #include <map>
 #include <memory>
 
@@ skipping 22 matching lines @@
   void Shutdown();
 
   // Create a PaymentAppManager that is owned by this. Call on the UI
   // thread.
   void CreateService(
       mojo::InterfaceRequest<payments::mojom::PaymentAppManager> request);
 
   // Called by PaymentAppManager objects so that they can
   // be deleted. Call on the IO thread.
   void ServiceHadConnectionError(PaymentAppManager* service);
 

[please use gerrit instead, 2016/12/12 20:17:39]

// Should be accessed only on the IO thread.

[zino, 2016/12/16 19:45:40]

Done.

   PaymentAppDatabase* payment_app_database() const;
 
+  // Call on the UI thread.

[please use gerrit instead, 2016/12/12 20:17:39]

// Should be called on the UI thread.

[zino, 2016/12/16 19:45:40]

Done.

   void GetAllManifests(const GetAllManifestsCallback& callback) override;
 
  protected:
   friend class base::RefCountedThreadSafe<PaymentAppContextImpl>;
-  friend class PaymentAppManagerTest;
+  friend class PaymentAppContentUnitTestBase;
   virtual ~PaymentAppContextImpl();
 
  private:
   void CreatePaymentAppDatabaseOnIO(
       scoped_refptr<ServiceWorkerContextWrapper> service_worker_context);
 
   void CreateServiceOnIOThread(
       mojo::InterfaceRequest<payments::mojom::PaymentAppManager> request);
 
   void ShutdownOnIO();
 
+  void GetAllManifestsOnIO(const GetAllManifestsCallback& callback);
+  // void DidGetAllManifestsOnIO(const PaymentAppDatabase::Manifests&

[please use gerrit instead, 2016/12/12 20:17:39]

Seems you forgot to remove this.

[zino, 2016/12/16 19:45:40]

Done.

+  // manifests);
+  void DidGetAllManifestsOnIO(const GetAllManifestsCallback& callback,
+                              PaymentAppDatabase::Manifests manifests);
+
   // Only accessed on the IO thread.
   std::unique_ptr<PaymentAppDatabase> payment_app_database_;
 
   // The services are owned by this. They're either deleted
   // during ShutdownOnIO or when the channel is closed via
   // ServiceHadConnectionError. Only accessed on the IO thread.
   std::map<PaymentAppManager*, std::unique_ptr<PaymentAppManager>> services_;
 
   DISALLOW_COPY_AND_ASSIGN(PaymentAppContextImpl);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_PAYMENTS_PAYMENT_APP_CONTEXT_IMPL_H_