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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: content/browser/device_orientation/provider_unittest.cc

Issue 11275088: Remove implicit scoped_refptr operator T* Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 8 years, 2 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
« no previous file with comments | « content/browser/debugger/devtools_http_handler_impl.cc ('k') | content/browser/dom_storage/dom_storage_message_filter.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/browser/device_orientation/provider_unittest.cc
diff --git a/content/browser/device_orientation/provider_unittest.cc b/content/browser/device_orientation/provider_unittest.cc
index 47944cf655abbddd3cd109d6ef64d5a3783ee4dd..86732db85540973c945b67781ae6389da3ddb90e 100644
--- a/content/browser/device_orientation/provider_unittest.cc
+++ b/content/browser/device_orientation/provider_unittest.cc
@@ -87,7 +87,7 @@ class MotionUpdateChecker : public UpdateChecker {
ASSERT_EQ(DeviceData::kTypeMotion, device_data_type);
scoped_refptr<const Motion> motion(static_cast<const Motion*>(device_data));
- if (motion == NULL)
+ if (motion.get() == NULL)
motion = new Motion();
scoped_refptr<const Motion> expected(static_cast<const Motion*>(
@@ -172,7 +172,7 @@ class OrientationUpdateChecker : public UpdateChecker {
scoped_refptr<const Orientation> orientation(
static_cast<const Orientation*>(device_data));
- if (orientation == NULL)
+ if (orientation.get() == NULL)
orientation = new Orientation();
scoped_refptr<const Orientation> expected(static_cast<const Orientation*>(
@@ -216,7 +216,7 @@ class TestDataUpdateChecker : public UpdateChecker {
scoped_refptr<const TestData> test_data(
static_cast<const TestData*>(device_data));
- if (test_data == NULL)
+ if (test_data.get() == NULL)
test_data = new TestData();
scoped_refptr<const TestData> expected(static_cast<const TestData*>(
@@ -283,7 +283,7 @@ class MockDeviceDataFactory
base::AutoLock auto_lock(device_data_factory_->lock_);
if (device_data_factory_->is_failing_)
return NULL;
- return device_data_factory_->device_data_map_[device_data_type];
+ return device_data_factory_->device_data_map_[device_data_type].get();
}
private:
@@ -318,7 +318,7 @@ class DeviceOrientationProviderTest : public testing::Test {
// DataFetcherFactory factory.
void Init(ProviderImpl::DataFetcherFactory factory) {
provider_ = new ProviderImpl(factory);
- Provider::SetInstanceForTests(provider_);
+ Provider::SetInstanceForTests(provider_.get());
}
protected:
@@ -380,9 +380,9 @@ TEST_F(DeviceOrientationProviderTest, BasicPushTest) {
scoped_ptr<OrientationUpdateChecker> checker(
new OrientationUpdateChecker(&pending_expectations_));
- checker->AddExpectation(test_orientation);
- device_data_factory->SetDeviceData(test_orientation,
- DeviceData::kTypeOrientation);
+ checker->AddExpectation(test_orientation.get());
+ device_data_factory->SetDeviceData(test_orientation.get(),
+ DeviceData::kTypeOrientation);
provider_->AddObserver(checker.get());
MessageLoop::current()->Run();
@@ -421,26 +421,26 @@ TEST_F(DeviceOrientationProviderTest, MultipleObserversPushTest) {
scoped_ptr<OrientationUpdateChecker> checker_c(
new OrientationUpdateChecker(&pending_expectations_));
- checker_a->AddExpectation(test_orientations[0]);
- device_data_factory->SetDeviceData(test_orientations[0],
- DeviceData::kTypeOrientation);
+ checker_a->AddExpectation(test_orientations[0].get());
+ device_data_factory->SetDeviceData(test_orientations[0].get(),
+ DeviceData::kTypeOrientation);
provider_->AddObserver(checker_a.get());
MessageLoop::current()->Run();
- checker_a->AddExpectation(test_orientations[1]);
- checker_b->AddExpectation(test_orientations[0]);
- checker_b->AddExpectation(test_orientations[1]);
- device_data_factory->SetDeviceData(test_orientations[1],
- DeviceData::kTypeOrientation);
+ checker_a->AddExpectation(test_orientations[1].get());
+ checker_b->AddExpectation(test_orientations[0].get());
+ checker_b->AddExpectation(test_orientations[1].get());
+ device_data_factory->SetDeviceData(test_orientations[1].get(),
+ DeviceData::kTypeOrientation);
provider_->AddObserver(checker_b.get());
MessageLoop::current()->Run();
provider_->RemoveObserver(checker_a.get());
- checker_b->AddExpectation(test_orientations[2]);
- checker_c->AddExpectation(test_orientations[1]);
- checker_c->AddExpectation(test_orientations[2]);
- device_data_factory->SetDeviceData(test_orientations[2],
- DeviceData::kTypeOrientation);
+ checker_b->AddExpectation(test_orientations[2].get());
+ checker_c->AddExpectation(test_orientations[1].get());
+ checker_c->AddExpectation(test_orientations[2].get());
+ device_data_factory->SetDeviceData(test_orientations[2].get(),
+ DeviceData::kTypeOrientation);
provider_->AddObserver(checker_c.get());
MessageLoop::current()->Run();
@@ -473,9 +473,9 @@ TEST_F(DeviceOrientationProviderTest, FailingFirstDataTypeTest) {
provider_->AddObserver(test_data_checker.get());
MessageLoop::current()->Run();
- orientation_checker->AddExpectation(test_orientation);
- device_data_factory->SetDeviceData(test_orientation,
- DeviceData::kTypeOrientation);
+ orientation_checker->AddExpectation(test_orientation.get());
+ device_data_factory->SetDeviceData(test_orientation.get(),
+ DeviceData::kTypeOrientation);
provider_->AddObserver(orientation_checker.get());
MessageLoop::current()->Run();
@@ -510,15 +510,15 @@ TEST_F(DeviceOrientationProviderTest, MAYBE_ObserverNotRemoved) {
scoped_ptr<OrientationUpdateChecker> checker(
new OrientationUpdateChecker(&pending_expectations_));
- checker->AddExpectation(test_orientation);
- device_data_factory->SetDeviceData(test_orientation,
- DeviceData::kTypeOrientation);
+ checker->AddExpectation(test_orientation.get());
+ device_data_factory->SetDeviceData(test_orientation.get(),
+ DeviceData::kTypeOrientation);
provider_->AddObserver(checker.get());
MessageLoop::current()->Run();
- checker->AddExpectation(test_orientation2);
- device_data_factory->SetDeviceData(test_orientation2,
- DeviceData::kTypeOrientation);
+ checker->AddExpectation(test_orientation2.get());
+ device_data_factory->SetDeviceData(test_orientation2.get(),
+ DeviceData::kTypeOrientation);
MessageLoop::current()->Run();
MockDeviceDataFactory::SetCurInstance(NULL);
@@ -548,9 +548,9 @@ TEST_F(DeviceOrientationProviderTest, MAYBE_StartFailing) {
scoped_ptr<OrientationUpdateChecker> checker_b(new OrientationUpdateChecker(
&pending_expectations_));
- device_data_factory->SetDeviceData(test_orientation,
- DeviceData::kTypeOrientation);
- checker_a->AddExpectation(test_orientation);
+ device_data_factory->SetDeviceData(test_orientation.get(),
+ DeviceData::kTypeOrientation);
+ checker_a->AddExpectation(test_orientation.get());
provider_->AddObserver(checker_a.get());
MessageLoop::current()->Run();
@@ -590,17 +590,17 @@ TEST_F(DeviceOrientationProviderTest, StartStopStart) {
scoped_ptr<OrientationUpdateChecker> checker_b(new OrientationUpdateChecker(
&pending_expectations_));
- checker_a->AddExpectation(test_orientation);
- device_data_factory->SetDeviceData(test_orientation,
- DeviceData::kTypeOrientation);
+ checker_a->AddExpectation(test_orientation.get());
+ device_data_factory->SetDeviceData(test_orientation.get(),
+ DeviceData::kTypeOrientation);
provider_->AddObserver(checker_a.get());
MessageLoop::current()->Run();
provider_->RemoveObserver(checker_a.get()); // This stops the Provider.
- checker_b->AddExpectation(test_orientation2);
- device_data_factory->SetDeviceData(test_orientation2,
- DeviceData::kTypeOrientation);
+ checker_b->AddExpectation(test_orientation2.get());
+ device_data_factory->SetDeviceData(test_orientation2.get(),
+ DeviceData::kTypeOrientation);
provider_->AddObserver(checker_b.get());
MessageLoop::current()->Run();
@@ -630,14 +630,16 @@ TEST_F(DeviceOrientationProviderTest, MotionAlwaysFires) {
scoped_ptr<MotionUpdateChecker> checker(new MotionUpdateChecker(
&pending_expectations_));
- device_data_factory->SetDeviceData(test_motion, DeviceData::kTypeMotion);
- checker->AddExpectation(test_motion);
+ device_data_factory->
+ SetDeviceData(test_motion.get(), DeviceData::kTypeMotion);
+ checker->AddExpectation(test_motion.get());
provider_->AddObserver(checker.get());
MessageLoop::current()->Run();
// The observer should receive the same motion again.
- device_data_factory->SetDeviceData(test_motion, DeviceData::kTypeMotion);
- checker->AddExpectation(test_motion);
+ device_data_factory->
+ SetDeviceData(test_motion.get(), DeviceData::kTypeMotion);
+ checker->AddExpectation(test_motion.get());
MessageLoop::current()->Run();
provider_->RemoveObserver(checker.get());
@@ -680,23 +682,23 @@ TEST_F(DeviceOrientationProviderTest, OrientationSignificantlyDifferent) {
scoped_ptr<OrientationUpdateChecker> checker_b(new OrientationUpdateChecker(
&pending_expectations_));
- device_data_factory->SetDeviceData(first_orientation,
- DeviceData::kTypeOrientation);
- checker_a->AddExpectation(first_orientation);
+ device_data_factory->SetDeviceData(first_orientation.get(),
+ DeviceData::kTypeOrientation);
+ checker_a->AddExpectation(first_orientation.get());
provider_->AddObserver(checker_a.get());
MessageLoop::current()->Run();
// The observers should not see this insignificantly different orientation.
- device_data_factory->SetDeviceData(second_orientation,
- DeviceData::kTypeOrientation);
- checker_b->AddExpectation(first_orientation);
+ device_data_factory->SetDeviceData(second_orientation.get(),
+ DeviceData::kTypeOrientation);
+ checker_b->AddExpectation(first_orientation.get());
provider_->AddObserver(checker_b.get());
MessageLoop::current()->Run();
- device_data_factory->SetDeviceData(third_orientation,
- DeviceData::kTypeOrientation);
- checker_a->AddExpectation(third_orientation);
- checker_b->AddExpectation(third_orientation);
+ device_data_factory->SetDeviceData(third_orientation.get(),
+ DeviceData::kTypeOrientation);
+ checker_a->AddExpectation(third_orientation.get());
+ checker_b->AddExpectation(third_orientation.get());
MessageLoop::current()->Run();
provider_->RemoveObserver(checker_a.get());
« no previous file with comments | « content/browser/debugger/devtools_http_handler_impl.cc ('k') | content/browser/dom_storage/dom_storage_message_filter.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698