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

Unified Diff: chrome/browser/managed_mode/managed_mode_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 | « chrome/browser/intents/cws_intents_registry.cc ('k') | chrome/browser/memory_purger.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/managed_mode/managed_mode_unittest.cc
diff --git a/chrome/browser/managed_mode/managed_mode_unittest.cc b/chrome/browser/managed_mode/managed_mode_unittest.cc
index 27200236f8f84decb588ab247cde512c6f560599..587b46ad7e2e59a86c0ab895333ccdeecee49753 100644
--- a/chrome/browser/managed_mode/managed_mode_unittest.cc
+++ b/chrome/browser/managed_mode/managed_mode_unittest.cc
@@ -127,7 +127,7 @@ class ManagedModeTest : public ::testing::Test {
base::Callback<void(bool)> CreateExpectedCallback(bool success) {
scoped_refptr<MockCallback> callback = CreateCallback();
- EXPECT_CALL(*callback, DidEnterManagedMode(success));
+ EXPECT_CALL(*callback.get(), DidEnterManagedMode(success));
return base::Bind(&MockCallback::CheckManagedMode, callback);
}
@@ -175,7 +175,7 @@ TEST_F(ManagedModeTest, QueueRequests) {
&managed_mode_profile_,
base::Bind(&MockCallback::CheckManagedMode, callback));
// The callbacks should run as soon as |other_browser| is closed.
- EXPECT_CALL(*callback, DidEnterManagedMode(true)).Times(2);
+ EXPECT_CALL(*callback.get(), DidEnterManagedMode(true)).Times(2);
}
TEST_F(ManagedModeTest, OpenNewBrowser) {
@@ -194,7 +194,7 @@ TEST_F(ManagedModeTest, OpenNewBrowser) {
&managed_mode_profile_);
// Opening another browser should cancel entering managed mode.
- EXPECT_CALL(*callback, DidEnterManagedMode(false));
+ EXPECT_CALL(*callback.get(), DidEnterManagedMode(false));
BrowserFixture yet_another_browser(&managed_mode_, &other_profile_);
}
@@ -218,7 +218,7 @@ TEST_F(ManagedModeTest, DifferentProfile) {
// The first request should still succeed as soon as the other browser is
// closed.
- EXPECT_CALL(*callback, DidEnterManagedMode(true));
+ EXPECT_CALL(*callback.get(), DidEnterManagedMode(true));
}
TEST_F(ManagedModeTest, Cancelled) {
« no previous file with comments | « chrome/browser/intents/cws_intents_registry.cc ('k') | chrome/browser/memory_purger.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698