| Index: chrome/browser/chromeos/policy/device_policy_cros_browser_test.cc
|
| diff --git a/chrome/browser/chromeos/policy/device_policy_cros_browser_test.cc b/chrome/browser/chromeos/policy/device_policy_cros_browser_test.cc
|
| index 4797ee4c6ca7356de2d18c8a60ddfe08b69293b3..782cf3d7fe9812e0085a5ad4d9b9050188f382dd 100644
|
| --- a/chrome/browser/chromeos/policy/device_policy_cros_browser_test.cc
|
| +++ b/chrome/browser/chromeos/policy/device_policy_cros_browser_test.cc
|
| @@ -14,6 +14,7 @@
|
| #include "chrome/browser/chromeos/policy/device_policy_builder.h"
|
| #include "chrome/browser/chromeos/policy/enterprise_install_attributes.h"
|
| #include "chrome/browser/chromeos/policy/proto/install_attributes.pb.h"
|
| +#include "chrome/common/chrome_paths.h"
|
| #include "chromeos/chromeos_paths.h"
|
| #include "chromeos/dbus/fake_dbus_thread_manager.h"
|
| #include "chromeos/dbus/fake_session_manager_client.h"
|
| @@ -27,13 +28,13 @@ using ::testing::Return;
|
|
|
| namespace policy {
|
|
|
| -DevicePolicyCrosTestHelper::DevicePolicyCrosTestHelper() {
|
| - CHECK(temp_dir_.CreateUniqueTempDir());
|
| -}
|
| +DevicePolicyCrosTestHelper::DevicePolicyCrosTestHelper() {}
|
|
|
| DevicePolicyCrosTestHelper::~DevicePolicyCrosTestHelper() {}
|
|
|
| void DevicePolicyCrosTestHelper::MarkAsEnterpriseOwned() {
|
| + OverridePaths();
|
| +
|
| cryptohome::SerializedInstallAttributes install_attrs_proto;
|
| cryptohome::SerializedInstallAttributes::Attribute* attribute = NULL;
|
|
|
| @@ -45,20 +46,22 @@ void DevicePolicyCrosTestHelper::MarkAsEnterpriseOwned() {
|
| attribute->set_name(EnterpriseInstallAttributes::kAttrEnterpriseUser);
|
| attribute->set_value(device_policy_.policy_data().username());
|
|
|
| - base::FilePath install_attrs_file =
|
| - temp_dir_.path().AppendASCII("install_attributes.pb");
|
| + base::FilePath install_attrs_file;
|
| + ASSERT_TRUE(
|
| + PathService::Get(chromeos::FILE_INSTALL_ATTRIBUTES, &install_attrs_file));
|
| const std::string install_attrs_blob(
|
| install_attrs_proto.SerializeAsString());
|
| ASSERT_EQ(static_cast<int>(install_attrs_blob.size()),
|
| base::WriteFile(install_attrs_file,
|
| install_attrs_blob.c_str(),
|
| install_attrs_blob.size()));
|
| - ASSERT_TRUE(PathService::Override(chromeos::FILE_INSTALL_ATTRIBUTES,
|
| - install_attrs_file));
|
| }
|
|
|
| void DevicePolicyCrosTestHelper::InstallOwnerKey() {
|
| - base::FilePath owner_key_file = temp_dir_.path().AppendASCII("owner.key");
|
| + OverridePaths();
|
| +
|
| + base::FilePath owner_key_file;
|
| + ASSERT_TRUE(PathService::Get(chromeos::FILE_OWNER_KEY, &owner_key_file));
|
| std::vector<uint8> owner_key_bits;
|
| ASSERT_TRUE(
|
| device_policy()->GetSigningKey()->ExportPublicKey(&owner_key_bits));
|
| @@ -67,7 +70,15 @@ void DevicePolicyCrosTestHelper::InstallOwnerKey() {
|
| reinterpret_cast<const char*>(vector_as_array(&owner_key_bits)),
|
| owner_key_bits.size()),
|
| static_cast<int>(owner_key_bits.size()));
|
| - ASSERT_TRUE(PathService::Override(chromeos::FILE_OWNER_KEY, owner_key_file));
|
| +}
|
| +
|
| +void DevicePolicyCrosTestHelper::OverridePaths() {
|
| + // This is usually done by ChromeBrowserMainChromeOS, but some tests
|
| + // use the overridden paths before ChromeBrowserMain starts. Make sure that
|
| + // the paths are overridden before using them.
|
| + base::FilePath user_data_dir;
|
| + ASSERT_TRUE(PathService::Get(chrome::DIR_USER_DATA, &user_data_dir));
|
| + chromeos::RegisterStubPathOverrides(user_data_dir);
|
| }
|
|
|
| DevicePolicyCrosBrowserTest::DevicePolicyCrosBrowserTest()
|
|
|