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

Unified Diff: chrome/browser/android/webapk/webapk_update_manager.cc

Issue 2641973003: Implement server-suggested update check backoff (Closed)
Patch Set: 2 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/android/webapk/webapk_update_manager.cc
diff --git a/chrome/browser/android/webapk/webapk_update_manager.cc b/chrome/browser/android/webapk/webapk_update_manager.cc
index 235dab3f0e8b2eb842b52493e64f3fb4ae8bcd84..06b3e162ceea44f8cb3b485cf75ae08ab2ff8be4 100644
--- a/chrome/browser/android/webapk/webapk_update_manager.cc
+++ b/chrome/browser/android/webapk/webapk_update_manager.cc
@@ -30,7 +30,8 @@ bool WebApkUpdateManager::Register(JNIEnv* env) {
// static
void WebApkUpdateManager::OnBuiltWebApk(const std::string& id,
bool success,
- const std::string& webapk_package) {
+ const std::string& webapk_package,
+ bool less_updates) {
dominickn 2017/02/14 04:13:34 relax_updates
Xi Han 2017/02/14 22:25:47 Done.
JNIEnv* env = base::android::AttachCurrentThread();
if (success) {
@@ -42,7 +43,8 @@ void WebApkUpdateManager::OnBuiltWebApk(const std::string& id,
base::android::ScopedJavaLocalRef<jstring> java_id =
base::android::ConvertUTF8ToJavaString(env, id);
- Java_WebApkUpdateManager_onBuiltWebApk(env, java_id.obj(), success);
+ Java_WebApkUpdateManager_onBuiltWebApk(env, java_id.obj(), success,
+ less_updates);
}
// static JNI method.
@@ -116,7 +118,7 @@ static void UpdateAsync(
if (install_service->IsInstallInProgress(info.manifest_url)) {
base::ThreadTaskRunnerHandle::Get()->PostTask(
FROM_HERE,
- base::Bind(&WebApkUpdateManager::OnBuiltWebApk, id, false, ""));
+ base::Bind(&WebApkUpdateManager::OnBuiltWebApk, id, false, "", false));
dominickn 2017/02/14 04:13:34 Can you annotate the bool arguments here with an i
Xi Han 2017/02/14 22:25:47 Done.
return;
}
install_service->UpdateAsync(

Powered by Google App Engine
This is Rietveld 408576698