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

Side by Side Diff: content/browser/geolocation/wifi_data_provider_common_unittest.cc

Issue 678073006: Standardize usage of virtual/override/final specifiers. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix bogus formatting Created 6 years, 1 month 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 unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include <vector> 5 #include <vector>
6 6
7 #include "base/memory/scoped_ptr.h" 7 #include "base/memory/scoped_ptr.h"
8 #include "base/strings/string_util.h" 8 #include "base/strings/string_util.h"
9 #include "base/strings/utf_string_conversions.h" 9 #include "base/strings/utf_string_conversions.h"
10 #include "base/third_party/dynamic_annotations/dynamic_annotations.h" 10 #include "base/third_party/dynamic_annotations/dynamic_annotations.h"
(...skipping 95 matching lines...) Expand 10 before | Expand all | Expand 10 after
106 return new WifiDataProviderCommonWithMock; 106 return new WifiDataProviderCommonWithMock;
107 } 107 }
108 108
109 // Main test fixture 109 // Main test fixture
110 class GeolocationWifiDataProviderCommonTest : public testing::Test { 110 class GeolocationWifiDataProviderCommonTest : public testing::Test {
111 public: 111 public:
112 GeolocationWifiDataProviderCommonTest() 112 GeolocationWifiDataProviderCommonTest()
113 : loop_quitter_(&main_message_loop_) { 113 : loop_quitter_(&main_message_loop_) {
114 } 114 }
115 115
116 virtual void SetUp() { 116 void SetUp() override {
117 provider_ = new WifiDataProviderCommonWithMock; 117 provider_ = new WifiDataProviderCommonWithMock;
118 wlan_api_ = provider_->new_wlan_api_.get(); 118 wlan_api_ = provider_->new_wlan_api_.get();
119 polling_policy_ = provider_->new_polling_policy_.get(); 119 polling_policy_ = provider_->new_polling_policy_.get();
120 provider_->AddCallback(&loop_quitter_.callback_); 120 provider_->AddCallback(&loop_quitter_.callback_);
121 } 121 }
122 virtual void TearDown() { 122 void TearDown() override {
123 provider_->RemoveCallback(&loop_quitter_.callback_); 123 provider_->RemoveCallback(&loop_quitter_.callback_);
124 provider_->StopDataProvider(); 124 provider_->StopDataProvider();
125 provider_ = NULL; 125 provider_ = NULL;
126 } 126 }
127 127
128 protected: 128 protected:
129 base::MessageLoop main_message_loop_; 129 base::MessageLoop main_message_loop_;
130 MessageLoopQuitter loop_quitter_; 130 MessageLoopQuitter loop_quitter_;
131 scoped_refptr<WifiDataProviderCommonWithMock> provider_; 131 scoped_refptr<WifiDataProviderCommonWithMock> provider_;
132 MockWlanApi* wlan_api_; 132 MockWlanApi* wlan_api_;
(...skipping 98 matching lines...) Expand 10 before | Expand all | Expand 10 after
231 MessageLoopQuitter loop_quitter(&main_message_loop_); 231 MessageLoopQuitter loop_quitter(&main_message_loop_);
232 WifiDataProviderManager::SetFactoryForTesting( 232 WifiDataProviderManager::SetFactoryForTesting(
233 CreateWifiDataProviderCommonWithMock); 233 CreateWifiDataProviderCommonWithMock);
234 WifiDataProviderManager::Register(&loop_quitter.callback_); 234 WifiDataProviderManager::Register(&loop_quitter.callback_);
235 main_message_loop_.Run(); 235 main_message_loop_.Run();
236 WifiDataProviderManager::Unregister(&loop_quitter.callback_); 236 WifiDataProviderManager::Unregister(&loop_quitter.callback_);
237 WifiDataProviderManager::ResetFactoryForTesting(); 237 WifiDataProviderManager::ResetFactoryForTesting();
238 } 238 }
239 239
240 } // namespace content 240 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698