New issue
Advanced search Search tips

Issue 648796 link

Starred by 3 users

Issue metadata

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



Sign in to add a comment

arc: PlayStore icon is shown for Arc managed (disabled) profile

Project Member Reported by khmel@chromium.org, Sep 20 2016

Issue description

Inner bug:
b/31577285
 

Comment 2 by khmel@chromium.org, Oct 5 2016

Labels: Merge-Request-54

Comment 3 by dimu@chromium.org, Oct 6 2016

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

Comment 5 by bugdroid1@chromium.org, Oct 18 2016

Labels: -merge-approved-54 merge-merged-2840
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/e5f09a72652bf4a361dc494229e331f8ca762969

commit e5f09a72652bf4a361dc494229e331f8ca762969
Author: khmel <khmel@chromium.org>
Date: Tue Oct 18 23:03:11 2016

[Merge-M54] arc: Don't show default Arc apps for managed users with Arc-disabled.

TEST=Manually on device. @chromium account has no PlayStore icon
     visible.
TEST=extended browser_tests
BUG=b/31577285
BUG= 648796 
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

TBR=reviewer@chromium.org
NOTRY=true
NOPRESUBMIT=true

Review-Url: https://codereview.chromium.org/2357763002
Review-Url: https://codereview.chromium.org/2433553002
Cr-Original-Commit-Position: refs/heads/master@{#423178}
Cr-Commit-Position: refs/branch-heads/2840@{#753}
Cr-Branched-From: 1ae106dbab4bddd85132d5b75c670794311f4c57-refs/heads/master@{#414607}

[modify] https://crrev.com/e5f09a72652bf4a361dc494229e331f8ca762969/chrome/browser/policy/policy_browsertest.cc
[modify] https://crrev.com/e5f09a72652bf4a361dc494229e331f8ca762969/chrome/browser/ui/app_list/arc/arc_app_list_prefs.cc
[modify] https://crrev.com/e5f09a72652bf4a361dc494229e331f8ca762969/chrome/browser/ui/app_list/arc/arc_app_list_prefs.h
[modify] https://crrev.com/e5f09a72652bf4a361dc494229e331f8ca762969/chrome/browser/ui/app_list/arc/arc_default_app_list.cc
[modify] https://crrev.com/e5f09a72652bf4a361dc494229e331f8ca762969/chrome/browser/ui/app_list/arc/arc_default_app_list.h

Comment 6 by khmel@chromium.org, Oct 18 2016

Labels: -Hotlist-Merge-review
Status: Fixed (was: Started)
Status: Verified (was: Fixed)
743.69.0/54.0.2840.68 
Project Member

Comment 9 by bugdroid1@chromium.org, Oct 27 2016

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

commit e5f09a72652bf4a361dc494229e331f8ca762969
Author: khmel <khmel@chromium.org>
Date: Tue Oct 18 23:03:11 2016

[Merge-M54] arc: Don't show default Arc apps for managed users with Arc-disabled.

TEST=Manually on device. @chromium account has no PlayStore icon
     visible.
TEST=extended browser_tests
BUG=b/31577285
BUG= 648796 
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

TBR=reviewer@chromium.org
NOTRY=true
NOPRESUBMIT=true

Review-Url: https://codereview.chromium.org/2357763002
Review-Url: https://codereview.chromium.org/2433553002
Cr-Original-Commit-Position: refs/heads/master@{#423178}
Cr-Commit-Position: refs/branch-heads/2840@{#753}
Cr-Branched-From: 1ae106dbab4bddd85132d5b75c670794311f4c57-refs/heads/master@{#414607}

[modify] https://crrev.com/e5f09a72652bf4a361dc494229e331f8ca762969/chrome/browser/policy/policy_browsertest.cc
[modify] https://crrev.com/e5f09a72652bf4a361dc494229e331f8ca762969/chrome/browser/ui/app_list/arc/arc_app_list_prefs.cc
[modify] https://crrev.com/e5f09a72652bf4a361dc494229e331f8ca762969/chrome/browser/ui/app_list/arc/arc_app_list_prefs.h
[modify] https://crrev.com/e5f09a72652bf4a361dc494229e331f8ca762969/chrome/browser/ui/app_list/arc/arc_default_app_list.cc
[modify] https://crrev.com/e5f09a72652bf4a361dc494229e331f8ca762969/chrome/browser/ui/app_list/arc/arc_default_app_list.h

Sign in to add a comment