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

Unified Diff: chrome/browser/geolocation/win7_location_api_unittest_win.cc

Issue 5676004: Cleanup: Remove unneeded browser_thread.h usage. (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: fix deps Created 10 years 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: chrome/browser/geolocation/win7_location_api_unittest_win.cc
===================================================================
--- chrome/browser/geolocation/win7_location_api_unittest_win.cc (revision 68879)
+++ chrome/browser/geolocation/win7_location_api_unittest_win.cc (working copy)
@@ -2,9 +2,10 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
+#include <Objbase.h>
+
#include <algorithm>
#include <cmath>
-#include <Objbase.h>
#include "base/compiler_specific.h"
#include "base/logging.h"
@@ -13,7 +14,6 @@
#include "base/time.h"
#include "base/win_util.h"
#include "chrome/common/geoposition.h"
-#include "chrome/browser/browser_thread.h"
#include "chrome/browser/geolocation/win7_location_api_win.h"
#include "testing/gmock/include/gmock/gmock.h"
#include "testing/gtest/include/gtest/gtest.h"
@@ -25,12 +25,12 @@
using testing::Return;
class MockLatLongReport : public ILatLongReport {
-public:
+ public:
MockLatLongReport() : ref_count_(1) {
ON_CALL(*this, GetAltitude(_))
.WillByDefault(Invoke(this, &MockLatLongReport::GetAltitudeValid));
ON_CALL(*this, GetAltitudeError(_))
- .WillByDefault(Invoke(this,
+ .WillByDefault(Invoke(this,
&MockLatLongReport::GetAltitudeErrorValid));
ON_CALL(*this, GetErrorRadius(_))
.WillByDefault(Invoke(this, &MockLatLongReport::GetErrorRadiusValid));
@@ -38,7 +38,7 @@
.WillByDefault(Invoke(this, &MockLatLongReport::GetLatitudeValid));
ON_CALL(*this, GetLongitude(_))
.WillByDefault(Invoke(this, &MockLatLongReport::GetLongitudeValid));
- ON_CALL(*this, GetValue(_,_))
+ ON_CALL(*this, GetValue(_, _))
.WillByDefault(Invoke(this, &MockLatLongReport::GetValueValid));
ON_CALL(*this, Release())
.WillByDefault(Invoke(this, &MockLatLongReport::ReleaseInternal));
@@ -120,7 +120,7 @@
delete this;
return new_ref_count;
}
-
+
LONG ref_count_;
};
@@ -136,7 +136,7 @@
ON_CALL(*this, AddRef())
.WillByDefault(Invoke(this, &MockReport::AddRefInternal));
}
-
+
// ILocationReport
MOCK_METHOD1_WITH_CALLTYPE(STDMETHODCALLTYPE,
GetSensorID,
@@ -160,7 +160,7 @@
MockLatLongReport* mock_lat_long_report_;
-private:
+ private:
~MockReport() {
mock_lat_long_report_->Release();
}
@@ -242,10 +242,10 @@
MockReport* mock_report_;
private:
- ~MockLocation() {
+ ~MockLocation() {
mock_report_->Release();
}
-
+
HRESULT GetReportValid(REFIID report_type,
ILocationReport** location_report) {
*location_report = reinterpret_cast<ILocationReport*>(mock_report_);
@@ -288,7 +288,7 @@
locator_->Release();
api_.reset();
}
- ~GeolocationApiWin7Tests(){
+ ~GeolocationApiWin7Tests() {
}
protected:
Win7LocationApi* CreateMock() {

Powered by Google App Engine
This is Rietveld 408576698