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

Unified Diff: device/geolocation/wifi_data_provider_common.cc

Issue 2833273002: Geolocation cleanup: use unique_ptr<>s and rename New{WlanApi/PollingPolicy} to CreateBla() (Closed)
Patch Set: reillyg@s nits Created 3 years, 8 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: device/geolocation/wifi_data_provider_common.cc
diff --git a/device/geolocation/wifi_data_provider_common.cc b/device/geolocation/wifi_data_provider_common.cc
index 527de27e79c47a23ed12e268e9da410a99eb694e..9d80043609e9c84a84702048fb2a335ac84bf48d 100644
--- a/device/geolocation/wifi_data_provider_common.cc
+++ b/device/geolocation/wifi_data_provider_common.cc
@@ -28,7 +28,7 @@ WifiDataProviderCommon::~WifiDataProviderCommon() {}
void WifiDataProviderCommon::StartDataProvider() {
DCHECK(!wlan_api_);
- wlan_api_.reset(NewWlanApi());
+ wlan_api_ = CreateWlanApi();
if (!wlan_api_) {
// Error! Can't do scans, so don't try and schedule one.
is_first_scan_complete_ = true;
@@ -36,7 +36,7 @@ void WifiDataProviderCommon::StartDataProvider() {
}
DCHECK(!polling_policy_);
- polling_policy_.reset(NewPollingPolicy());
+ polling_policy_ = CreatePollingPolicy();
DCHECK(polling_policy_);
// Perform first scan ASAP regardless of the polling policy. If this scan
« no previous file with comments | « device/geolocation/wifi_data_provider_common.h ('k') | device/geolocation/wifi_data_provider_common_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698