commit | 30bc0ecfcdad79f652eab10c7cda9adca33a04fa | [log] [tgz] |
---|---|---|
author | Eric Shienbrood <ers@chromium.org> | Wed Mar 21 18:19:46 2012 -0400 |
committer | Gerrit <chrome-bot@google.com> | Tue Mar 27 14:06:40 2012 -0700 |
tree | 570e6c7e050eab8fb9227d343900e5f5d476a2ac | |
parent | 271fe5247103d0b76c48796c6f10ba45c6db59c4 [diff] |
shill: Make cellular Connect work for GSM networks. Add the APN to the properties map passed to Connect. Set up the list of APNs to try, and step through it when a connect attempt fails with the InvalidApn error. The order in which APNs are tried is - APN which most recently resulted in a successful connection on the current network - a user-specified APN, specified by setting the Cellular.APN property on the service - the list of APNs found for the home network provider in the mobile broadband provider database - if all those fail, a null APN BUG=chromium-os:23259 TEST=manual testing, some of which involved modifying the mobile provider DB to create invalid APN entries. Created two new unit tests as well, and ran all unit tests. Change-Id: I38c869228fe1aaf7421de8a826c54e7b62c209b2 Reviewed-on: https://gerrit.chromium.org/gerrit/19122 Tested-by: Eric Shienbrood <ers@chromium.org> Reviewed-by: Jason Glasgow <jglasgow@chromium.org> Commit-Ready: Eric Shienbrood <ers@chromium.org>