arc: Support UNINSTALL_SHORTCUT |
||||||
Issue descriptionCurrently UNINSTALL_SHORTCUT is not support in ARC. This is also requested by Crossover team. Inner bug: b/34749139
,
May 6 2017
,
May 6 2017
Your change meets the bar and is auto-approved for M59. Please go ahead and merge the CL to branch 3071 manually. Please contact milestone owner if you have questions. Owners: amineer@(Android), cmasso@(iOS), gkihumba@(ChromeOS), Abdul Syed@(Desktop) For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
May 8 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/2fced5a6012d10701e8a5e2d6901949fa035fa1a commit 2fced5a6012d10701e8a5e2d6901949fa035fa1a Author: khmel <khmel@chromium.org> Date: Mon May 08 20:00:06 2017 [Merge M59] arc: Support UNINSTALL_SHORTCUT This adds handling UNINSTALL_SHORTCUT intent. BUG= 717661 BUG=b/34749139 TEST=Manually with test app. Shortcuts added and removed properly. Extended unit_tests TBR=xiyuan@chromium.org,lhchavez@chromium.org,nasko@chromium.org NOTRY=true NOPRESUBMIT=true Review-Url: https://codereview.chromium.org/2858703002 Cr-Commit-Position: refs/heads/master@{#469063} (cherry picked from commit cf0b40a80365c1c0de0d70e785ae91a26e547f74) Review-Url: https://codereview.chromium.org/2861403003 Cr-Commit-Position: refs/branch-heads/3071@{#459} Cr-Branched-From: a106f0abbf69dad349d4aaf4bcc4f5d376dd2377-refs/heads/master@{#464641} [modify] https://crrev.com/2fced5a6012d10701e8a5e2d6901949fa035fa1a/chrome/browser/ui/app_list/arc/arc_app_list_prefs.cc [modify] https://crrev.com/2fced5a6012d10701e8a5e2d6901949fa035fa1a/chrome/browser/ui/app_list/arc/arc_app_list_prefs.h [modify] https://crrev.com/2fced5a6012d10701e8a5e2d6901949fa035fa1a/chrome/browser/ui/app_list/arc/arc_app_unittest.cc [modify] https://crrev.com/2fced5a6012d10701e8a5e2d6901949fa035fa1a/components/arc/common/app.mojom [modify] https://crrev.com/2fced5a6012d10701e8a5e2d6901949fa035fa1a/components/arc/test/fake_app_instance.cc [modify] https://crrev.com/2fced5a6012d10701e8a5e2d6901949fa035fa1a/components/arc/test/fake_app_instance.h
,
May 8 2017
,
May 12 2017
khmel@ Trying to verify this bug. Could you please provide verification steps?
,
May 12 2017
I used test App I wrote to test this feature. ArcHelloAndroid.apk Once started it creates 5 shortcuts. It has button "Uninstall". Once clicked one shortcut should be removed.
,
May 12 2017
Thanks khmel@. Verified on TOT 9542.0.0, 60.0.3096.0 using above steps. |
||||||
►
Sign in to add a comment |
||||||
Comment 1 by bugdroid1@chromium.org
, May 3 2017