commit ac635a8fb539fd44f3e24c33872b61fd064c0d60
author Darin Petkov <petkov@chromium.org> Tue Jan 10 16:51:58 2012 +0100
committer Gerrit <chrome-bot@google.com> Wed Jan 11 16:22:30 2012 -0800
tree c7261513c97d35745f79aba664ca3f435fd16252
parent 2c3773810323292df11b8359ad3f368989f4884a

shill: Set the cellular service friendly name.

BUG=chromium-os:24897
TEST=unit tests, tested on device with GSM

Change-Id: I2fe803822f2ddc5c52daddcf625b3327f95563ad
Reviewed-on: https://gerrit.chromium.org/gerrit/13933
Commit-Ready: Darin Petkov <petkov@chromium.org>
Reviewed-by: Darin Petkov <petkov@chromium.org>
Tested-by: Darin Petkov <petkov@chromium.org>

cellular_service.cc

diff --git a/cellular_service.cc b/cellular_service.cc
index 7a9a7d4..dbd6e70 100644
--- a/cellular_service.cc
+++ b/cellular_service.cc
@@ -41,6 +41,8 @@
                                 &serving_operator_.ToDict());
   store->RegisterConstUint8(flimflam::kSignalStrengthProperty, &strength_);
   store->RegisterConstString(flimflam::kUsageURLProperty, &usage_url_);
+
+  set_friendly_name(device->CreateFriendlyServiceName());
 }
 
 CellularService::~CellularService() { }
@@ -61,6 +63,7 @@
 }
 
 string CellularService::GetStorageIdentifier() const {
+  // TODO(petkov): Fix the return value (crosbug.com/24952).
   string id = base::StringPrintf("%s_%s_%s",
                                  kServiceType,
                                  cellular_->address().c_str(),