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>
diff --git a/property_store_unittest.h b/property_store_unittest.h
index 639649f..42a0e8d 100644
--- a/property_store_unittest.h
+++ b/property_store_unittest.h
@@ -58,10 +58,12 @@
   const std::string &run_path() const { return path_; }
   const std::string &storage_path() const { return path_; }
 
+  const std::string &internal_error() const { return internal_error_; }
   const std::string &invalid_args() const { return invalid_args_; }
   const std::string &invalid_prop() const { return invalid_prop_; }
 
  private:
+  const std::string internal_error_;
   const std::string invalid_args_;
   const std::string invalid_prop_;
   ScopedTempDir dir_;