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

Unified Diff: components/payments/content/android/payment_manifest_parser_android.h

Issue 2759283002: Make payment manifest download/parse cross-platform (Closed)
Patch Set: Address comments Created 3 years, 9 months 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: components/payments/content/android/payment_manifest_parser_android.h
diff --git a/components/payments/content/android/payment_manifest_parser_android.h b/components/payments/content/android/payment_manifest_parser_android.h
index 0b6250af74a27a514a2c4e4463280f749b5d4e8a..3dc312ac3dc26059d5b45802cb2c88be62dc7ec2 100644
--- a/components/payments/content/android/payment_manifest_parser_android.h
+++ b/components/payments/content/android/payment_manifest_parser_android.h
@@ -13,16 +13,12 @@
#include "base/android/jni_android.h"
#include "base/android/scoped_java_ref.h"
#include "base/macros.h"
-#include "components/payments/content/android/payment_manifest_parser.mojom.h"
-
-namespace content {
-template <class MojoInterface>
-class UtilityProcessMojoClient;
-}
+#include "components/payments/content/payment_manifest_parser_host.h"
namespace payments {
-// Host of the utility process that parses manifest contents.
+// Android wrapper for the host of the utility process that parses manifest
+// contents.
class PaymentManifestParserAndroid {
public:
PaymentManifestParserAndroid();
@@ -42,19 +38,7 @@ class PaymentManifestParserAndroid {
const base::android::JavaParamRef<jobject>& jcaller);
private:
- class ParseCallback;
-
- // The |callback_identifier| parameter is a pointer to one of the owned
- // elements in the |pending_callbacks_| list.
- void OnParse(ParseCallback* callback_identifier,
- std::vector<mojom::PaymentManifestSectionPtr> manifest);
-
- void OnUtilityProcessStopped();
-
- std::unique_ptr<
- content::UtilityProcessMojoClient<mojom::PaymentManifestParser>>
- mojo_client_;
- std::vector<std::unique_ptr<ParseCallback>> pending_callbacks_;
+ PaymentManifestParserHost host_;
DISALLOW_COPY_AND_ASSIGN(PaymentManifestParserAndroid);
};

Powered by Google App Engine
This is Rietveld 408576698