Pass icon and icon murmur2 hash to WebApkInstaller when updating WebAPKs.

chrome/browser/android/webapk/webapk_update_manager.cc

 // 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.
 
 #include "chrome/browser/android/webapk/webapk_update_manager.h"
 
 #include <jni.h>
 
 #include "base/android/jni_string.h"
 #include "base/bind.h"
@@ skipping 33 matching lines @@
 
 // static JNI method.
 static void UpdateAsync(JNIEnv* env,
                         const JavaParamRef<jclass>& clazz,
                         const JavaParamRef<jstring>& java_start_url,
                         const JavaParamRef<jstring>& java_scope,
                         const JavaParamRef<jstring>& java_name,
                         const JavaParamRef<jstring>& java_short_name,
                         const JavaParamRef<jstring>& java_icon_url,
                         const JavaParamRef<jobject>& java_icon_bitmap,
+                        jlong java_icon_murmur2_hash,
                         jint java_display_mode,
                         jint java_orientation,
                         jlong java_theme_color,
                         jlong java_background_color,
                         const JavaParamRef<jstring>& java_web_manifest_url,
                         const JavaParamRef<jstring>& java_webapk_package,
                         jint java_webapk_version) {
   DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
 
   Profile* profile = ProfileManager::GetLastUsedProfile();
@@ skipping 22 matching lines @@
   SkBitmap icon_bitmap = gfx::CreateSkBitmapFromJavaBitmap(java_bitmap_lock);
   icon_bitmap.setImmutable();
 
   std::string webapk_package;
   ConvertJavaStringToUTF8(env, java_webapk_package, &webapk_package);
 
   WebApkInstaller* installer = new WebApkInstaller(info, icon_bitmap);
   installer->UpdateAsync(
       profile,
       base::Bind(&WebApkUpdateManager::OnBuiltWebApk, webapk_package),
+      std::to_string(java_icon_murmur2_hash),

[pkotwicz, 2016/08/19 23:38:17]

I think that using base::Uint64ToString() is preferred. However, I do not know
what the difference between base::Uint64ToString() and std::to_string() is

Note: Once I use webapk_icon_hasher.cc in manifest_upgrade_detector_fetcher.cc,
the hash will be passed as a String to ManifestUpgradeDetector.java

[Xi Han, 2016/08/26 17:54:08]

I move the conversion to Java and pass a string directly to here. Thanks for the
heads up.

       webapk_package, java_webapk_version);
 }