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

Unified Diff: chrome/browser/android/webapps/add_to_homescreen_manager.cc

Issue 2676863002: Update WebApkInstaller to support badge icon in installation. (Closed)
Patch Set: Revert git cl format 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/webapps/add_to_homescreen_manager.cc
diff --git a/chrome/browser/android/webapps/add_to_homescreen_manager.cc b/chrome/browser/android/webapps/add_to_homescreen_manager.cc
index 45b0459ecd3b0a54e23e33667f249893c27350e1..9c4ff9126732f1338cd1911e9ce81e4eb4fe9100 100644
--- a/chrome/browser/android/webapps/add_to_homescreen_manager.cc
+++ b/chrome/browser/android/webapps/add_to_homescreen_manager.cc
@@ -168,14 +168,13 @@ void AddToHomescreenManager::OnDataAvailable(const ShortcutInfo& info,
const SkBitmap& primary_icon,
const SkBitmap& badge_icon) {
if (is_webapk_compatible_) {
- // TODO(zpeng): Add badge to WebAPK installation flow.
WebApkInstallService* install_service =
WebApkInstallService::Get(
data_fetcher_->web_contents()->GetBrowserContext());
if (install_service->IsInstallInProgress(info.manifest_url))
ShortcutHelper::ShowWebApkInstallInProgressToast();
else
- CreateInfoBarForWebApk(info, primary_icon);
+ CreateInfoBarForWebApk(info, primary_icon, badge_icon);
return;
}
@@ -190,12 +189,16 @@ void AddToHomescreenManager::OnDataAvailable(const ShortcutInfo& info,
AddShortcut(info, primary_icon);
}
-void AddToHomescreenManager::CreateInfoBarForWebApk(const ShortcutInfo& info,
- const SkBitmap& icon) {
+void AddToHomescreenManager::CreateInfoBarForWebApk(
+ const ShortcutInfo& info,
+ const SkBitmap& primary_icon,
+ const SkBitmap& badge_icon) {
banners::AppBannerInfoBarDelegateAndroid::Create(
data_fetcher_->web_contents(), nullptr, info.user_title,
- base::MakeUnique<ShortcutInfo>(info), base::MakeUnique<SkBitmap>(icon),
- -1 /* event_request_id */, webapk::INSTALL_SOURCE_MENU);
+ base::MakeUnique<ShortcutInfo>(info),
+ base::MakeUnique<SkBitmap>(primary_icon),
+ base::MakeUnique<SkBitmap>(badge_icon), -1 /* event_request_id */,
+ webapk::INSTALL_SOURCE_MENU);
}
SkBitmap AddToHomescreenManager::FinalizeLauncherIconInBackground(

Powered by Google App Engine
This is Rietveld 408576698