New issue
Advanced search Search tips

Issue 674656 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Closed: Jan 2017
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug



Sign in to add a comment

Installation of app causes the Chrome Launcher to show up

Project Member Reported by khmel@chromium.org, Dec 15 2016

Issue description

Inner bug: b/33420574
 
Project Member

Comment 1 by bugdroid1@chromium.org, Dec 21 2016

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

commit 09a187e794db3a8cd261d7652658e51864a5a64d
Author: khmel <khmel@chromium.org>
Date: Wed Dec 21 23:35:02 2016

arc: Prevent App list popping on each app installed from batch.

There is possibility to manually install packages in batch mode.
In this mode App list is shown on each package installation. This
is annoying for user. This CL show App launcher only for first
installed in batch app. If user does not close App list he may
observer how next apps appear in list. If he closes it, now App
list will be shown unless next batch installation is started.
This also introduces new mojo notifications to support installation
sessions from Play Store. These notifications are used in current
task and will be used in next planned task to detect the case
when default app is not available after Arc start. There is yet
another idea to implement installation progress in App launcher.

TEST=Manually on device.
TEST=unit_tests + extended browser_tests.
TBR=skuhne@chromium.org
BUG=b/33420574
BUG= 674656 

Review-Url: https://codereview.chromium.org/2585503002
Cr-Commit-Position: refs/heads/master@{#440262}

[modify] https://crrev.com/09a187e794db3a8cd261d7652658e51864a5a64d/chrome/browser/ui/app_list/arc/arc_app_list_prefs.cc
[modify] https://crrev.com/09a187e794db3a8cd261d7652658e51864a5a64d/chrome/browser/ui/app_list/arc/arc_app_list_prefs.h
[modify] https://crrev.com/09a187e794db3a8cd261d7652658e51864a5a64d/chrome/browser/ui/ash/launcher/arc_app_launcher_browsertest.cc
[modify] https://crrev.com/09a187e794db3a8cd261d7652658e51864a5a64d/components/arc/common/app.mojom

Comment 2 by khmel@chromium.org, Jan 11 2017

Status: Fixed (was: Started)
Status: Verified (was: Fixed)

Sign in to add a comment