Have ExtensionAppModelBuilder call RemoveApp
There's no behavior change, other than removing a DVLOG(2) message. But
calling AppListModelBuilder's (the superclass') RemoveApp makes the
ExtensionAppModelBuilder consistent with the other AppListModelBuilder
sub-classes' methods (which all call AppListModelBuilder::RemoveApp):
- AppServiceAppModelBuilder::OnAppUpdate
- ArcAppModelBuilder::RemoveApp
- CrostiniAppModelBuilder::OnRegistryUpdated
Change-Id: Ic77e49f3e30125031796d40b629550326275da92
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1705480
Reviewed-by: Steven Bennetts <stevenjb@chromium.org>
Commit-Queue: Nigel Tao <nigeltao@chromium.org>
Cr-Commit-Position: refs/heads/master@{#678926}
diff --git a/chrome/browser/ui/app_list/extension_app_model_builder.cc b/chrome/browser/ui/app_list/extension_app_model_builder.cc
index 4545a2332..5a4aa47 100644
--- a/chrome/browser/ui/app_list/extension_app_model_builder.cc
+++ b/chrome/browser/ui/app_list/extension_app_model_builder.cc
@@ -149,12 +149,8 @@
void ExtensionAppModelBuilder::OnAppUninstalled(
content::BrowserContext* browser_context,
const std::string& app_id) {
- if (service()) {
- DVLOG(2) << service() << ": OnAppUninstalled: " << app_id.substr(0, 8);
- service()->RemoveUninstalledItem(app_id);
- return;
- }
- model_updater()->RemoveUninstalledItem(app_id);
+ const bool unsynced_change = false;
+ RemoveApp(app_id, unsynced_change);
}
void ExtensionAppModelBuilder::OnDisabledExtensionUpdated(
