commit | e7c7e65bba9e964aa500ef53cc79215426acf50c | [log] [tgz] |
---|---|---|
author | mukesh agrawal <quiche@chromium.org> | Tue Jun 18 17:19:39 2013 -0700 |
committer | ChromeBot <chrome-bot@google.com> | Wed Jun 19 15:22:53 2013 -0700 |
tree | 47702962b2c9dbf192d408203fa3a58e8bf38ef0 | |
parent | ee293000e73e5cd4915cd32c123d2530dc231ed1 [diff] |
shill: add FrequencyList property to WiFiServices The main intended change in this CL is to expose the known frequencies for a WiFi network via D-Bus properties. This required adding a new property type (array of uint16). While there, though, I made some changes to the DBusAdaptor and PropertyStore unit tests. These changes should make it easier to identify and update the relevant unit tests when we add new property types. Specifically: - add a GetProperties test to dbus_adaptor_unittest.cc - add a GetProperty test to property_store_unittest.cc - make the list of PropertyTypes accessible from outside property_store_unittest - make the RegisterProperty shims accessible from outside property_store_unittest - create GetProperty shims, and make them accessible as well Additional changes: - remove some unnecessary (as of c++11) whitespace between angle brackets - make initialization of DBusAdaptorTest fields more uniform, by using uniform initialization syntax and initializer lists - update service-api documentation for WiFi.Frequency BUG=chromium:248791 TEST=unit tests, manual CQ-DEPEND=CL:59119 Manual test ----------- 1. chrome://system 2. network-services -> Expand... 3. find GoogleGuest, check that it has at least one entry for "WiFi.FrequencyList". e.g., "WiFi.FrequencyList/0: 2414" Change-Id: Ie3b4d89853e22b7c4e80ad017738223d9753dd7c Reviewed-on: https://gerrit.chromium.org/gerrit/59149 Commit-Queue: mukesh agrawal <quiche@chromium.org> Reviewed-by: mukesh agrawal <quiche@chromium.org> Tested-by: mukesh agrawal <quiche@chromium.org>