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

Unified Diff: chrome/browser/ui/app_list/arc/arc_app_list_prefs.h

Issue 2819413003: Refactor extension app icon. (Closed)
Patch Set: Devlin's comments Created 3 years, 8 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/ui/app_list/arc/arc_app_list_prefs.h
diff --git a/chrome/browser/ui/app_list/arc/arc_app_list_prefs.h b/chrome/browser/ui/app_list/arc/arc_app_list_prefs.h
index 05d9b13180513a6e8081e7f6e8ab44a4d69d3d55..8af3f097650d884bf3afa627d42c194a41309c94 100644
--- a/chrome/browser/ui/app_list/arc/arc_app_list_prefs.h
+++ b/chrome/browser/ui/app_list/arc/arc_app_list_prefs.h
@@ -138,8 +138,12 @@ class ArcAppListPrefs
// Notifies that package has been modified.
virtual void OnPackageModified(
const arc::mojom::ArcPackageInfo& package_info) {}
- // Notifies that package has been uninstalled.
- virtual void OnPackageRemoved(const std::string& package_name) {}
+ // Notifies that package has been removed from the system. |uninstalled| is
+ // set to true in case package was uninstalled by user or sync.
+ // OnPackageRemoved is called for each active package with |uninstalled| is
+ // set to false in case the user opts out the Play Store.
+ virtual void OnPackageRemoved(const std::string& package_name,
+ bool uninstalled) {}
// Notifies sync date type controller the model is ready to start.
virtual void OnPackageListInitialRefreshed() {}
@@ -303,7 +307,8 @@ class ArcAppListPrefs
// returns currently installed packages. If not, returns list of packages that
// where uninstalled. Note, we store uninstall packages only for packages of
// default apps.
- std::vector<std::string> GetPackagesFromPrefs(bool installed) const;
+ std::vector<std::string> GetPackagesFromPrefs(bool check_arc_alive,
+ bool installed) const;
void AddApp(const arc::mojom::AppInfo& app_info);
void AddAppAndShortcut(bool app_ready,
@@ -325,7 +330,7 @@ class ArcAppListPrefs
const std::string& package_name);
void DisableAllApps();
- void RemoveAllApps();
+ void RemoveAllAppsAndPackages();
std::vector<std::string> GetAppIdsNoArcEnabledCheck() const;
// Enumerates apps from preferences and notifies listeners about available
// apps while ARC is not started yet. All apps in this case have disabled

Powered by Google App Engine
This is Rietveld 408576698