Pinned Crostini apps not removed on uninstall |
|||||
Issue descriptionUpon uninstall of Crostini apps (e.g. apt-get remove, or removing Crostini entirely), the app list is updated but apps pinned on the shelf remain.
,
May 9 2018
,
May 10 2018
,
May 14 2018
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/7c6293de2931fae672f1f2efdc4ba509a7d496b1 commit 7c6293de2931fae672f1f2efdc4ba509a7d496b1 Author: Timothy Loh <timloh@chromium.org> Date: Mon May 14 02:22:04 2018 Remove pinned Crostini apps on uninstall This CL adds a LauncherCrostiniAppUpdater that ensures the Shelf knows about app uninstallation so it can remove pinned Crostini apps. TEST=Manually uninstalled an app and saw the pin disappear. Manually uninstalled Crostini and saw pins disappear. Bug: 839699 Change-Id: I81e6b60414ad8766ef159a9ea3969809beb5112c Reviewed-on: https://chromium-review.googlesource.com/1046605 Reviewed-by: Stefan Kuhne <skuhne@chromium.org> Commit-Queue: Timothy Loh <timloh@chromium.org> Cr-Commit-Position: refs/heads/master@{#558175} [modify] https://crrev.com/7c6293de2931fae672f1f2efdc4ba509a7d496b1/chrome/browser/ui/BUILD.gn [modify] https://crrev.com/7c6293de2931fae672f1f2efdc4ba509a7d496b1/chrome/browser/ui/ash/launcher/chrome_launcher_controller.cc [add] https://crrev.com/7c6293de2931fae672f1f2efdc4ba509a7d496b1/chrome/browser/ui/ash/launcher/launcher_crostini_app_updater.cc [add] https://crrev.com/7c6293de2931fae672f1f2efdc4ba509a7d496b1/chrome/browser/ui/ash/launcher/launcher_crostini_app_updater.h
,
May 14 2018
,
May 14 2018
|
|||||
►
Sign in to add a comment |
|||||
Comment 1 by vapier@chromium.org
, May 9 2018