Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(677)

Unified Diff: content/browser/payments/payment_app_context_impl.h

Issue 2556433002: PaymentApp: Implement GetAllManifests() in PaymentAppContext. (Closed)
Patch Set: Created 4 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/browser/payments/payment_app_context_impl.h
diff --git a/content/browser/payments/payment_app_context_impl.h b/content/browser/payments/payment_app_context_impl.h
index 79a85a68327da578064acfbd98a5e4c91d736170..1362f3bb4bcce24c20184c33a20c9a904b170314 100644
--- a/content/browser/payments/payment_app_context_impl.h
+++ b/content/browser/payments/payment_app_context_impl.h
@@ -43,11 +43,12 @@ class CONTENT_EXPORT PaymentAppContextImpl
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:
@@ -59,6 +60,12 @@ class CONTENT_EXPORT PaymentAppContextImpl
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_;

Powered by Google App Engine
This is Rietveld 408576698