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.h b/cellular_capability.h
index 3638c27..712e1bf 100644
--- a/cellular_capability.h
+++ b/cellular_capability.h
@@ -31,8 +31,11 @@
   EventDispatcher *dispatcher() const;
 
   // Invoked on starting and stopping the cellular device.
-  virtual void OnStart() = 0;
-  virtual void OnStop() = 0;
+  virtual void OnDeviceStarted() = 0;
+  virtual void OnDeviceStopped() = 0;
+
+  // Invoked by the parent Cellular device when a new service is created.
+  virtual void OnServiceCreated() = 0;
 
   virtual void UpdateStatus(const DBusPropertiesMap &properties) = 0;
 
@@ -77,9 +80,6 @@
   virtual void OnModemManagerPropertiesChanged(
       const DBusPropertiesMap &properties) = 0;
 
-  // Invoked by the parent Cellular device when a new service is created.
-  virtual void OnServiceCreated() = 0;
-
  private:
   friend class CellularTest;