commit c90fe52d7eed164b63f9a3af4bcb505ee78d3f43
author Darin Petkov <petkov@chromium.org> Fri Jul 15 13:59:47 2011 -0700
committer Darin Petkov <petkov@chromium.org> Tue Jul 19 10:00:35 2011 -0700
tree 447e7a8937b6dfc99cfd01902e2ca5ae23c5aac5
parent af57195acad7aebca521e2d67bd58f4c0abb676b

shill: Create mockable ModemManagerProxy on appearance of new ModemManagers.

BUG=chromium-os:17649
TEST=unit tests, tested on device

Change-Id: Id9e6500168d63493eb94bd7939e379b964bef063
Reviewed-on: http://gerrit.chromium.org/gerrit/4269
Tested-by: Darin Petkov <petkov@chromium.org>
Reviewed-by: Chris Masone <cmasone@chromium.org>

modem_manager.cc

diff --git a/modem_manager.cc b/modem_manager.cc
index df4c4ae..94536fb 100644
--- a/modem_manager.cc
+++ b/modem_manager.cc
@@ -6,6 +6,9 @@
 
 #include <base/logging.h>
 
+#include "shill/modem_manager_proxy.h"
+#include "shill/proxy_factory.h"
+
 using std::string;
 
 namespace shill {
@@ -30,6 +33,7 @@
 void ModemManager::Start() {
   LOG(INFO) << "Start watching modem manager service: " << service_;
   CHECK_EQ(0, watcher_id_);
+  // TODO(petkov): Implement DBus name watching through dbus-c++.
   watcher_id_ = glib_->BusWatchName(G_BUS_TYPE_SYSTEM,
                                     service_.c_str(),
                                     G_BUS_NAME_WATCHER_FLAGS_NONE,
@@ -50,12 +54,13 @@
 
 void ModemManager::Connect(const string &owner) {
   owner_ = owner;
-  // TODO(petkov): Create a proxy to the manager and enumerate its devices.
+  proxy_.reset(
+      ProxyFactory::factory()->CreateModemManagerProxy(this, path_, owner_));
 }
 
 void ModemManager::Disconnect() {
   owner_.clear();
-  // TODO(petkov): Destroy manager's proxy and its devices.
+  proxy_.reset();
 }
 
 void ModemManager::OnAppear(GDBusConnection *connection,