New issue
Advanced search Search tips

Issue 716688 link

Starred by 1 user

Issue metadata

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



Sign in to add a comment

"Add to preferred" network option still available when disallowed

Project Member Reported by michae...@chromium.org, Apr 29 2017

Issue description

1. chrome://md-settings
2. Wi-Fi
3. open subpage for Google-A
4. notice "Prefer this network" is disabled due to enterprise policy
5. back
6. open "Known networks" subpage
7. tap 3-dot menu next to Google-A

Expected: "Add to preferred" missing or disabled
Actual: "Add to preferred" is available. Clicking it closes the 3-dot menu but does nothing else.
 
Labels: -Pri-2 M-59 Pri-1
Status: Started (was: Assigned)
Thanks for noticing this!

Labels: Hotlist-MD-Settings-Internet
Labels: Merge-Request-59
Status: Fixed (was: Started)
Project Member

Comment 5 by sheriffbot@chromium.org, May 9 2017

Labels: -Merge-Request-59 Hotlist-Merge-Approved Merge-Approved-59
Your change meets the bar and is auto-approved for M59. Please go ahead and merge the CL to branch 3071 manually. Please contact milestone owner if you have questions.
Owners: amineer@(Android), cmasso@(iOS), gkihumba@(ChromeOS), Abdul Syed@(Desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Labels: -Hotlist-Merge-Approved
Project Member

Comment 7 by bugdroid1@chromium.org, May 12 2017

Labels: -merge-approved-59 merge-merged-3071
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/84b2e32bb3d7dd84b2ad094f2304bc0516dedd52

commit 84b2e32bb3d7dd84b2ad094f2304bc0516dedd52
Author: Steven Bennetts <stevenjb@chromium.org>
Date: Fri May 12 19:09:18 2017

MD Settings: Network: Check managed properties for known networks menu

BUG= 716688 
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2856863010
Cr-Original-Commit-Position: refs/heads/master@{#470112}
Review-Url: https://codereview.chromium.org/2879533004 .
Cr-Commit-Position: refs/branch-heads/3071@{#532}
Cr-Branched-From: a106f0abbf69dad349d4aaf4bcc4f5d376dd2377-refs/heads/master@{#464641}

[modify] https://crrev.com/84b2e32bb3d7dd84b2ad094f2304bc0516dedd52/chrome/browser/resources/settings/internet_page/compiled_resources2.gyp
[modify] https://crrev.com/84b2e32bb3d7dd84b2ad094f2304bc0516dedd52/chrome/browser/resources/settings/internet_page/internet_known_networks_page.html
[modify] https://crrev.com/84b2e32bb3d7dd84b2ad094f2304bc0516dedd52/chrome/browser/resources/settings/internet_page/internet_known_networks_page.js

Status: Verified (was: Fixed)
9460.42.0, 59.0.3071.57

Sign in to add a comment