PaymentApp: Implement GetAllManifests() in PaymentAppContext.

content/browser/payments/payment_app_content_unittest_base.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_CONTENT_UNITTEST_BASE_H_
+#define CONTENT_BROWSER_PAYMENTS_PAYMENT_APP_CONTENT_UNITTEST_BASE_H_
+
+#include <memory>
+#include <string>
+#include <vector>
+
+#include "content/browser/payments/payment_app_context_impl.h"

[please use gerrit instead, 2016/12/16 21:41:21]

You may be able to use forward declaration:

class PaymentAppContextImpl;

[zino, 2016/12/17 17:11:54]

Done.

+#include "content/browser/payments/payment_app_manager.h"
+#include "testing/gtest/include/gtest/gtest.h"
+
+namespace content {
+
+class EmbeddedWorkerTestHelper;
+class StoragePartitionImpl;
+class TestBrowserThreadBundle;
+
+class PaymentAppContentUnitTestBase : public testing::Test {
+ protected:
+  PaymentAppContentUnitTestBase();
+  ~PaymentAppContentUnitTestBase() override;
+
+  PaymentAppContextImpl* GetPaymentAppContext() const;

[please use gerrit instead, 2016/12/16 21:41:21]

simple getters can be defined in the header with underscores in the name instead
of CamelCase:

PaymentAppContextImpl* payment_app_context() const { return
payment_app_context_.get(); }

[zino, 2016/12/17 17:11:54]

Done.

+  PaymentAppManager* CreatePaymentAppManager(const GURL& scope_url,

[please use gerrit instead, 2016/12/16 21:41:21]

#include "url/gurl.h"

[zino, 2016/12/17 17:11:54]

Done.

+                                             const GURL& sw_script_url);
+  void SetManifest(PaymentAppManager* manager,
+                   const std::string& scope,
+                   payments::mojom::PaymentAppManifestPtr manifest,

[please use gerrit instead, 2016/12/16 21:41:21]

#include "components/payments/payment_app.mojom.h"

[zino, 2016/12/17 17:11:54]

Done.

+                   const PaymentAppManager::SetManifestCallback& callback);
+  void GetManifest(PaymentAppManager* manager,
+                   const std::string& scope,
+                   const PaymentAppManager::GetManifestCallback& callback);
+
+ private:
+  std::unique_ptr<TestBrowserThreadBundle> thread_bundle_;
+  std::unique_ptr<EmbeddedWorkerTestHelper> embedded_worker_helper_;
+  std::unique_ptr<StoragePartitionImpl> storage_partition_impl_;
+  scoped_refptr<PaymentAppContextImpl> payment_app_context_;

[please use gerrit instead, 2016/12/16 21:41:21]

#include "base/memory/ref_counted.h"

[zino, 2016/12/17 17:11:54]

Done.

+  std::vector<payments::mojom::PaymentAppManagerPtr> payment_app_managers_;
+
+  DISALLOW_COPY_AND_ASSIGN(PaymentAppContentUnitTestBase);

[please use gerrit instead, 2016/12/16 21:41:21]

#include "base/macros.h"

[zino, 2016/12/17 17:11:54]

Done.

+};
+
+}  // namespace content
+
+#endif  // CONTENT_BROWSER_PAYMENTS_PAYMENT_APP_CONTENT_UNITTEST_BASE_H_