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

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

Issue 2939773004: [Android WebAPK] Do not add padding to WebAPK primary icon (Closed)
Patch Set: Merge branch 'master' into reenable_tests0000000000 Created 3 years, 6 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/android/webapps/add_to_homescreen_data_fetcher.cc
diff --git a/chrome/browser/android/webapps/add_to_homescreen_data_fetcher.cc b/chrome/browser/android/webapps/add_to_homescreen_data_fetcher.cc
index 5cd67754eb479e05b205493d199d6ac4ff5ca946..e71e2f7cc6177a2a88a63310fbdcb19fc161b24f 100644
--- a/chrome/browser/android/webapps/add_to_homescreen_data_fetcher.cc
+++ b/chrome/browser/android/webapps/add_to_homescreen_data_fetcher.cc
@@ -250,7 +250,10 @@ void AddToHomescreenDataFetcher::OnDidPerformInstallableCheck(
if (data.primary_icon) {
shortcut_info_.best_primary_icon_url = data.primary_icon_url;
- CreateLauncherIcon(*(data.primary_icon));
+ if (webapk_compatible)
+ NotifyObserver(*data.primary_icon);
+ else
+ CreateLauncherIcon(*(data.primary_icon));
return;
}
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698