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

Unified Diff: chrome/browser/policy/managed_mode_policy_provider_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
Index: chrome/browser/policy/managed_mode_policy_provider_unittest.cc
diff --git a/chrome/browser/policy/managed_mode_policy_provider_unittest.cc b/chrome/browser/policy/managed_mode_policy_provider_unittest.cc
index 92782a1c2d081d48ef7ffff81ada8f98d907e9b8..8bc72930838b7922fd5c5d13ba9fcbbbcdc5d9d0 100644
--- a/chrome/browser/policy/managed_mode_policy_provider_unittest.cc
+++ b/chrome/browser/policy/managed_mode_policy_provider_unittest.cc
@@ -66,7 +66,7 @@ void TestHarness::SetUp() {
ConfigurationPolicyProvider* TestHarness::CreateProvider(
const PolicyDefinitionList* policy_definition_list) {
- return new ManagedModePolicyProvider(pref_store_);
+ return new ManagedModePolicyProvider(pref_store_.get());
}
void TestHarness::InstallEmptyPolicy() {}
@@ -124,8 +124,7 @@ INSTANTIATE_TEST_CASE_P(
class ManagedModePolicyProviderAPITest : public PolicyTestBase {
protected:
ManagedModePolicyProviderAPITest()
- : pref_store_(new TestingPrefStore),
- provider_(pref_store_) {
+ : pref_store_(new TestingPrefStore), provider_(pref_store_.get()) {
pref_store_->SetInitializationCompleted();
}
virtual ~ManagedModePolicyProviderAPITest() {}
@@ -170,7 +169,7 @@ TEST_F(ManagedModePolicyProviderAPITest, SetPolicy) {
EXPECT_TRUE(provider_.policies().Equals(expected_bundle));
// A newly-created provider should have the same policies.
- ManagedModePolicyProvider new_provider(pref_store_);
+ ManagedModePolicyProvider new_provider(pref_store_.get());
new_provider.Init();
EXPECT_TRUE(new_provider.policies().Equals(expected_bundle));

Powered by Google App Engine
This is Rietveld 408576698