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

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

Issue 2641973003: Implement server-suggested update check backoff (Closed)
Patch Set: Nits. 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 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 12 matching lines...) Expand all
23 using base::android::JavaParamRef; 23 using base::android::JavaParamRef;
24 24
25 // static 25 // static
26 bool WebApkUpdateManager::Register(JNIEnv* env) { 26 bool WebApkUpdateManager::Register(JNIEnv* env) {
27 return RegisterNativesImpl(env); 27 return RegisterNativesImpl(env);
28 } 28 }
29 29
30 // static 30 // static
31 void WebApkUpdateManager::OnBuiltWebApk(const std::string& id, 31 void WebApkUpdateManager::OnBuiltWebApk(const std::string& id,
32 bool success, 32 bool success,
33 bool relax_updates,
33 const std::string& webapk_package) { 34 const std::string& webapk_package) {
34 JNIEnv* env = base::android::AttachCurrentThread(); 35 JNIEnv* env = base::android::AttachCurrentThread();
35 36
36 if (success) { 37 if (success) {
37 DVLOG(1) 38 DVLOG(1)
38 << "Sent request to update WebAPK to server. Seems to have worked."; 39 << "Sent request to update WebAPK to server. Seems to have worked.";
39 } else { 40 } else {
40 LOG(WARNING) << "Server request to update WebAPK failed."; 41 LOG(WARNING) << "Server request to update WebAPK failed.";
41 } 42 }
42 43
43 base::android::ScopedJavaLocalRef<jstring> java_id = 44 base::android::ScopedJavaLocalRef<jstring> java_id =
44 base::android::ConvertUTF8ToJavaString(env, id); 45 base::android::ConvertUTF8ToJavaString(env, id);
45 Java_WebApkUpdateManager_onBuiltWebApk(env, java_id.obj(), success); 46 Java_WebApkUpdateManager_onBuiltWebApk(env, java_id.obj(), success,
47 relax_updates);
46 } 48 }
47 49
48 // static JNI method. 50 // static JNI method.
49 static void UpdateAsync( 51 static void UpdateAsync(
50 JNIEnv* env, 52 JNIEnv* env,
51 const JavaParamRef<jclass>& clazz, 53 const JavaParamRef<jclass>& clazz,
52 const JavaParamRef<jstring>& java_id, 54 const JavaParamRef<jstring>& java_id,
53 const JavaParamRef<jstring>& java_start_url, 55 const JavaParamRef<jstring>& java_start_url,
54 const JavaParamRef<jstring>& java_scope, 56 const JavaParamRef<jstring>& java_scope,
55 const JavaParamRef<jstring>& java_name, 57 const JavaParamRef<jstring>& java_name,
(...skipping 53 matching lines...) Expand 10 before | Expand all | Expand 10 after
109 gfx::CreateSkBitmapFromJavaBitmap(java_bitmap_lock); 111 gfx::CreateSkBitmapFromJavaBitmap(java_bitmap_lock);
110 best_primary_icon_bitmap.setImmutable(); 112 best_primary_icon_bitmap.setImmutable();
111 113
112 std::string webapk_package; 114 std::string webapk_package;
113 ConvertJavaStringToUTF8(env, java_webapk_package, &webapk_package); 115 ConvertJavaStringToUTF8(env, java_webapk_package, &webapk_package);
114 116
115 WebApkInstallService* install_service = WebApkInstallService::Get(profile); 117 WebApkInstallService* install_service = WebApkInstallService::Get(profile);
116 if (install_service->IsInstallInProgress(info.manifest_url)) { 118 if (install_service->IsInstallInProgress(info.manifest_url)) {
117 base::ThreadTaskRunnerHandle::Get()->PostTask( 119 base::ThreadTaskRunnerHandle::Get()->PostTask(
118 FROM_HERE, 120 FROM_HERE,
119 base::Bind(&WebApkUpdateManager::OnBuiltWebApk, id, false, "")); 121 base::Bind(&WebApkUpdateManager::OnBuiltWebApk, id, false /* success */,
122 false /* relax_updates */, "" /* webapk_package */));
120 return; 123 return;
121 } 124 }
122 install_service->UpdateAsync( 125 install_service->UpdateAsync(
123 info, best_primary_icon_bitmap, webapk_package, java_webapk_version, 126 info, best_primary_icon_bitmap, webapk_package, java_webapk_version,
124 icon_url_to_murmur2_hash, java_is_manifest_stale, 127 icon_url_to_murmur2_hash, java_is_manifest_stale,
125 base::Bind(&WebApkUpdateManager::OnBuiltWebApk, id)); 128 base::Bind(&WebApkUpdateManager::OnBuiltWebApk, id));
126 } 129 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698