commit 7a4e4008dc09effe39c94a5f9575644a79ee1388
author mukesh agrawal <quiche@chromium.org> Tue Sep 06 11:26:05 2011 -0700
committer mukesh agrawal <quiche@chromium.org> Fri Sep 23 13:55:27 2011 -0700
tree 5548268fbae288a48369303ebed3ad17baf029c3
parent 877ff9894e21c4c78b63777025f6c8d577ccd855

shill: implement Manager.GetService (error-case only)

BUG=chromium-os:20254
TEST=unittests, WiFiManager/000_SSID_Length_Limit

this gives us enough to pass the autotest for
network_WiFiManager/000_SSID_Length_Limit.

Change-Id: Ib0305e707d2203327d846be3e0b206033d6a884a
Reviewed-on: http://gerrit.chromium.org/gerrit/7567
Commit-Ready: mukesh agrawal <quiche@chromium.org>
Reviewed-by: mukesh agrawal <quiche@chromium.org>
Tested-by: mukesh agrawal <quiche@chromium.org>

cellular_service.cc

diff --git a/cellular_service.cc b/cellular_service.cc
index 26556d3..f2c2b10 100644
--- a/cellular_service.cc
+++ b/cellular_service.cc
@@ -23,10 +23,9 @@
                                  EventDispatcher *dispatcher,
                                  Manager *manager,
                                  const CellularRefPtr &device)
-    : Service(control_interface, dispatcher, manager),
+    : Service(control_interface, dispatcher, manager, flimflam::kTypeCellular),
       strength_(0),
-      cellular_(device),
-      type_(flimflam::kTypeCellular) {
+      cellular_(device) {
   PropertyStore *store = this->mutable_store();
   store->RegisterConstString(flimflam::kActivationStateProperty,
                              &activation_state_);
@@ -40,7 +39,6 @@
   store->RegisterConstStringmap(flimflam::kServingOperatorProperty,
                                 &serving_operator_.ToDict());
   store->RegisterConstUint8(flimflam::kSignalStrengthProperty, &strength_);
-  store->RegisterConstString(flimflam::kTypeProperty, &type_);
   store->RegisterConstString(flimflam::kUsageURLProperty, &usage_url_);
 }