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

Side by Side Diff: chrome/browser/extensions/api/enterprise_device_attributes/enterprise_device_attributes_apitest.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: Fixed another test. 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 unified diff | Download patch
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "base/strings/stringprintf.h" 5 #include "base/strings/stringprintf.h"
6 #include "chrome/browser/chromeos/policy/browser_policy_connector_chromeos.h" 6 #include "chrome/browser/chromeos/policy/browser_policy_connector_chromeos.h"
7 #include "chrome/browser/chromeos/policy/device_policy_cros_browser_test.h" 7 #include "chrome/browser/chromeos/policy/device_policy_cros_browser_test.h"
8 #include "chrome/browser/chromeos/policy/stub_enterprise_install_attributes.h" 8 #include "chrome/browser/chromeos/policy/stub_enterprise_install_attributes.h"
9 #include "chrome/browser/extensions/extension_apitest.h" 9 #include "chrome/browser/extensions/extension_apitest.h"
10 #include "chrome/browser/net/url_request_mock_util.h" 10 #include "chrome/browser/net/url_request_mock_util.h"
11 #include "chromeos/dbus/fake_session_manager_client.h" 11 #include "chromeos/dbus/fake_session_manager_client.h"
12 #include "chromeos/login/user_names.h" 12 #include "chromeos/login/user_names.h"
13 #include "components/policy/core/common/mock_configuration_policy_provider.h" 13 #include "components/policy/core/common/mock_configuration_policy_provider.h"
14 #include "components/policy/core/common/policy_types.h"
14 #include "content/public/browser/notification_service.h" 15 #include "content/public/browser/notification_service.h"
15 #include "content/public/test/test_utils.h" 16 #include "content/public/test/test_utils.h"
16 #include "extensions/browser/api_test_utils.h" 17 #include "extensions/browser/api_test_utils.h"
17 #include "extensions/browser/extension_registry.h" 18 #include "extensions/browser/extension_registry.h"
18 #include "extensions/browser/test_extension_registry_observer.h" 19 #include "extensions/browser/test_extension_registry_observer.h"
19 #include "net/test/url_request/url_request_mock_http_job.h" 20 #include "net/test/url_request/url_request_mock_http_job.h"
20 #include "policy/policy_constants.h" 21 #include "policy/policy_constants.h"
21 22
22 namespace { 23 namespace {
23 24
(...skipping 73 matching lines...) Expand 10 before | Expand all | Expand 10 after
97 GURL update_manifest_url( 98 GURL update_manifest_url(
98 net::URLRequestMockHTTPJob::GetMockUrl(update_manifest_path)); 99 net::URLRequestMockHTTPJob::GetMockUrl(update_manifest_path));
99 100
100 scoped_ptr<base::ListValue> forcelist(new base::ListValue); 101 scoped_ptr<base::ListValue> forcelist(new base::ListValue);
101 forcelist->AppendString(base::StringPrintf( 102 forcelist->AppendString(base::StringPrintf(
102 "%s;%s", kTestExtensionID, update_manifest_url.spec().c_str())); 103 "%s;%s", kTestExtensionID, update_manifest_url.spec().c_str()));
103 104
104 policy::PolicyMap policy; 105 policy::PolicyMap policy;
105 policy.Set(policy::key::kExtensionInstallForcelist, 106 policy.Set(policy::key::kExtensionInstallForcelist,
106 policy::POLICY_LEVEL_MANDATORY, policy::POLICY_SCOPE_MACHINE, 107 policy::POLICY_LEVEL_MANDATORY, policy::POLICY_SCOPE_MACHINE,
107 forcelist.release(), NULL); 108 policy::POLICY_SOURCE_CLOUD, forcelist.release(), nullptr);
108 109
109 // Set the policy and wait until the extension is installed. 110 // Set the policy and wait until the extension is installed.
110 extensions::TestExtensionRegistryObserver observer( 111 extensions::TestExtensionRegistryObserver observer(
111 ExtensionRegistry::Get(profile())); 112 ExtensionRegistry::Get(profile()));
112 policy_provider_.UpdateChromePolicy(policy); 113 policy_provider_.UpdateChromePolicy(policy);
113 observer.WaitForExtensionLoaded(); 114 observer.WaitForExtensionLoaded();
114 } 115 }
115 116
116 chromeos::FakeSessionManagerClient* const fake_session_manager_client_; 117 chromeos::FakeSessionManagerClient* const fake_session_manager_client_;
117 policy::MockConfigurationPolicyProvider policy_provider_; 118 policy::MockConfigurationPolicyProvider policy_provider_;
(...skipping 56 matching lines...) Expand 10 before | Expand all | Expand 10 after
174 const extensions::Extension* extension = 175 const extensions::Extension* extension =
175 GetExtensionByPath(registry->enabled_extensions(), extension_path); 176 GetExtensionByPath(registry->enabled_extensions(), extension_path);
176 ASSERT_FALSE(extension->install_warnings().empty()); 177 ASSERT_FALSE(extension->install_warnings().empty());
177 EXPECT_EQ( 178 EXPECT_EQ(
178 "'enterprise.deviceAttributes' is not allowed for specified install " 179 "'enterprise.deviceAttributes' is not allowed for specified install "
179 "location.", 180 "location.",
180 extension->install_warnings()[0].message); 181 extension->install_warnings()[0].message);
181 } 182 }
182 183
183 } // namespace extensions 184 } // namespace extensions
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698