1ee315fe16a7e91f43f1b24f5f2c0a47b52e3455 |
|
06-Jan-2016 |
Victor Dodon <printesoi@google.com> |
shill: Add ByteArray properties Bug: None BUG=chromium:574578 TEST=New unit tests Change-Id: Ifddfa995d1a8ab8f9d9e82b4a897ce51c92f092b
/system/connectivity/shill/accessor_interface.h
|
c0beca55d290fe0b1c96d78cbbcf94b05c23f5a5 |
|
03-Sep-2015 |
Peter Qiu <zqiu@google.com> |
License shill to Apache2 This patch automatically replaced the license on all text files from Chromium OS (BSD style) to AOSP (Apache2), keeping the original year as a reference. The license header was added to .gyp and .gypi, the NOTICE was added with a copy of the Apache2 license and MODULE_LICENSE_* file was added. BUG=23587594 TEST=grep 'Chromium OS Authors' doesn't find anything. Change-Id: If41ede586c2465f5831fb7fee270ff41dbfdb596
/system/connectivity/shill/accessor_interface.h
|
a794cd60a7339d576ea2eed263a4f0a20fb255af |
|
16-Jun-2015 |
Paul Stewart <pstew@chromium.org> |
shill: Top-level files a-e: Switch pointer/ref spacing Switch to Chrome style spacing. BUG=chromium:501026 TEST=Unit tests Change-Id: Idb7252202e142098999fe0672712aded7d369259 Reviewed-on: https://chromium-review.googlesource.com/278038 Reviewed-by: Rebecca Silberstein <silberst@chromium.org> Commit-Queue: Paul Stewart <pstew@chromium.org> Tested-by: Paul Stewart <pstew@chromium.org>
/system/connectivity/shill/accessor_interface.h
|
cc67c52a2c00f90e877971d552208dd99825d84e |
|
03-Sep-2014 |
Ben Chan <benchan@chromium.org> |
shill: Include base/macros.h instead of base/basictypes.h shill no longer uses the integer types from base/basictypes.h. It should simply include base/macros.h for the DISALLOW_COPY_AND_ASSIGN macro instead. BUG=None TEST=`USE='cellular wimax' FEATURES=test emerge-$BOARD platform2` Change-Id: I9b350c1bd9eff88e2c87de2ef8d607e90049e102 Reviewed-on: https://chromium-review.googlesource.com/216055 Reviewed-by: Paul Stewart <pstew@chromium.org> Commit-Queue: Ben Chan <benchan@chromium.org> Tested-by: Ben Chan <benchan@chromium.org>
/system/connectivity/shill/accessor_interface.h
|
7fab89734d88724a288e96a9996b15548c5294c7 |
|
11-Aug-2014 |
Ben Chan <benchan@chromium.org> |
shill: Use integer types from stdint.h This CL replaces the deprecated int* and uint* types from 'base/basictypes.h' with the int*_t and uint*_t types from 'stdint.h'. BUG=chromium:401356 TEST=`USE='cellular gdmwimax wimax' FEATURES=test emerge-$BOARD platform2` Change-Id: I3d4c195881203dd2a47dbb5af150b6c90b9c206e Reviewed-on: https://chromium-review.googlesource.com/211770 Reviewed-by: Mike Frysinger <vapier@chromium.org> Commit-Queue: Ben Chan <benchan@chromium.org> Tested-by: Ben Chan <benchan@chromium.org>
/system/connectivity/shill/accessor_interface.h
|
a41ab517725d036b63420f8445550246f8f50b99 |
|
23-Jul-2014 |
Alex Vakulenko <avakulenko@chromium.org> |
shill: Sort headers alphabetically (build/include_alpha) We are going to enable build/include_alpha linter warning soon, so in preparation for this, fixed the warnings in shill. BUG=None TEST=cpplint.py --filter=-build/include_order,+build/include_alpha shill/* Change-Id: I113ed7f9dd3f465a3f660599320cdb434e7cc214 Reviewed-on: https://chromium-review.googlesource.com/209636 Tested-by: Alex Vakulenko <avakulenko@chromium.org> Reviewed-by: Ben Chan <benchan@chromium.org> Commit-Queue: Alex Vakulenko <avakulenko@chromium.org>
/system/connectivity/shill/accessor_interface.h
|
c45688bb3881f0c2216e6ec0e19ebda0be33e871 |
|
03-Jul-2014 |
Ben Chan <benchan@chromium.org> |
shill: Fix header guards to conform to coding style. BUG=chromium:385849 TEST=Tested the following: 1. `USE='cellular wimax' FEATURES=test emerge-$BOARD platform2` 2. `cpplint.py --filter=-build/include_order shill/*.h` reports no 'build/header_guard' issue. Change-Id: I1a3972a1f88e5001550eccd561633bd4e5c5faba Reviewed-on: https://chromium-review.googlesource.com/206680 Reviewed-by: Ben Chan <benchan@chromium.org> Commit-Queue: Ben Chan <benchan@chromium.org> Tested-by: Ben Chan <benchan@chromium.org>
/system/connectivity/shill/accessor_interface.h
|
8a5322984f2d81bcbfd8d44c59747a11bd9b904b |
|
17-Jun-2014 |
Alex Vakulenko <avakulenko@chromium.org> |
shill: Fix most of warnings from cpplint Fix most cpplinter's warnings. On a few occasisions, the warnings produced were false-positives, so added explicit // NOLINT comment overrides. There were a few instances of disallowed non-const reference usage (reported as runtime/reference error by cpplint) for "DBus::Error &error" which should potentially be changed to pointers to comply with C++ coding style but I will let Shill owners do that since there is quite a few of those and the change isn't brain-dead simple... BUG=None TEST=platform2 still compiles and all unit tests pass. Change-Id: Ic2e31896aa13d20eeb1a85dee74f3db8cccfde2e Reviewed-on: https://chromium-review.googlesource.com/204164 Tested-by: Alex Vakulenko <avakulenko@chromium.org> Reviewed-by: mukesh agrawal <quiche@chromium.org> Commit-Queue: Alex Vakulenko <avakulenko@chromium.org>
/system/connectivity/shill/accessor_interface.h
|
e7c7e65bba9e964aa500ef53cc79215426acf50c |
|
19-Jun-2013 |
mukesh agrawal <quiche@chromium.org> |
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>
/system/connectivity/shill/accessor_interface.h
|
bebf1b8bce52b88c2cc2d93200b9405f9c19cf21 |
|
24-Apr-2013 |
mukesh agrawal <quiche@chromium.org> |
shill: make setting a property to its current value a no-op Before this change, setting the EAP authentication properties on a WiFiService would cause the connection to be dropped. The connection would drop even if the new values were the same as the old. With this change, the connection is only dropped if the new values differ from the old. Overview of changes: - AccessorInterface: have property setters return a bool (rather than void). Setters should return true to indicate the value was changed, and false otherwise. - PropertyAccessor and derived classes: - Implement the new AccessorInterface. - Add tests that we implement the new AccessorInterface. - Custom property setters (various classes): - Update existing custom setters to return true if the value was changed, and false otherwise. - Add tests that custom setters implement the desired behavior. - DBusAdaptor - Change SetProperty to propagate the return value of PropertyStore's setters, rather than Error::IsSuccess(). - In combination with other changes, this means DBusAdaptor::SetProperty now returns false if the new value is the same as the old. - This also suppresses some spurious change notifications from IPConfig, Manager, and Profile objects. - Add tests that DBusAdaptor::SetProperty implements the desired behavior. - PropertyStore - Add a change callback. This optional argument to the ctor is invoked if a setter or clearer modified its property. This is so that classes don't have to depend on their RPC adaptors to inform them of a change. (See changes in Service.) - Have setters pass through the return value of the Accessor, rather than returning Error::IsSuccess(). In combination with other changes, this means that setters now returns false if the new value is the same as the old. - Add tests that PropertyStore invokes the change callback appropriately. - ClearPropertyNonexistent, SetPropertyNonexistent: no callback - ClearProperty: callback - SetProperty: callback if and only if property changed - Service - Register OnPropertyChanged with PropertyStore, instead of relying on a callback from ServiceDBusAdaptor. Two reasons for the change: 1) The RPC adaptors should be as trivial as possible, and 2) We can't test code in the RPC adaptors. 3) If we can't test code in the RPC adaptors, go to 1. - ServiceDBusAdaptor: remove OnPropertyChange callback in SetProperty. See Service for the rationale. - Update existing SetProperty tests (various classes) We now use values that differ from the current value of the property. This ensures that the setter returns true. - WiFiServiceTest: add a case to test that EAP authentication property changes caused cached credentials to be cleared appropriately. This is redundant given some of the other tests. But given that this was the original problem in the bug, it seems worth testing specifically. - HACKING: add some guidelines for what to do when adding properties. While there: - Change some HelpRegister... functions to HelpRegisterConst... - Update some tests to check error.is_set() before reading error.name(). This avoids a stray pointer dereference. - Add SetStringmapsProperty to PropertyStore. This is needed because PropertyStoreTypedTest now tests setters. - Remove duplicate kAutoConnectProperty test case in ServiceTest.SetProperty - Remove unused local in WiFiServiceTest.SetPassphraseRemovedCachedCredentials - Remove unused method Device::HelpRegisterDerivedStrings - Remove KeyValueStore from the set of types exercised by PropertyStoreTypedTest. We only use KeyValueStore for const properties, and PropertyStoreTypedTest tests setting and clearing. - Add PropertyChanges test to EthernetEapServiceTest. BUG=chromium:233681 TEST=new unit tests Change-Id: I9bdd89fbe6f19101dfcd5f126f2ba9c81533ff97 Reviewed-on: https://gerrit.chromium.org/gerrit/49733 Commit-Queue: mukesh agrawal <quiche@chromium.org> Reviewed-by: mukesh agrawal <quiche@chromium.org> Tested-by: mukesh agrawal <quiche@chromium.org>
/system/connectivity/shill/accessor_interface.h
|
e18c33bc332e558a51a1a29c20b00a935483fb96 |
|
11-Jul-2012 |
Paul Stewart <pstew@chromium.org> |
shill: Add uint64 properties This is a fairly minimal addition, since we will only need read-only derived types. BUG=chromium-os:31584 TEST=New unit tests; list-devices on a real machine Change-Id: I7b65224ae329443066f563b620b379f29006f8a0 Reviewed-on: https://gerrit.chromium.org/gerrit/27157 Reviewed-by: Paul Stewart <pstew@chromium.org> Tested-by: Paul Stewart <pstew@chromium.org> Commit-Ready: Paul Stewart <pstew@chromium.org>
/system/connectivity/shill/accessor_interface.h
|
acdc11f4961994f894f978713d1a22f2eb23687d |
|
30-Mar-2012 |
Jason Glasgow <jglasgow@chromium.org> |
shill: Change service.device property to be a DBus::Path Change service.device property to be a DBus::Path because it is a path not a string. Flimflam exports the property as a path. Exporting it as a string causes cashew to die. Change the unit tests to use valid DBus paths which can contain elements separated by /. Each element can contain only [A-z][a-z][0-9]_ and cannot end in a trailing /. BUG=chromium-os:28734 TEST=run shill unit tests, run shill on DUT. Run cashew. Change-Id: I65fb843fb93f76d95eea6da334ad326ccb9fbd6f Reviewed-on: https://gerrit.chromium.org/gerrit/19366 Commit-Ready: Jason Glasgow <jglasgow@chromium.org> Reviewed-by: Jason Glasgow <jglasgow@chromium.org> Tested-by: Jason Glasgow <jglasgow@chromium.org>
/system/connectivity/shill/accessor_interface.h
|
2366eede28daa298f240ddbc3c8f36cc0e7b7ad1 |
|
21-Mar-2012 |
mukesh agrawal <quiche@chromium.org> |
shill: serialize Manager.Services as an array of object paths Previously, we serialized this property as an array of strings. That was contrary to the API documentation, and caused a crash in update_engine_client. BUG=chromium-os:27987 TEST=unit tests, manual Manual testing: ran on device, and checked that update_engine_client was able to identify the technology of the default service. Collateral change: replace PathArray with Paths (and similar for variations, such as path_array). This makes the naming of Paths conform to the style used for Strings, Stringmaps, etc. Change-Id: I4ee4b9df04d3f006ab974c2092f6e515ebf5a9b8 Reviewed-on: https://gerrit.chromium.org/gerrit/18592 Commit-Ready: mukesh agrawal <quiche@chromium.org> Reviewed-by: mukesh agrawal <quiche@chromium.org> Tested-by: mukesh agrawal <quiche@chromium.org>
/system/connectivity/shill/accessor_interface.h
|
292dc0f18ce6ba7ea1419efc4d8ef909712bb7c2 |
|
27-Jan-2012 |
mukesh agrawal <quiche@chromium.org> |
shill: teach property accessors how to clear properties, and update code that instantiates property accessors accordingly. BUG=chromium-os:24814 TEST=new unit tests Change-Id: Iae385c331648e74916c2eb2b69c41ccc9cdcafdf Reviewed-on: https://gerrit.chromium.org/gerrit/15289 Reviewed-by: mukesh agrawal <quiche@chromium.org> Tested-by: mukesh agrawal <quiche@chromium.org> Commit-Ready: mukesh agrawal <quiche@chromium.org>
/system/connectivity/shill/accessor_interface.h
|
63138a9b8249fd69c83a772ee3170551a589d57a |
|
06-Feb-2012 |
Darin Petkov <petkov@chromium.org> |
shill: Add SIMLockEnabled to the SIM lock status properties and emit changes. This patch replaces the StrIntPair class with the more generic KeyValueStore and updates cellular's SIMLockStatus property to include the enabled key/value pair. BUG=chromium-os:25850 TEST=unit tests Change-Id: I2662cb468807e3afa04c3699e323f31282e1a50e Reviewed-on: https://gerrit.chromium.org/gerrit/15325 Tested-by: Darin Petkov <petkov@chromium.org> Reviewed-by: Eric Shienbrood <ers@chromium.org> Reviewed-by: mukesh agrawal <quiche@chromium.org> Commit-Ready: Darin Petkov <petkov@chromium.org>
/system/connectivity/shill/accessor_interface.h
|
1b7a616197af7ff753dbe7614a8e207b1e10ac1a |
|
09-Nov-2011 |
Gaurav Shah <gauravsh@chromium.org> |
shill: Implement write-only properties Certain properties (e.g. WIFI Passphrase) are write only and must not be returned when Service.GetProperties() is called over D-Bus. This CL implements WriteOnlyProperties, a write-only analog of the read-only ConstProperties. Also add a ReadablePropertyConstIterator which only returns the readable properties. Switch over DBus adaptor and PropertyStore to use that. BUG=chromium-os:21196 TEST=Added 2 new unittests. Change-Id: I52815cc395650e0b49e1acac8d4954deeebcee5d Reviewed-on: https://gerrit.chromium.org/gerrit/11402 Commit-Ready: Gaurav Shah <gauravsh@chromium.org> Reviewed-by: Gaurav Shah <gauravsh@chromium.org> Tested-by: Gaurav Shah <gauravsh@chromium.org>
/system/connectivity/shill/accessor_interface.h
|
ced6a0b2c2c3952ba79e10709345b573d2e06af6 |
|
09-Nov-2011 |
Paul Stewart <pstew@chromium.org> |
shill: wifi: Scan for hidden SSIDs Find all services with the "hidden" property set. If any exist, specify the "SSIDs" argument to wpa_supplicant and supply up to MAX_SSIDS-1 of them as arguments to the scan, as well as the broadcast SSID. Side effect -- removed the "services_" vector from the Service superclass. I don't think a non-subclassed Service vector makes any sense to keep around. BUG=chromium-os:22074 TEST=New Unit Test Change-Id: I52af8b81d301a9561812af04a16df6e6fcc644bc Reviewed-on: https://gerrit.chromium.org/gerrit/11347 Reviewed-by: mukesh agrawal <quiche@chromium.org> Commit-Ready: Paul Stewart <pstew@chromium.org> Tested-by: Paul Stewart <pstew@chromium.org>
/system/connectivity/shill/accessor_interface.h
|
ffa3d0433d419aa9251f3768a0090a27b6b1c434 |
|
07-Oct-2011 |
mukesh agrawal <quiche@chromium.org> |
shill: allow property accessors to return an error message if a property's value cannot be changed. also, reduce some code duplication in property_store.cc BUG=chromium-os:21384 TEST=unittests Change-Id: Iaac8d40bbb9e9a1341d6c6d01642885d88ac0e27 Reviewed-on: http://gerrit.chromium.org/gerrit/8925 Reviewed-by: mukesh agrawal <quiche@chromium.org> Tested-by: mukesh agrawal <quiche@chromium.org>
/system/connectivity/shill/accessor_interface.h
|
889666b49d97f61a993a4e642e5728cce1c3758a |
|
03-Jul-2011 |
Chris Masone <cmasone@chromium.org> |
[shill] Add support for weird Cellular.SimLockStatus property SimLockStatus is bizarre in that its type is a dictionary that maps names to values of different types. It's { string:string, string:uint32 }, which doesn't fit nicely into dbus-c++, because you can't represent it trivially with an STL container -- and therefore I have to do something special to convert it. BUG=chromium-os:17281 TEST=unit tests STATUS=Verified Change-Id: I9c4e43f75c666570caae66e04293bce6207a1b99 Reviewed-on: http://gerrit.chromium.org/gerrit/3619 Reviewed-by: Chris Masone <cmasone@chromium.org> Tested-by: Chris Masone <cmasone@chromium.org>
/system/connectivity/shill/accessor_interface.h
|
a8a2c25df27d77f278511769f0c9029b03dff053 |
|
28-Jun-2011 |
Chris Masone <cmasone@chromium.org> |
[shill] Enable the getting of properties, and wire to dbus. PropertyStore exposes getters for iterators pointing to the beginning and end of its property maps, and a static utility method in DBusAdaptor uses these to iterate through all the properties in the PropertyStore and build up a map of property name -> DBus::Variant-wrapped value. BUG=chromium-os:16343 TEST=unit tests Change-Id: I85ebbaee167ab2feed0fcf8fe654274de352aca0 Reviewed-on: http://gerrit.chromium.org/gerrit/3359 Reviewed-by: Chris Masone <cmasone@chromium.org> Tested-by: Chris Masone <cmasone@chromium.org>
/system/connectivity/shill/accessor_interface.h
|
b925cc8f481d21fddd9569fc68861f6e5b6e3eae |
|
23-Jun-2011 |
Chris Masone <cmasone@chromium.org> |
[shill] Add properties to Manager, Device and subclasses. Clean up unit tests 1) Pull much of the properties stuff (the maps, Register*(), Contains()) up into PropertyStore, and make it a class, not an interface. 2) Add proper property stuff to Manager and Device. Move Cellular- and Wifi- specific properties to Cellular, CellularService, WiFi, WiFiService as appropriate. 3) clean up unit tests. BUG=chromium-os:16343 TEST=unit tests Change-Id: Iba88f384a5fbe383608cb79fa2134d978f1b81e8 Reviewed-on: http://gerrit.chromium.org/gerrit/3136 Reviewed-by: Chris Masone <cmasone@chromium.org> Tested-by: Chris Masone <cmasone@chromium.org>
/system/connectivity/shill/accessor_interface.h
|
3bd3c8c33917221d1074f1aa19272e45c0ce2793 |
|
13-Jun-2011 |
Chris Masone <cmasone@chromium.org> |
[shill] Add support for setting properties. This CL adds a framework for supporting RPC-exposed properties in Shill. It also plumbs the code for setting properties on Service objects to prove the approach. Device and Manager settings will follow. BUG=chromium-os:16343 TEST=build shill, run unit tests. Change-Id: I55869453d6039e688f1a49be9dfb1ba1315efe0a Reviewed-on: http://gerrit.chromium.org/gerrit/3004 Reviewed-by: Darin Petkov <petkov@chromium.org> Tested-by: Chris Masone <cmasone@chromium.org>
/system/connectivity/shill/accessor_interface.h
|