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

Issue 754037 link

Starred by 3 users

Issue metadata

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



Sign in to add a comment

Don't show launcher after app is installed

Project Member Reported by warx@chromium.org, Aug 9 2017

Issue description

Don't show launcher after app is installed
 
Project Member

Comment 1 by bugdroid1@chromium.org, Aug 10 2017

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

commit 2c3f8bcda0d86796c4a621da65db80fda73270aa
Author: Qiang Xu <warx@chromium.org>
Date: Thu Aug 10 00:49:47 2017

app_list: do not show launcher after app is installed

Test: test with fullscreen app list flag
Bug:  754037 
Change-Id: I9ce62d3d608e0526aa62027665ecae374205abbe
Reviewed-on: https://chromium-review.googlesource.com/609122
Reviewed-by: Xiyuan Xia <xiyuan@chromium.org>
Commit-Queue: Qiang(Joe) Xu <warx@chromium.org>
Cr-Commit-Position: refs/heads/master@{#493196}
[modify] https://crrev.com/2c3f8bcda0d86796c4a621da65db80fda73270aa/chrome/browser/ui/ash/app_list/app_list_service_ash.cc

Comment 2 by warx@chromium.org, Aug 10 2017

Labels: Merge-Request-61

Comment 3 Deleted

Comment 4 by ketakid@google.com, Aug 10 2017

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

Comment 5 by bugdroid1@chromium.org, Aug 11 2017

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

commit b2f6449463726ecf7ee56428086fd29e6eefd724
Author: Maajid <maajid@chromium.org>
Date: Fri Aug 11 17:11:14 2017

app_list: do not show launcher after app is installed

TBR=warx@chromium.org

(cherry picked from commit 2c3f8bcda0d86796c4a621da65db80fda73270aa)

Test: test with fullscreen app list flag
Bug:  754037 
Change-Id: I9ce62d3d608e0526aa62027665ecae374205abbe
Reviewed-on: https://chromium-review.googlesource.com/609122
Reviewed-by: Xiyuan Xia <xiyuan@chromium.org>
Commit-Queue: Qiang(Joe) Xu <warx@chromium.org>
Cr-Original-Commit-Position: refs/heads/master@{#493196}
Reviewed-on: https://chromium-review.googlesource.com/612110
Reviewed-by: Qiang(Joe) Xu <warx@chromium.org>
Cr-Commit-Position: refs/branch-heads/3163@{#489}
Cr-Branched-From: ff259bab28b35d242e10186cd63af7ed404fae0d-refs/heads/master@{#488528}
[modify] https://crrev.com/b2f6449463726ecf7ee56428086fd29e6eefd724/chrome/browser/ui/ash/app_list/app_list_service_ash.cc

Comment 6 by warx@chromium.org, Aug 11 2017

Status: Fixed (was: Assigned)
Status: Verified (was: Fixed)
9765.32.0, 61.0.3163.53

Sign in to add a comment