tree 9a3d522a106e1996d77411588da78a7e88771ac0
parent bf01129bbfa74e86cf902f80eb1434e7c6b539b3
author Ian Barkley-Yeung <iby@google.com> 1550002735 -0800
committer chrome-bot <chrome-bot@chromium.org> 1550150852 -0800

Ensure Uninstall option appears for Linux apps

Fixes an issue where the Uninstall option would not appear after
installing some Linux apps. This was because the desktop file for the
app was created before the installation was finished. When garcon
noticed the new desktop and asked PackageKit if a package owned the
file, PackageKit said no because the owning package wasn't fully
installed.

Worked around this by adding a post-install hook to dpkg which will
touch a file in the /usr/share/applications directory after each install
is finished. garcon will notice the file change and rescan after the
install is done. The second scan should see that the desktop is owned by
the now-fully-installed package.

Alternatives considered: I tried to have garcon look at the dpkg lock
and only do the scan when dpkg wasn't in the middle of an operation.
This didn't work because garcon doesn't run as root and can't open the
dpkg lock file.

BUG=chromium:898295
TEST=Installed various packages, especially eclipse, both using apt-get
install and right-clicking the .deb file in Files app. Ensured uninstall
option appeared consistently.

Change-Id: I04605356afe7dfdd7bf9c70a4a3982e58b9dc8fa
Reviewed-on: https://chromium-review.googlesource.com/1468041
Commit-Ready: Ian Barkley-Yeung <iby@chromium.org>
Tested-by: kokoro <noreply+kokoro@google.com>
Reviewed-by: Stephen Barber <smbarber@chromium.org>