Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(307)

Issue 9296009: Fix WiFi & VPN connection dialogs when controls a disabled through policy. (Closed)

Created:
8 years, 11 months ago by Mattias Nissler (ping if slow)
Modified:
8 years, 10 months ago
Reviewers:
stevenjb, Charlie Lee
CC:
chromium-reviews, stevenjb+watch_chromium.org, nkostylev+watch_chromium.org, davemoore+watch_chromium.org
Visibility:
Public.

Description

Fix WiFi & VPN connection dialogs when controls a disabled through policy. Previously, the UI would make decision based on the enabled() state of controls, indicating that the control is not relevant for the currently chosen configuration. This doesn't work for policy-configured networks, since the control might just be locked down, in which case it is relevant, but disabled. BUG=chromium-os:23124 TEST=Configure ONC networks through policy, observe that the connection dialogs show the configured values even though disabled. Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=119664

Patch Set 1 #

Total comments: 2

Patch Set 2 : Address nit, rebase #

Unified diffs Side-by-side diffs Delta from patch set Stats (+98 lines, -68 lines) Patch
M chrome/browser/chromeos/options/vpn_config_view.cc View 4 chunks +4 lines, -4 lines 0 comments Download
M chrome/browser/chromeos/options/wifi_config_view.h View 2 chunks +13 lines, -1 line 0 comments Download
M chrome/browser/chromeos/options/wifi_config_view.cc View 1 12 chunks +81 lines, -63 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
Mattias Nissler (ping if slow)
Can you review this? It's basically some UI massaging for options locked down through policy. ...
8 years, 11 months ago (2012-01-27 15:22:15 UTC) #1
stevenjb
lgtm http://codereview.chromium.org/9296009/diff/1/chrome/browser/chromeos/options/wifi_config_view.cc File chrome/browser/chromeos/options/wifi_config_view.cc (right): http://codereview.chromium.org/9296009/diff/1/chrome/browser/chromeos/options/wifi_config_view.cc#newcode937 chrome/browser/chromeos/options/wifi_config_view.cc:937: passphrase_label_->SetEnabled(wifi); nit: I'd use (wifi ? true : ...
8 years, 11 months ago (2012-01-27 19:14:07 UTC) #2
Mattias Nissler (ping if slow)
8 years, 10 months ago (2012-01-30 12:56:49 UTC) #3
http://codereview.chromium.org/9296009/diff/1/chrome/browser/chromeos/options...
File chrome/browser/chromeos/options/wifi_config_view.cc (right):

http://codereview.chromium.org/9296009/diff/1/chrome/browser/chromeos/options...
chrome/browser/chromeos/options/wifi_config_view.cc:937:
passphrase_label_->SetEnabled(wifi);
On 2012/01/27 19:14:07, stevenjb (chromium) wrote:
> nit: I'd use (wifi ? true : false) to make it clear that a bool is getting
> passed here.

Switched it to wifi != NULL.

Powered by Google App Engine
This is Rietveld 408576698