Commit f56337f3 authored by Melissa Zhang's avatar Melissa Zhang Committed by Commit Bot

Implement ArcApps::Uninstall.

This CL implements the uninstall method in arc_apps.

BUG=916380

Change-Id: I2ea78604d3eeee68f72f1c5985eb6bc71371090d
Reviewed-on: https://chromium-review.googlesource.com/c/1451497
Commit-Queue: Melissa Zhang <melzhang@google.com>
Reviewed-by: default avatarNigel Tao <nigeltao@chromium.org>
Reviewed-by: default avatarDominick Ng <dominickn@chromium.org>
Cr-Commit-Position: refs/heads/master@{#630686}
parent 4f92a815
...@@ -255,7 +255,21 @@ void ArcApps::SetPermission(const std::string& app_id, ...@@ -255,7 +255,21 @@ void ArcApps::SetPermission(const std::string& app_id,
} }
void ArcApps::Uninstall(const std::string& app_id) { void ArcApps::Uninstall(const std::string& app_id) {
NOTIMPLEMENTED(); const std::unique_ptr<ArcAppListPrefs::AppInfo> app_info =
prefs_->GetApp(app_id);
if (!app_info) {
LOG(ERROR) << "Uninstall failed, could not find app with id " << app_id;
return;
}
auto* instance = ARC_GET_INSTANCE_FOR_METHOD(prefs_->app_connection_holder(),
UninstallPackage);
if (!instance) {
LOG(ERROR) << "Uninstall failed, could not find instance";
return;
}
instance->UninstallPackage(app_info->package_name);
} }
void ArcApps::OpenNativeSettings(const std::string& app_id) { void ArcApps::OpenNativeSettings(const std::string& app_id) {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment