shill: Remove Cellular's obsolete |type_| data member.

Also, some cleanup and renaming.

BUG=chromium-os:18735
TEST=unit tests

Change-Id: Ia201113a9514c884a69209a2fd06a01068e8bd14
Reviewed-on: https://gerrit.chromium.org/gerrit/11901
Commit-Ready: Darin Petkov <petkov@chromium.org>
Reviewed-by: Darin Petkov <petkov@chromium.org>
Tested-by: Darin Petkov <petkov@chromium.org>
diff --git a/cellular_capability_gsm.h b/cellular_capability_gsm.h
index 6e5796b..dbfe8ca 100644
--- a/cellular_capability_gsm.h
+++ b/cellular_capability_gsm.h
@@ -26,8 +26,9 @@
   CellularCapabilityGSM(Cellular *cellular);
 
   // Inherited from CellularCapability.
-  virtual void OnStart();
-  virtual void OnStop();
+  virtual void OnDeviceStarted();
+  virtual void OnDeviceStopped();
+  virtual void OnServiceCreated();
   virtual void UpdateStatus(const DBusPropertiesMap &properties);
   virtual void SetupConnectProperties(DBusPropertiesMap *properties);
   virtual void GetSignalQuality();
@@ -48,7 +49,6 @@
   virtual std::string GetRoamingStateString() const;
   virtual void OnModemManagerPropertiesChanged(
       const DBusPropertiesMap &properties);
-  virtual void OnServiceCreated();
 
   // Obtains the IMEI, IMSI, SPN and MSISDN.
   virtual void GetIdentifiers();