New issue
Advanced search Search tips

Issue 640226 link

Starred by 5 users

Issue metadata

Status: Verified
Owner:
Closed: Sep 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug



Sign in to add a comment

arc: Open launcher after installing app from Android

Project Member Reported by khmel@chromium.org, Aug 23 2016

Issue description

Inner bug: b/30953079

Open launcher after installing app from Android so users know where they can find the app

 
Project Member

Comment 1 by bugdroid1@chromium.org, Aug 24 2016

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

commit 29f5caf832d9e2e14c1ad5de1ab6c94cc5c41e33
Author: khmel <khmel@chromium.org>
Date: Wed Aug 24 17:55:03 2016

arc: Open launcher after installing app from Android

This Implements showing app list when user installs new package.
App list is not shown for app from system packages, or auxulary
packages. App list is shown on the page that contains newly
installed app.

BUG=b/30953079
BUG= 640226 
TEST=unit_tests extended browser_tests
TEST=manually on device

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

[modify] https://crrev.com/29f5caf832d9e2e14c1ad5de1ab6c94cc5c41e33/chrome/browser/ui/app_list/arc/arc_app_list_prefs.cc
[modify] https://crrev.com/29f5caf832d9e2e14c1ad5de1ab6c94cc5c41e33/chrome/browser/ui/app_list/arc/arc_app_list_prefs.h
[modify] https://crrev.com/29f5caf832d9e2e14c1ad5de1ab6c94cc5c41e33/chrome/browser/ui/app_list/arc/arc_package_syncable_service.cc
[modify] https://crrev.com/29f5caf832d9e2e14c1ad5de1ab6c94cc5c41e33/chrome/browser/ui/app_list/arc/arc_package_syncable_service.h
[modify] https://crrev.com/29f5caf832d9e2e14c1ad5de1ab6c94cc5c41e33/chrome/browser/ui/ash/launcher/arc_app_launcher_browsertest.cc
[modify] https://crrev.com/29f5caf832d9e2e14c1ad5de1ab6c94cc5c41e33/ui/app_list/views/app_list_main_view.cc
[modify] https://crrev.com/29f5caf832d9e2e14c1ad5de1ab6c94cc5c41e33/ui/app_list/views/app_list_main_view.h

Comment 2 by lgcheng@google.com, Aug 25 2016

Cc: lgcheng@google.com

Comment 3 by khmel@chromium.org, Aug 29 2016

Labels: Merge-Request-53

Comment 4 by dimu@chromium.org, Aug 29 2016

Labels: -Merge-Request-53 Merge-Review-53 Hotlist-Merge-Review
[Automated comment] Less than 2 weeks to go before stable on M53, manual review required.
Labels: -Merge-Review-53 Merge-Approved-53
Approving merge to M53.
Project Member

Comment 6 by bugdroid1@chromium.org, Sep 2 2016

Labels: -merge-approved-53 merge-merged-2785
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/532a5e0c63f7e01f23970d0604a9edadb0d6245d

commit 532a5e0c63f7e01f23970d0604a9edadb0d6245d
Author: khmel <khmel@chromium.org>
Date: Fri Sep 02 20:06:31 2016

arc: Open launcher after installing app from Android

This Implements showing app list when user installs new package.
App list is not shown for app from system packages, or auxulary
packages. App list is shown on the page that contains newly
installed app.

BUG=b/30953079
BUG= 640226 
TEST=unit_tests extended browser_tests
TEST=manually on device
TBR=reviewer@chromium.org
NOTRY=true
NOPRESUBMIT=true

Review-Url: https://codereview.chromium.org/2276553002
Cr-Commit-Position: refs/heads/master@{#414100}
(cherry picked from commit 29f5caf832d9e2e14c1ad5de1ab6c94cc5c41e33)

Review-Url: https://codereview.chromium.org/2304233002
Cr-Commit-Position: refs/branch-heads/2785@{#815}
Cr-Branched-From: 68623971be0cfc492a2cb0427d7f478e7b214c24-refs/heads/master@{#403382}

[modify] https://crrev.com/532a5e0c63f7e01f23970d0604a9edadb0d6245d/chrome/browser/ui/app_list/arc/arc_app_list_prefs.cc
[modify] https://crrev.com/532a5e0c63f7e01f23970d0604a9edadb0d6245d/chrome/browser/ui/app_list/arc/arc_app_list_prefs.h
[modify] https://crrev.com/532a5e0c63f7e01f23970d0604a9edadb0d6245d/chrome/browser/ui/app_list/arc/arc_package_syncable_service.cc
[modify] https://crrev.com/532a5e0c63f7e01f23970d0604a9edadb0d6245d/chrome/browser/ui/app_list/arc/arc_package_syncable_service.h
[modify] https://crrev.com/532a5e0c63f7e01f23970d0604a9edadb0d6245d/chrome/browser/ui/ash/launcher/arc_app_launcher_browsertest.cc
[modify] https://crrev.com/532a5e0c63f7e01f23970d0604a9edadb0d6245d/ui/app_list/views/app_list_main_view.cc
[modify] https://crrev.com/532a5e0c63f7e01f23970d0604a9edadb0d6245d/ui/app_list/views/app_list_main_view.h

Comment 7 by khmel@chromium.org, Sep 2 2016

Status: Fixed (was: Started)
Status: Verified (was: Fixed)
Chrome:53.0.2785.103/ChromeOS:8530.81.0/ARC:3251841 

Sign in to add a comment