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

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: 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 fa237dbbe659b5f7b8d07d902d2d0e9f016dd1b5..d6a9545e32cc57eb0591c098623ddd49391f6a9d 100644
--- a/chrome/browser/android/webapps/add_to_homescreen_data_fetcher.h
+++ b/chrome/browser/android/webapps/add_to_homescreen_data_fetcher.h
@@ -30,6 +30,7 @@ class Message;
}
class GURL;
+class InstallableManager;
struct InstallableData;
struct WebApplicationInfo;
@@ -38,8 +39,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 {
@@ -102,6 +102,9 @@ class AddToHomescreenDataFetcher
void OnDataTimedout();
// Called when InstallableManager finishes looking for a manifest and icon.
+ void OnDidGetManifestAndIcon(const InstallableData& data);
+
+ // Called when InstallableManager finishes checking for installability.
void OnDidPerformInstallableCheck(const InstallableData& data);
// Grabs the favicon for the current URL.
@@ -124,6 +127,7 @@ class AddToHomescreenDataFetcher
scoped_refptr<base::TaskRunner> background_task_runner_;
+ InstallableManager* installable_manager_;
Observer* weak_observer_;
// The icons must only be set on the UI thread for thread safety.

Powered by Google App Engine
This is Rietveld 408576698