commit 27526f1dad75e135f437e023cb34b0919d9a981f
author Prathmesh Prabhu <pprabhu@chromium.org> Mon Mar 25 19:42:18 2013 -0700
committer ChromeBot <chrome-bot@google.com> Fri Mar 29 12:45:05 2013 -0700
tree 99c01339059ada0285504f80d1bc9ab62ce4c960
parent c4c9d7c48886fbd34b507d44f12c3722864f57e9

shill: cellular: Consolidate singletons passed through constructors

The following are singletons that were stored individually by many
classes, passed down constructors as the objects were created. Move
singletons to ModemInfo, and pass only a ModemInfo object down.
Singletons: ControlInterface, EventDispatcher, Metrics, Manager, GLib,
ActivatingIccidStore, CellularOperatorInfo, and mobile_provider_db.
Classes who stored all these singletons: ModemInfo, Cellular,
ModemManager (and flavours), Modem (and flavours), CellularCpability
(and flavours).

BUG=chromium:222543
TEST=(1) Build and run unit tests. (2) Test on device that LTE
activation and cellular functionality related to refactored singletons
still works. (3) Run autotest: network_3GSmokeTest.

Change-Id: I0c2866e0cd0bab63e3c3078ff73361322a8f509b
Reviewed-on: https://gerrit.chromium.org/gerrit/46485
Reviewed-by: Ben Chan <benchan@chromium.org>
Commit-Queue: Prathmesh Prabhu <pprabhu@chromium.org>
Tested-by: Prathmesh Prabhu <pprabhu@chromium.org>

modem_manager_1.cc

diff --git a/modem_manager_1.cc b/modem_manager_1.cc
index 37e8271..b19ad20 100644
--- a/modem_manager_1.cc
+++ b/modem_manager_1.cc
@@ -22,24 +22,10 @@
 
 ModemManager1::ModemManager1(const string &service,
                              const string &path,
-                             ControlInterface *control_interface,
-                             EventDispatcher *dispatcher,
-                             Metrics *metrics,
-                             Manager *manager,
-                             GLib *glib,
-                             ActivatingIccidStore *activating_iccid_store,
-                             CellularOperatorInfo *cellular_operator_info,
-                             mobile_provider_db *provider_db)
+                             ModemInfo *modem_info)
     : ModemManager(service,
                    path,
-                   control_interface,
-                   dispatcher,
-                   metrics,
-                   manager,
-                   glib,
-                   activating_iccid_store,
-                   cellular_operator_info,
-                   provider_db),
+                   modem_info),
       weak_ptr_factory_(this) {}
 
 ModemManager1::~ModemManager1() {}
@@ -76,13 +62,7 @@
   shared_ptr<Modem1> modem1(new Modem1(owner(),
                                        service(),
                                        path,
-                                       control_interface(),
-                                       dispatcher(),
-                                       metrics(),
-                                       manager(),
-                                       activating_iccid_store(),
-                                       cellular_operator_info(),
-                                       provider_db()));
+                                       modem_info()));
   RecordAddedModem(modem1);
   InitModem1(modem1, properties);
 }