Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(73)

Unified Diff: chromeos/components/tether/host_scan_cache.cc

Issue 2915833003: Tether: Break helper TimerFactory out of BleConnectionManager and HostScanCache. (Closed)
Patch Set: khorimoto@ comments. Created 3 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chromeos/components/tether/host_scan_cache.cc
diff --git a/chromeos/components/tether/host_scan_cache.cc b/chromeos/components/tether/host_scan_cache.cc
index 38a7aae5762c510ba49718b05ff7d64d724f403c..c059aebe095dd932b3f3b7da58ad1ec1914e107f 100644
--- a/chromeos/components/tether/host_scan_cache.cc
+++ b/chromeos/components/tether/host_scan_cache.cc
@@ -11,6 +11,7 @@
#include "chromeos/components/tether/active_host.h"
#include "chromeos/components/tether/device_id_tether_network_guid_map.h"
#include "chromeos/components/tether/tether_host_response_recorder.h"
+#include "chromeos/components/tether/timer_factory.h"
#include "chromeos/network/network_state_handler.h"
#include "components/proximity_auth/logging/logging.h"
@@ -18,27 +19,12 @@ namespace chromeos {
namespace tether {
-namespace {
-
-class TimerFactoryImpl : public HostScanCache::TimerFactory {
- public:
- TimerFactoryImpl() {}
- ~TimerFactoryImpl() {}
-
- // HostScanCache::TimerFactory:
- std::unique_ptr<base::Timer> CreateOneShotTimer() override {
- return base::MakeUnique<base::OneShotTimer>();
- }
-};
-
-} // namespace
-
HostScanCache::HostScanCache(
NetworkStateHandler* network_state_handler,
ActiveHost* active_host,
TetherHostResponseRecorder* tether_host_response_recorder,
DeviceIdTetherNetworkGuidMap* device_id_tether_network_guid_map)
- : timer_factory_(base::MakeUnique<TimerFactoryImpl>()),
+ : timer_factory_(base::MakeUnique<TimerFactory>()),
network_state_handler_(network_state_handler),
active_host_(active_host),
tether_host_response_recorder_(tether_host_response_recorder),
« no previous file with comments | « chromeos/components/tether/host_scan_cache.h ('k') | chromeos/components/tether/host_scan_cache_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698