diff options
author | Gaurav Shah <gauravsh@chromium.org> | 2011-11-09 11:41:01 -0800 |
---|---|---|
committer | Gerrit <chrome-bot@google.com> | 2011-11-11 11:21:15 -0800 |
commit | 1b7a616197af7ff753dbe7614a8e207b1e10ac1a (patch) | |
tree | 1bffa47b7dd9a98d8c33045860ddc31c6a58cdae /profile.h | |
parent | 1272a43bf73f6a99f51a6cc6345183aa503cfb67 (diff) | |
download | shill-1b7a616197af7ff753dbe7614a8e207b1e10ac1a.tar.gz |
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>
Diffstat (limited to 'profile.h')
-rw-r--r-- | profile.h | 6 |
1 files changed, 3 insertions, 3 deletions
@@ -89,8 +89,8 @@ class Profile : public base::RefCounted<Profile> { // Return whether |service| can configure itself from the profile. bool ContainsService(const ServiceConstRefPtr &service); - std::vector<std::string> EnumerateAvailableServices(); - std::vector<std::string> EnumerateEntries(); + std::vector<std::string> EnumerateAvailableServices(Error *error); + std::vector<std::string> EnumerateEntries(Error *error); // Write all in-memory state to disk via |storage_|. virtual bool Save(); @@ -129,7 +129,7 @@ class Profile : public base::RefCounted<Profile> { void HelpRegisterDerivedStrings( const std::string &name, - Strings(Profile::*get)(void), + Strings(Profile::*get)(Error *), void(Profile::*set)(const Strings&, Error *)); // Data members shared with subclasses via getter/setters above in the |