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

Unified Diff: chrome/browser/android/banners/app_banner_infobar_delegate_android.cc

Issue 2641973003: Implement server-suggested update check backoff (Closed)
Patch Set: Created 3 years, 11 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/banners/app_banner_infobar_delegate_android.cc
diff --git a/chrome/browser/android/banners/app_banner_infobar_delegate_android.cc b/chrome/browser/android/banners/app_banner_infobar_delegate_android.cc
index 8ed416a796caedb807fa666b130c2714a1d4a298..126b2e8f7dc9e6a0aed1d6e5eee28279081c56fe 100644
--- a/chrome/browser/android/banners/app_banner_infobar_delegate_android.cc
+++ b/chrome/browser/android/banners/app_banner_infobar_delegate_android.cc
@@ -13,7 +13,6 @@
#include "chrome/browser/android/shortcut_info.h"
#include "chrome/browser/android/tab_android.h"
#include "chrome/browser/android/webapk/chrome_webapk_host.h"
-#include "chrome/browser/android/webapk/webapk_installer.h"
#include "chrome/browser/banners/app_banner_manager.h"
#include "chrome/browser/banners/app_banner_metrics.h"
#include "chrome/browser/banners/app_banner_settings_helper.h"
@@ -351,7 +350,7 @@ bool AppBannerInfoBarDelegateAndroid::AcceptWebApk(
Java_AppBannerInfoBarDelegateAndroid_setWebApkInstallingState(
env, java_delegate_, true);
UpdateInstallState(env, nullptr);
- WebApkInstaller::FinishCallback callback =
+ WebApkInstallService::FinishCallback callback =
base::Bind(&AppBannerInfoBarDelegateAndroid::OnWebApkInstallFinished,
weak_ptr_factory_.GetWeakPtr());
ShortcutHelper::InstallWebApkWithSkBitmap(web_contents->GetBrowserContext(),
@@ -380,7 +379,7 @@ void AppBannerInfoBarDelegateAndroid::SendBannerAccepted() {
void AppBannerInfoBarDelegateAndroid::OnWebApkInstallFinished(
bool success,
- const std::string& webapk_package_name) {
+ const WebApkInstallService::FinishCallbackData& data) {
JNIEnv* env = base::android::AttachCurrentThread();
if (!success) {
DVLOG(1) << "The WebAPK installation failed.";
@@ -391,7 +390,7 @@ void AppBannerInfoBarDelegateAndroid::OnWebApkInstallFinished(
return;
}
- UpdateStateForInstalledWebAPK(webapk_package_name);
+ UpdateStateForInstalledWebAPK(data.webapk_package_name);
webapk::TrackInstallEvent(webapk::INSTALL_COMPLETED);
}

Powered by Google App Engine
This is Rietveld 408576698