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

Unified Diff: chrome/browser/ui/ash/launcher/launcher_extension_app_updater.h

Issue 2819413003: Refactor extension app icon. (Closed)
Patch Set: 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/ash/launcher/launcher_extension_app_updater.h
diff --git a/chrome/browser/ui/ash/launcher/launcher_extension_app_updater.h b/chrome/browser/ui/ash/launcher/launcher_extension_app_updater.h
index a9588d05b3d068e2607424124bde4bc5ee8233ba..c44be090d2db349ee75e89d558134366f9691bb9 100644
--- a/chrome/browser/ui/ash/launcher/launcher_extension_app_updater.h
+++ b/chrome/browser/ui/ash/launcher/launcher_extension_app_updater.h
@@ -14,6 +14,8 @@ namespace extensions {
class ExtensionSet;
} // namespace extensions
+// TODO (khmel), this is not Launcher class. Consider moving this to proper
+// place.
class LauncherExtensionAppUpdater
: public LauncherAppUpdater,
public extensions::ExtensionRegistryObserver,
@@ -38,7 +40,8 @@ class LauncherExtensionAppUpdater
// ArcAppListPrefs::Observer
void OnPackageInstalled(
const arc::mojom::ArcPackageInfo& package_info) override;
- void OnPackageRemoved(const std::string& package_name) override;
+ void OnPackageRemoved(const std::string& package_name,
+ bool uninstalled) override;
private:
void StartObservingExtensionRegistry();

Powered by Google App Engine
This is Rietveld 408576698