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

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

Issue 2949993002: Don't ignore manifest icons for sites that don't have a service worker. (Closed)
Patch Set: Comments, clean up 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
Index: chrome/browser/android/webapps/add_to_homescreen_data_fetcher.h
diff --git a/chrome/browser/android/webapps/add_to_homescreen_data_fetcher.h b/chrome/browser/android/webapps/add_to_homescreen_data_fetcher.h
index 77679690d61eab35b03463ad41c6607702cc9e87..d93085e99f5ea072c7cbc58adfdc54fbeb6b9b41 100644
--- a/chrome/browser/android/webapps/add_to_homescreen_data_fetcher.h
+++ b/chrome/browser/android/webapps/add_to_homescreen_data_fetcher.h
@@ -19,6 +19,7 @@ struct FaviconRawBitmapResult;
}
class GURL;
+class InstallableManager;
struct InstallableData;
struct WebApplicationInfo;
@@ -27,8 +28,7 @@ struct WebApplicationInfo;
//
// Because of the various asynchronous calls made by this class, it is
// refcounted to prevent the class from being prematurely deleted. If the
-// pointer to the ShortcutHelper becomes invalid, the pipeline should kill
-// itself.
+// |weak_observer_| pointer becomes invalid, the pipeline should kill itself.
class AddToHomescreenDataFetcher
: public base::RefCounted<AddToHomescreenDataFetcher>,
public content::WebContentsObserver {
@@ -92,6 +92,9 @@ class AddToHomescreenDataFetcher
void OnDataTimedout();
// Called when InstallableManager finishes looking for a manifest and icon.
+ void OnDidGetManifestAndIcons(const InstallableData& data);
+
+ // Called when InstallableManager finishes checking for installability.
void OnDidPerformInstallableCheck(const InstallableData& data);
// Grabs the favicon for the current URL.
@@ -105,16 +108,18 @@ class AddToHomescreenDataFetcher
SkBitmap CreateLauncherIconFromFaviconInBackground(
const favicon_base::FaviconRawBitmapResult& bitmap_result);
- // Creates the launcher icon from the given |raw_icon|.
- void CreateLauncherIcon(const SkBitmap& raw_icon);
- SkBitmap CreateLauncherIconInBackground(const SkBitmap& raw_icon);
+ // Creates the primary launcher icon from the given |icon|.
+ void CreateLauncherIcon(const SkBitmap& icon);
+ SkBitmap CreateLauncherIconInBackground(const SkBitmap& icon);
// Notifies the observer that the shortcut data is all available.
void NotifyObserver(const SkBitmap& icon);
+ InstallableManager* installable_manager_;
Observer* weak_observer_;
// The icons must only be set on the UI thread for thread safety.
+ SkBitmap raw_primary_icon_;
SkBitmap badge_icon_;
SkBitmap primary_icon_;
ShortcutInfo shortcut_info_;

Powered by Google App Engine
This is Rietveld 408576698