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

Unified Diff: components/policy/core/common/policy_loader_mac_unittest.cc

Issue 1304843004: Add source column to chrome://policy showing the origins of policies. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 3 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: components/policy/core/common/policy_loader_mac_unittest.cc
diff --git a/components/policy/core/common/policy_loader_mac_unittest.cc b/components/policy/core/common/policy_loader_mac_unittest.cc
index ec18d311805c09ba468688d0dd664d0649bfd5bc..2e53903d05a43cc3ea355a89a9cde06dac6ce409 100644
--- a/components/policy/core/common/policy_loader_mac_unittest.cc
+++ b/components/policy/core/common/policy_loader_mac_unittest.cc
@@ -59,7 +59,8 @@ class TestHarness : public PolicyProviderTestHarness {
};
TestHarness::TestHarness()
- : PolicyProviderTestHarness(POLICY_LEVEL_MANDATORY, POLICY_SCOPE_USER) {}
+ : PolicyProviderTestHarness(POLICY_LEVEL_MANDATORY, POLICY_SCOPE_USER,
+ POLICY_SOURCE_PLATFORM) {}
bartfab (slow) 2015/09/14 14:42:27 Nit: #include "components/policy/core/common/polic
fhorschig 2015/09/16 13:52:05 Done.
TestHarness::~TestHarness() {}
@@ -187,11 +188,12 @@ TEST_F(PolicyLoaderMacTest, TestNonForcedValue) {
loop_.RunUntilIdle();
PolicyBundle expected_bundle;
expected_bundle.Get(PolicyNamespace(POLICY_DOMAIN_CHROME, std::string()))
- .Set(test_keys::kKeyString,
- POLICY_LEVEL_RECOMMENDED,
- POLICY_SCOPE_USER,
- new base::StringValue("string value"),
- NULL);
+ .SetWithSource(test_keys::kKeyString,
+ POLICY_LEVEL_RECOMMENDED,
+ POLICY_SCOPE_USER,
+ new base::StringValue("string value"),
+ NULL,
bartfab (slow) 2015/09/14 14:42:27 Nit: s/NULL/nullptr/
fhorschig 2015/09/16 13:52:05 Done.
+ POLICY_SOURCE_PLATFORM);
EXPECT_TRUE(provider_->policies().Equals(expected_bundle));
}

Powered by Google App Engine
This is Rietveld 408576698