commit | 0b95044595cb87a71a7ab054e6b53b50d2947c5f | [log] [tgz] |
---|---|---|
author | Paul Stewart <pstew@chromium.org> | Tue Sep 11 13:10:08 2012 -0700 |
committer | Gerrit <chrome-bot@google.com> | Fri Sep 14 14:46:14 2012 -0700 |
tree | 58e61aae8b81bff33141305cd53148f91659b21d | |
parent | f1961f836f8f6fc8afaa5b97872b45d31cd081f6 [diff] |
shill: WiFiService: Don't ignore Passphrase for Configure Do not ignore the Passphrase parameter when configuring a WiFi service. This allows the Manager::ConfigureService to change the passphrase on a service which is found by GUID. Conversely, we should not directly set the passphrase for a WiFi service in WiFi::GetService, and instead rely on the Configure call to handle applying the passphrase. We can now make WiFiService::SetPassphrase private, since there are no remaining non-unittest consumers for this method. BUG=chromium-os:34004 TEST=Unit-tests, small modification network_WiFiManager/095_ProfileGUID Full WiFiManager suite. Change-Id: Ie23c1b03dee89aea54a4a86c46734165488a885b Reviewed-on: https://gerrit.chromium.org/gerrit/32997 Commit-Ready: Paul Stewart <pstew@chromium.org> Reviewed-by: Paul Stewart <pstew@chromium.org> Tested-by: Paul Stewart <pstew@chromium.org>