OLD | NEW |
---|---|
1 // Copyright 2016 The Chromium Authors. All rights reserved. | 1 // Copyright 2016 The Chromium Authors. All rights reserved. |
2 // Use of this source code is governed by a BSD-style license that can be | 2 // Use of this source code is governed by a BSD-style license that can be |
3 // found in the LICENSE file. | 3 // found in the LICENSE file. |
4 | 4 |
5 #ifndef CONTENT_BROWSER_PAYMENTS_PAYMENT_APP_CONTENT_UNITTEST_BASE_H_ | 5 #ifndef CONTENT_BROWSER_PAYMENTS_PAYMENT_APP_CONTENT_UNITTEST_BASE_H_ |
6 #define CONTENT_BROWSER_PAYMENTS_PAYMENT_APP_CONTENT_UNITTEST_BASE_H_ | 6 #define CONTENT_BROWSER_PAYMENTS_PAYMENT_APP_CONTENT_UNITTEST_BASE_H_ |
7 | 7 |
8 #include <memory> | 8 #include <memory> |
9 #include <string> | 9 #include <string> |
10 #include <vector> | 10 #include <vector> |
(...skipping 16 matching lines...) Expand all Loading... | |
27 protected: | 27 protected: |
28 PaymentAppContentUnitTestBase(); | 28 PaymentAppContentUnitTestBase(); |
29 ~PaymentAppContentUnitTestBase() override; | 29 ~PaymentAppContentUnitTestBase() override; |
30 | 30 |
31 PaymentAppContextImpl* payment_app_context() const { | 31 PaymentAppContextImpl* payment_app_context() const { |
32 return payment_app_context_.get(); | 32 return payment_app_context_.get(); |
33 } | 33 } |
34 PaymentAppManager* CreatePaymentAppManager(const GURL& scope_url, | 34 PaymentAppManager* CreatePaymentAppManager(const GURL& scope_url, |
35 const GURL& sw_script_url); | 35 const GURL& sw_script_url); |
36 void SetManifest(PaymentAppManager* manager, | 36 void SetManifest(PaymentAppManager* manager, |
37 const std::string& scope, | |
38 payments::mojom::PaymentAppManifestPtr manifest, | 37 payments::mojom::PaymentAppManifestPtr manifest, |
39 const PaymentAppManager::SetManifestCallback& callback); | 38 const PaymentAppManager::SetManifestCallback& callback); |
40 void GetManifest(PaymentAppManager* manager, | 39 void GetManifest(PaymentAppManager* manager, |
41 const std::string& scope, | |
42 const PaymentAppManager::GetManifestCallback& callback); | 40 const PaymentAppManager::GetManifestCallback& callback); |
41 void UnregisterServiceWorker(const GURL& scope_url); | |
please use gerrit instead
2016/12/19 20:45:59
This change seems orthogonal to your goal. Can you
zino
2016/12/20 11:08:55
I agree that the smaller patches are better but th
| |
43 | 42 |
zino
2016/12/20 11:08:55
About flakiness unit test..
I still doubt the sta
| |
44 private: | 43 private: |
45 std::unique_ptr<TestBrowserThreadBundle> thread_bundle_; | 44 std::unique_ptr<TestBrowserThreadBundle> thread_bundle_; |
46 std::unique_ptr<EmbeddedWorkerTestHelper> embedded_worker_helper_; | 45 std::unique_ptr<EmbeddedWorkerTestHelper> embedded_worker_helper_; |
47 std::unique_ptr<StoragePartitionImpl> storage_partition_impl_; | 46 std::unique_ptr<StoragePartitionImpl> storage_partition_impl_; |
48 scoped_refptr<PaymentAppContextImpl> payment_app_context_; | 47 scoped_refptr<PaymentAppContextImpl> payment_app_context_; |
49 std::vector<payments::mojom::PaymentAppManagerPtr> payment_app_managers_; | 48 std::vector<payments::mojom::PaymentAppManagerPtr> payment_app_managers_; |
50 | 49 |
51 DISALLOW_COPY_AND_ASSIGN(PaymentAppContentUnitTestBase); | 50 DISALLOW_COPY_AND_ASSIGN(PaymentAppContentUnitTestBase); |
52 }; | 51 }; |
53 | 52 |
54 } // namespace content | 53 } // namespace content |
55 | 54 |
56 #endif // CONTENT_BROWSER_PAYMENTS_PAYMENT_APP_CONTENT_UNITTEST_BASE_H_ | 55 #endif // CONTENT_BROWSER_PAYMENTS_PAYMENT_APP_CONTENT_UNITTEST_BASE_H_ |
OLD | NEW |