commit aeeb2e015ef5ef046c744d3c5daa7f353eb71b4e
author Sen Jiang <senj@google.com> Thu Jun 09 15:00:16 2016 -0700
committer Sen Jiang <senj@google.com> Mon Jun 20 20:09:38 2016 +0000
tree 31ff1e2581138e8ca8859973653f929e937614a4
parent 72a68d829f6b0bc97368c64d35a860465716018c

Make UpdateAttempter own debugd_proxy.

It's the only class using this proxy.

Test: mma
Bug: 28800946

Change-Id: I1a98b417f213db5d47de451390367ca4975db4b0

real_system_state.h

diff --git a/real_system_state.h b/real_system_state.h
index 65175ae..18a0d98 100644
--- a/real_system_state.h
+++ b/real_system_state.h
@@ -21,7 +21,6 @@
 
 #include <memory>
 
-#include <debugd/dbus-proxies.h>
 #include <metrics/metrics_library.h>
 #include <policy/device_policy.h>
 
@@ -32,7 +31,6 @@
 #include "update_engine/common/prefs.h"
 #include "update_engine/connection_manager_interface.h"
 #include "update_engine/daemon_state_interface.h"
-#include "update_engine/dbus_connection.h"
 #include "update_engine/p2p_manager.h"
 #include "update_engine/payload_state.h"
 #include "update_engine/power_manager_interface.h"
@@ -48,9 +46,7 @@
  public:
   // Constructs all system objects that do not require separate initialization;
   // see Initialize() below for the remaining ones.
-  // TODO(senj): Remove this constructor once all proxies get DBus themselves.
-  explicit RealSystemState(const scoped_refptr<dbus::Bus>& bus);
-  RealSystemState() : RealSystemState(DBusConnection::Get()->GetDBus()){};
+  RealSystemState() = default;
   ~RealSystemState() override;
 
   // Initializes and sets systems objects that require an initialization
@@ -127,7 +123,6 @@
 
  private:
   // Real DBus proxies using the DBus connection.
-  org::chromium::debugdProxy debugd_proxy_;
   LibCrosProxy libcros_proxy_;
 
   // Interface for the power manager.