Use web-app manifest format for Android payment apps.

components/payments/content/android/payment_manifest_parser_android.cc

 // Copyright 2017 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.
 
 #include "components/payments/content/android/payment_manifest_parser_android.h"
 
 #include <stddef.h>
 
 #include <climits>
 
 #include "base/android/jni_array.h"
 #include "base/android/jni_string.h"
 #include "base/bind.h"
 #include "base/memory/ptr_util.h"
 #include "base/numerics/safe_conversions.h"
 #include "jni/PaymentManifestParser_jni.h"
+#include "url/mojo/url.mojom.h"
 
 namespace payments {
 namespace {
 
 class ParseCallback {
  public:
   explicit ParseCallback(const base::android::JavaParamRef<jobject>& jcallback)
       : jcallback_(jcallback) {}
 
   ~ParseCallback() {}
 
-  void OnManifestParsed(
-      std::vector<mojom::PaymentManifestSectionPtr> manifest) {
+  void OnPaymentMethodManifestParsed(std::vector<GURL> webAppManifestUrls) {
+    DCHECK_GE(100U, webAppManifestUrls.size());
+    JNIEnv* env = base::android::AttachCurrentThread();
+
+    if (webAppManifestUrls.empty()) {

[Mathieu, 2017/04/07 13:29:06]

c++ case plz

[please use gerrit instead, 2017/04/07 16:56:29]

Done.

+      // Can trigger synchronous deletion of PaymentManifestParserAndroid.
+      Java_ManifestParseCallback_onManifestParseFailure(env, jcallback_);
+      return;
+    }
+
+    base::android::ScopedJavaLocalRef<jobjectArray> juris =
+        Java_PaymentManifestParser_createWebAppManifestUris(
+            env, webAppManifestUrls.size());
+
+    for (size_t i = 0; i < webAppManifestUrls.size(); ++i) {
+      DCHECK(Java_PaymentManifestParser_addUri(
+          env, juris.obj(), base::checked_cast<int>(i),
+          base::android::ConvertUTF8ToJavaString(
+              env, webAppManifestUrls[i].spec())));
+    }
+
+    // Can trigger synchronous deletion of PaymentManifestParserAndroid.
+    Java_ManifestParseCallback_onPaymentMethodManifestParseSuccess(
+        env, jcallback_, juris.obj());
+  }
+
+  void OnWebAppManifestParsed(

[Mathieu, 2017/04/07 13:29:06]

Could benefit from comments. Now that the WebAppManifest is parsed, mention what
are the remaining steps. Is it strictly copying the results to java?

[please use gerrit instead, 2017/04/07 16:56:29]

Done. It's strictly a copy.

+      std::vector<mojom::WebAppManifestSectionPtr> manifest) {
+    DCHECK_GE(100U, manifest.size());
     JNIEnv* env = base::android::AttachCurrentThread();
 
     if (manifest.empty()) {
       // Can trigger synchronous deletion of PaymentManifestParserAndroid.
       Java_ManifestParseCallback_onManifestParseFailure(env, jcallback_);
       return;
     }
 
     base::android::ScopedJavaLocalRef<jobjectArray> jmanifest =
         Java_PaymentManifestParser_createManifest(env, manifest.size());
 
-    // Java array indices must be integers.
-    for (size_t i = 0; i < manifest.size() && i <= static_cast<size_t>(INT_MAX);
-         ++i) {
-      const mojom::PaymentManifestSectionPtr& section = manifest[i];
-      if (section->sha256_cert_fingerprints.size() >
-          static_cast<size_t>(INT_MAX)) {
-        continue;
-      }
+    for (size_t i = 0; i < manifest.size(); ++i) {
+      const mojom::WebAppManifestSectionPtr& section = manifest[i];
+      DCHECK_GE(100U, section->fingerprints.size());
 
       Java_PaymentManifestParser_addSectionToManifest(
           env, jmanifest.obj(), base::checked_cast<int>(i),
-          base::android::ConvertUTF8ToJavaString(env, section->package_name),
-          section->version,
-          base::checked_cast<int>(section->sha256_cert_fingerprints.size()));
+          base::android::ConvertUTF8ToJavaString(env, section->id),
+          section->min_version,
+          base::checked_cast<int>(section->fingerprints.size()));
 
-      for (size_t j = 0; j < section->sha256_cert_fingerprints.size() &&
-                         j <= static_cast<size_t>(INT_MAX);
-           ++j) {
-        const std::vector<uint8_t>& fingerprint =
-            section->sha256_cert_fingerprints[j];
+      for (size_t j = 0; j < section->fingerprints.size(); ++j) {
+        const std::vector<uint8_t>& fingerprint = section->fingerprints[j];
         Java_PaymentManifestParser_addFingerprintToSection(
             env, jmanifest.obj(), base::checked_cast<int>(i),
             base::checked_cast<int>(j),
             base::android::ToJavaByteArray(env, fingerprint));
       }
     }
 
     // Can trigger synchronous deletion of PaymentManifestParserAndroid.
-    Java_ManifestParseCallback_onManifestParseSuccess(env, jcallback_,
-                                                      jmanifest.obj());
+    Java_ManifestParseCallback_onWebAppManifestParseSuccess(env, jcallback_,
+                                                            jmanifest.obj());
   }
 
  private:
   base::android::ScopedJavaGlobalRef<jobject> jcallback_;
 
   DISALLOW_COPY_AND_ASSIGN(ParseCallback);
 };
 
 }  // namespace
 
 PaymentManifestParserAndroid::PaymentManifestParserAndroid() {}
 
 PaymentManifestParserAndroid::~PaymentManifestParserAndroid() {}
 
 void PaymentManifestParserAndroid::StartUtilityProcess(
     JNIEnv* env,
     const base::android::JavaParamRef<jobject>& jcaller) {
   host_.StartUtilityProcess();
 }
 
-void PaymentManifestParserAndroid::ParsePaymentManifest(
+void PaymentManifestParserAndroid::ParsePaymentMethodManifest(
     JNIEnv* env,
     const base::android::JavaParamRef<jobject>& jcaller,
     const base::android::JavaParamRef<jstring>& jcontent,
     const base::android::JavaParamRef<jobject>& jcallback) {
-  host_.ParsePaymentManifest(
+  host_.ParsePaymentMethodManifest(
       base::android::ConvertJavaStringToUTF8(env, jcontent),
-      base::BindOnce(&ParseCallback::OnManifestParsed,
+      base::BindOnce(&ParseCallback::OnPaymentMethodManifestParsed,
                      base::MakeUnique<ParseCallback>(jcallback)));
 }
 
+void PaymentManifestParserAndroid::ParseWebAppManifest(
+    JNIEnv* env,
+    const base::android::JavaParamRef<jobject>& jcaller,
+    const base::android::JavaParamRef<jstring>& jcontent,
+    const base::android::JavaParamRef<jobject>& jcallback) {
+  host_.ParseWebAppManifest(
+      base::android::ConvertJavaStringToUTF8(env, jcontent),
+      base::BindOnce(&ParseCallback::OnWebAppManifestParsed,
+                     base::MakeUnique<ParseCallback>(jcallback)));
+}
+
 void PaymentManifestParserAndroid::StopUtilityProcess(
     JNIEnv* env,
     const base::android::JavaParamRef<jobject>& jcaller) {
   delete this;
 }
 
 bool RegisterPaymentManifestParser(JNIEnv* env) {
   return RegisterNativesImpl(env);
 }
 
 // Caller owns the result.
 jlong CreatePaymentManifestParserAndroid(
     JNIEnv* env,
     const base::android::JavaParamRef<jclass>& jcaller) {
   return reinterpret_cast<jlong>(new PaymentManifestParserAndroid);
 }
 
 }  // namespace payments