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

Side by Side Diff: chrome/browser/android/webapk/webapk_update_manager.cc

Issue 2671853002: Rename best icon to best primary icon in Web app related code. (Closed)
Patch Set: Removing java changes Created 3 years, 10 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 unified diff | Download patch
OLDNEW
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 #include "chrome/browser/android/webapk/webapk_update_manager.h" 5 #include "chrome/browser/android/webapk/webapk_update_manager.h"
6 6
7 #include <jni.h> 7 #include <jni.h>
8 8
9 #include "base/android/jni_array.h" 9 #include "base/android/jni_array.h"
10 #include "base/android/jni_string.h" 10 #include "base/android/jni_string.h"
(...skipping 36 matching lines...) Expand 10 before | Expand all | Expand 10 after
47 47
48 // static JNI method. 48 // static JNI method.
49 static void UpdateAsync( 49 static void UpdateAsync(
50 JNIEnv* env, 50 JNIEnv* env,
51 const JavaParamRef<jclass>& clazz, 51 const JavaParamRef<jclass>& clazz,
52 const JavaParamRef<jstring>& java_id, 52 const JavaParamRef<jstring>& java_id,
53 const JavaParamRef<jstring>& java_start_url, 53 const JavaParamRef<jstring>& java_start_url,
54 const JavaParamRef<jstring>& java_scope, 54 const JavaParamRef<jstring>& java_scope,
55 const JavaParamRef<jstring>& java_name, 55 const JavaParamRef<jstring>& java_name,
56 const JavaParamRef<jstring>& java_short_name, 56 const JavaParamRef<jstring>& java_short_name,
57 const JavaParamRef<jstring>& java_best_icon_url, 57 const JavaParamRef<jstring>& java_best_primary_icon_url,
58 const JavaParamRef<jobject>& java_best_icon_bitmap, 58 const JavaParamRef<jobject>& java_best_primary_icon_bitmap,
59 const JavaParamRef<jobjectArray>& java_icon_urls, 59 const JavaParamRef<jobjectArray>& java_icon_urls,
60 const JavaParamRef<jobjectArray>& java_icon_hashes, 60 const JavaParamRef<jobjectArray>& java_icon_hashes,
61 jint java_display_mode, 61 jint java_display_mode,
62 jint java_orientation, 62 jint java_orientation,
63 jlong java_theme_color, 63 jlong java_theme_color,
64 jlong java_background_color, 64 jlong java_background_color,
65 const JavaParamRef<jstring>& java_web_manifest_url, 65 const JavaParamRef<jstring>& java_web_manifest_url,
66 const JavaParamRef<jstring>& java_webapk_package, 66 const JavaParamRef<jstring>& java_webapk_package,
67 jint java_webapk_version, 67 jint java_webapk_version,
68 jboolean java_is_manifest_stale) { 68 jboolean java_is_manifest_stale) {
69 DCHECK_CURRENTLY_ON(content::BrowserThread::UI); 69 DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
70 70
71 Profile* profile = ProfileManager::GetLastUsedProfile(); 71 Profile* profile = ProfileManager::GetLastUsedProfile();
72 if (profile == nullptr) { 72 if (profile == nullptr) {
73 NOTREACHED() << "Profile not found."; 73 NOTREACHED() << "Profile not found.";
74 return; 74 return;
75 } 75 }
76 76
77 std::string id(ConvertJavaStringToUTF8(env, java_id)); 77 std::string id(ConvertJavaStringToUTF8(env, java_id));
78 GURL start_url(ConvertJavaStringToUTF8(env, java_start_url)); 78 GURL start_url(ConvertJavaStringToUTF8(env, java_start_url));
79 GURL scope(ConvertJavaStringToUTF8(env, java_scope)); 79 GURL scope(ConvertJavaStringToUTF8(env, java_scope));
80 GURL web_manifest_url(ConvertJavaStringToUTF8(env, java_web_manifest_url)); 80 GURL web_manifest_url(ConvertJavaStringToUTF8(env, java_web_manifest_url));
81 GURL best_icon_url(ConvertJavaStringToUTF8(env, java_best_icon_url)); 81 GURL best_primary_icon_url(
82 ConvertJavaStringToUTF8(env, java_best_primary_icon_url));
82 ShortcutInfo info(start_url); 83 ShortcutInfo info(start_url);
83 info.scope = scope; 84 info.scope = scope;
84 info.name = ConvertJavaStringToUTF16(env, java_name); 85 info.name = ConvertJavaStringToUTF16(env, java_name);
85 info.short_name = ConvertJavaStringToUTF16(env, java_short_name); 86 info.short_name = ConvertJavaStringToUTF16(env, java_short_name);
86 info.user_title = info.short_name; 87 info.user_title = info.short_name;
87 info.display = static_cast<blink::WebDisplayMode>(java_display_mode); 88 info.display = static_cast<blink::WebDisplayMode>(java_display_mode);
88 info.orientation = 89 info.orientation =
89 static_cast<blink::WebScreenOrientationLockType>(java_orientation); 90 static_cast<blink::WebScreenOrientationLockType>(java_orientation);
90 info.theme_color = (long)java_theme_color; 91 info.theme_color = (long)java_theme_color;
91 info.background_color = (long)java_background_color; 92 info.background_color = (long)java_background_color;
92 info.best_icon_url = best_icon_url; 93 info.best_primary_icon_url = best_primary_icon_url;
93 info.manifest_url = web_manifest_url; 94 info.manifest_url = web_manifest_url;
94 95
95 base::android::AppendJavaStringArrayToStringVector( 96 base::android::AppendJavaStringArrayToStringVector(env, java_icon_urls.obj(),
96 env, java_icon_urls.obj(), &info.icon_urls); 97 &info.icon_urls);
97 98
98 std::vector<std::string> icon_hashes; 99 std::vector<std::string> icon_hashes;
99 base::android::AppendJavaStringArrayToStringVector( 100 base::android::AppendJavaStringArrayToStringVector(
100 env, java_icon_hashes.obj(), &icon_hashes); 101 env, java_icon_hashes.obj(), &icon_hashes);
101 102
102 std::map<std::string, std::string> icon_url_to_murmur2_hash; 103 std::map<std::string, std::string> icon_url_to_murmur2_hash;
103 for (size_t i = 0; i < info.icon_urls.size(); ++i) 104 for (size_t i = 0; i < info.icon_urls.size(); ++i)
104 icon_url_to_murmur2_hash[info.icon_urls[i]] = icon_hashes[i]; 105 icon_url_to_murmur2_hash[info.icon_urls[i]] = icon_hashes[i];
105 106
106 gfx::JavaBitmap java_bitmap_lock(java_best_icon_bitmap); 107 gfx::JavaBitmap java_bitmap_lock(java_best_primary_icon_bitmap);
107 SkBitmap best_icon_bitmap = 108 SkBitmap best_primary_icon_bitmap =
108 gfx::CreateSkBitmapFromJavaBitmap(java_bitmap_lock); 109 gfx::CreateSkBitmapFromJavaBitmap(java_bitmap_lock);
109 best_icon_bitmap.setImmutable(); 110 best_primary_icon_bitmap.setImmutable();
110 111
111 std::string webapk_package; 112 std::string webapk_package;
112 ConvertJavaStringToUTF8(env, java_webapk_package, &webapk_package); 113 ConvertJavaStringToUTF8(env, java_webapk_package, &webapk_package);
113 114
114 WebApkInstallService* install_service = WebApkInstallService::Get(profile); 115 WebApkInstallService* install_service = WebApkInstallService::Get(profile);
115 if (install_service->IsInstallInProgress(info.manifest_url)) { 116 if (install_service->IsInstallInProgress(info.manifest_url)) {
116 base::ThreadTaskRunnerHandle::Get()->PostTask( 117 base::ThreadTaskRunnerHandle::Get()->PostTask(
117 FROM_HERE, 118 FROM_HERE,
118 base::Bind(&WebApkUpdateManager::OnBuiltWebApk, id, false, "")); 119 base::Bind(&WebApkUpdateManager::OnBuiltWebApk, id, false, ""));
119 return; 120 return;
120 } 121 }
121 install_service->UpdateAsync( 122 install_service->UpdateAsync(
122 info, best_icon_bitmap, webapk_package, java_webapk_version, 123 info, best_primary_icon_bitmap, webapk_package, java_webapk_version,
123 icon_url_to_murmur2_hash, java_is_manifest_stale, 124 icon_url_to_murmur2_hash, java_is_manifest_stale,
124 base::Bind(&WebApkUpdateManager::OnBuiltWebApk, id)); 125 base::Bind(&WebApkUpdateManager::OnBuiltWebApk, id));
125 } 126 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698