Filter out Playstore results that have corresponding Web Apps installed |
||||||||
Issue descriptionLike: You have Google Keep Web App installed. You type Keep in Launcher. Playstore results shouldn't include Google Keep.
,
Aug 8 2017
Recently used instant apps will be treated as on-device ones, and will never be filtered out, even when where is an equivalent installed extension.
,
Aug 8 2017
,
Aug 8 2017
,
Aug 8 2017
This bug requires manual review: M61 has already been promoted to the beta branch, so this requires manual review Please contact the milestone owner if you have questions. Owners: amineer@(Android), cmasso@(iOS), ketakid@(ChromeOS), govind@(Desktop) For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Aug 8 2017
Approving merge to M61 Chrome OS.
,
Aug 8 2017
LGTM re: #2
,
Aug 10 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/8faad1ed9e732eff254d2c01315ceb0fd668b415 commit 8faad1ed9e732eff254d2c01315ceb0fd668b415 Author: Vadim Tryshev <vadimt@google.com> Date: Thu Aug 10 20:26:35 2017 Filtering out Playstore results that have Web Apps. If a Playstore result has equivalent extension already installed, skip this result. Bug: 753564 Test: Modified ArcPlayStoreSearchProviderTest.Basic test Change-Id: I30edbf6c6b3c02ca92a11c93e4de08a54ffb01e0 Reviewed-on: https://chromium-review.googlesource.com/607291 Reviewed-by: Yusuke Sato <yusukes@chromium.org> Reviewed-by: Greg Kerr <kerrnel@chromium.org> Reviewed-by: Yury Khmel <khmel@chromium.org> Commit-Queue: Vadim Tryshev <vadimt@chromium.org> Cr-Commit-Position: refs/heads/master@{#493512} [modify] https://crrev.com/8faad1ed9e732eff254d2c01315ceb0fd668b415/chrome/browser/ui/app_list/search/arc/arc_playstore_search_provider.cc [modify] https://crrev.com/8faad1ed9e732eff254d2c01315ceb0fd668b415/chrome/browser/ui/app_list/search/arc/arc_playstore_search_provider_unittest.cc [modify] https://crrev.com/8faad1ed9e732eff254d2c01315ceb0fd668b415/chrome/browser/ui/app_list/search/search_controller_factory.cc [modify] https://crrev.com/8faad1ed9e732eff254d2c01315ceb0fd668b415/components/arc/common/app.mojom [modify] https://crrev.com/8faad1ed9e732eff254d2c01315ceb0fd668b415/components/arc/test/fake_app_instance.cc
,
Aug 11 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/3cbfd01de79890a87172dd7be90c2326c4abc9f9 commit 3cbfd01de79890a87172dd7be90c2326c4abc9f9 Author: Vadim Tryshev <vadimt@google.com> Date: Fri Aug 11 17:47:02 2017 Filtering out Playstore results that have Web Apps. If a Playstore result has equivalent extension already installed, skip this result. TBR=vadimt@google.com (cherry picked from commit 8faad1ed9e732eff254d2c01315ceb0fd668b415) Bug: 753564 Test: Modified ArcPlayStoreSearchProviderTest.Basic test Change-Id: I30edbf6c6b3c02ca92a11c93e4de08a54ffb01e0 Reviewed-on: https://chromium-review.googlesource.com/607291 Reviewed-by: Yusuke Sato <yusukes@chromium.org> Reviewed-by: Greg Kerr <kerrnel@chromium.org> Reviewed-by: Yury Khmel <khmel@chromium.org> Commit-Queue: Vadim Tryshev <vadimt@chromium.org> Cr-Original-Commit-Position: refs/heads/master@{#493512} Reviewed-on: https://chromium-review.googlesource.com/611672 Reviewed-by: Vadim Tryshev <vadimt@chromium.org> Cr-Commit-Position: refs/branch-heads/3163@{#492} Cr-Branched-From: ff259bab28b35d242e10186cd63af7ed404fae0d-refs/heads/master@{#488528} [modify] https://crrev.com/3cbfd01de79890a87172dd7be90c2326c4abc9f9/chrome/browser/ui/app_list/search/arc/arc_playstore_search_provider.cc [modify] https://crrev.com/3cbfd01de79890a87172dd7be90c2326c4abc9f9/chrome/browser/ui/app_list/search/arc/arc_playstore_search_provider_unittest.cc [modify] https://crrev.com/3cbfd01de79890a87172dd7be90c2326c4abc9f9/chrome/browser/ui/app_list/search/search_controller_factory.cc [modify] https://crrev.com/3cbfd01de79890a87172dd7be90c2326c4abc9f9/components/arc/common/app.mojom [modify] https://crrev.com/3cbfd01de79890a87172dd7be90c2326c4abc9f9/components/arc/test/fake_app_instance.cc
,
Aug 14 2017
,
Jan 22 2018
|
||||||||
►
Sign in to add a comment |
||||||||
Comment 1 by vadimt@chromium.org
, Aug 8 2017