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

Unified Diff: chrome/browser/managed_mode/managed_mode_unittest.cc

Issue 16296002: Update chrome/ to use scoped_refptr<T>::get() rather than implicit "operator T*" (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebased Created 7 years, 7 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/io_thread.cc ('k') | chrome/browser/managed_mode/managed_user_service_unittest.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 4e26107bb07cb170ed80442053d98e2f9bfc54d4..d1d0241d23270360527cd6ba9192edbcd1edc7fb 100644
--- a/chrome/browser/managed_mode/managed_mode_unittest.cc
+++ b/chrome/browser/managed_mode/managed_mode_unittest.cc
@@ -129,7 +129,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);
}
@@ -178,7 +178,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) {
@@ -197,7 +197,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_);
}
@@ -221,7 +221,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/io_thread.cc ('k') | chrome/browser/managed_mode/managed_user_service_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698