diff options
| author | Alex Deymo <deymo@chromium.org> | 2015-07-31 12:35:22 -0700 |
|---|---|---|
| committer | ChromeOS Commit Bot <chromeos-commit-bot@chromium.org> | 2015-08-01 02:53:04 +0000 |
| commit | f6ee0163504eaf1a7cc136ba2c075d77ae2b4dd5 (patch) | |
| tree | 1ac88ce36185ac2e2a246123b4b80eb178d5d91c /payload_state.cc | |
| parent | 75eac7e523f825afdb88508922d7dc65fa366b19 (diff) | |
update_engine: Move ConnectionManager to an abstract Interface.
MockConnectionManager required to use one of ConnectionManager
constructors passing pointers that won't be use by the mock. This
patch moves the interface to its own ConnectionManagerInterface class.
BUG=None
TEST=unittests still pass.
Change-Id: I9ed09daf8e4256304be7dab30cfbe751901dc24b
Reviewed-on: https://chromium-review.googlesource.com/290120
Reviewed-by: Alex Vakulenko <avakulenko@chromium.org>
Commit-Queue: Alex Deymo <deymo@chromium.org>
Trybot-Ready: Alex Deymo <deymo@chromium.org>
Tested-by: Alex Deymo <deymo@chromium.org>
Diffstat (limited to 'payload_state.cc')
| -rw-r--r-- | payload_state.cc | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/payload_state.cc b/payload_state.cc index ed7b775d..a15d68bf 100644 --- a/payload_state.cc +++ b/payload_state.cc @@ -10,7 +10,6 @@ #include <base/logging.h> #include <base/strings/string_util.h> #include <base/strings/stringprintf.h> -#include <base/format_macros.h> #include <policy/device_policy.h> #include "update_engine/clock.h" @@ -176,7 +175,8 @@ void PayloadState::AttemptStarted(AttemptType attempt_type) { NetworkConnectionType network_connection_type; NetworkTethering tethering; RealDBusWrapper dbus_iface; - ConnectionManager* connection_manager = system_state_->connection_manager(); + ConnectionManagerInterface* connection_manager = + system_state_->connection_manager(); if (!connection_manager->GetConnectionProperties(&dbus_iface, &network_connection_type, &tethering)) { |