New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 753603 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Last visit > 30 days ago
Closed: Aug 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug



Sign in to add a comment

Touchable launcher:Clicking Install on Android app does not install it automatically

Project Member Reported by sdantul...@chromium.org, Aug 8 2017

Issue description

Google Chrome	61.0.3163.38 (Official Build) dev (64-bit)
Revision	0
Platform	9765.21.0 (Official Build) dev-channel

What steps will reproduce the problem?
1. Open launcher
2. Search for an Android app
3. From the Playstore results, right-click on desired app and click Install

What is the expected result?
Playstore app should open and app should install automatically

What happens instead?
Playstore app opens and need to click on Install again for the installation to start
 
Labels: crosuite-launcher

Comment 2 by vadimt@chromium.org, Aug 11 2017

Owner: hejq@chromium.org
Status: Assigned (was: Untriaged)

Comment 3 by vadimt@chromium.org, Aug 11 2017

Cc: omrilio@chromium.org
Need to remove this menu item.
Phonesky doesn't return an intent allowing direct installing.

Comment 4 by vadimt@chromium.org, Aug 15 2017

Labels: Merge-Request-61
Project Member

Comment 5 by sheriffbot@chromium.org, Aug 15 2017

Labels: -Merge-Request-61 Merge-Review-61 Hotlist-Merge-Review
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

Comment 6 by ketakid@google.com, Aug 15 2017

Labels: -Merge-Review-61 Merge-Approved-61
Approving merge to M61 Chrome OS.
Project Member

Comment 7 by bugdroid1@chromium.org, Aug 16 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/e9f82013e2bb578efd050689f3442ad9f27e95cf

commit e9f82013e2bb578efd050689f3442ad9f27e95cf
Author: Vadim Tryshev <vadimt@google.com>
Date: Wed Aug 16 00:15:54 2017

Disabling context menu for Play Store search results.

Currently Play Store API doesn't return both install and launch intents,
so it doesn't make sense to have a context menu that opens the result
like what happens upon clicking it.

Bug:  753603 
Change-Id: I4659eaeb0b89709afaec9e3ec242b5ee8e1bd6e4
Reviewed-on: https://chromium-review.googlesource.com/616022
Reviewed-by: Yury Khmel <khmel@chromium.org>
Commit-Queue: Vadim Tryshev <vadimt@chromium.org>
Cr-Commit-Position: refs/heads/master@{#494623}
[modify] https://crrev.com/e9f82013e2bb578efd050689f3442ad9f27e95cf/chrome/browser/ui/app_list/search/arc/arc_playstore_search_result.cc
[modify] https://crrev.com/e9f82013e2bb578efd050689f3442ad9f27e95cf/chrome/browser/ui/app_list/search/arc/arc_playstore_search_result.h

Project Member

Comment 8 by bugdroid1@chromium.org, Aug 16 2017

Labels: -merge-approved-61 merge-merged-3163
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/cc4ec89e9303866dabf5072222b3ebc37611a914

commit cc4ec89e9303866dabf5072222b3ebc37611a914
Author: Vadim Tryshev <vadimt@google.com>
Date: Wed Aug 16 17:38:23 2017

Disabling context menu for Play Store search results.

Currently Play Store API doesn't return both install and launch intents,
so it doesn't make sense to have a context menu that opens the result
like what happens upon clicking it.

TBR=vadimt@google.com

(cherry picked from commit e9f82013e2bb578efd050689f3442ad9f27e95cf)

Bug:  753603 
Change-Id: I4659eaeb0b89709afaec9e3ec242b5ee8e1bd6e4
Reviewed-on: https://chromium-review.googlesource.com/616022
Reviewed-by: Yury Khmel <khmel@chromium.org>
Commit-Queue: Vadim Tryshev <vadimt@chromium.org>
Cr-Original-Commit-Position: refs/heads/master@{#494623}
Reviewed-on: https://chromium-review.googlesource.com/617368
Reviewed-by: Vadim Tryshev <vadimt@chromium.org>
Cr-Commit-Position: refs/branch-heads/3163@{#600}
Cr-Branched-From: ff259bab28b35d242e10186cd63af7ed404fae0d-refs/heads/master@{#488528}
[modify] https://crrev.com/cc4ec89e9303866dabf5072222b3ebc37611a914/chrome/browser/ui/app_list/search/arc/arc_playstore_search_result.cc
[modify] https://crrev.com/cc4ec89e9303866dabf5072222b3ebc37611a914/chrome/browser/ui/app_list/search/arc/arc_playstore_search_result.h

Comment 9 by vadimt@chromium.org, Aug 16 2017

Status: Fixed (was: Assigned)
Fixed by removing context menu for Playstore search results
Status: Verified (was: Fixed)
9765.32.0, 61.0.3163.53

Sign in to add a comment