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

Side by Side Diff: chrome/browser/chromeos/policy/cloud_external_data_policy_observer_unittest.cc

Issue 2664753002: Remove base::StringValue (Closed)
Patch Set: Rebase Created 3 years, 9 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 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 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 "chrome/browser/chromeos/policy/cloud_external_data_policy_observer.h" 5 #include "chrome/browser/chromeos/policy/cloud_external_data_policy_observer.h"
6 6
7 #include <utility> 7 #include <utility>
8 #include <vector> 8 #include <vector>
9 9
10 #include "base/files/file_path.h" 10 #include "base/files/file_path.h"
(...skipping 336 matching lines...) Expand 10 before | Expand all | Expand 10 after
347 content::NotificationService::AllSources(), 347 content::NotificationService::AllSources(),
348 content::Details<Profile>(profile_.get())); 348 content::Details<Profile>(profile_.get()));
349 } 349 }
350 350
351 void CloudExternalDataPolicyObserverTest::SetRegularUserAvatarPolicy( 351 void CloudExternalDataPolicyObserverTest::SetRegularUserAvatarPolicy(
352 const std::string& value) { 352 const std::string& value) {
353 PolicyMap policy_map; 353 PolicyMap policy_map;
354 if (!value.empty()) { 354 if (!value.empty()) {
355 policy_map.Set(key::kUserAvatarImage, POLICY_LEVEL_MANDATORY, 355 policy_map.Set(key::kUserAvatarImage, POLICY_LEVEL_MANDATORY,
356 POLICY_SCOPE_USER, POLICY_SOURCE_CLOUD, 356 POLICY_SCOPE_USER, POLICY_SOURCE_CLOUD,
357 base::MakeUnique<base::StringValue>(value), 357 base::MakeUnique<base::Value>(value),
358 external_data_manager_.CreateExternalDataFetcher( 358 external_data_manager_.CreateExternalDataFetcher(
359 key::kUserAvatarImage)); 359 key::kUserAvatarImage));
360 } 360 }
361 user_policy_provider_.UpdateChromePolicy(policy_map); 361 user_policy_provider_.UpdateChromePolicy(policy_map);
362 } 362 }
363 363
364 void CloudExternalDataPolicyObserverTest::LogInAsRegularUser() { 364 void CloudExternalDataPolicyObserverTest::LogInAsRegularUser() {
365 user_manager_->AddUser(AccountId::FromUserEmail(kRegularUserID)); 365 user_manager_->AddUser(AccountId::FromUserEmail(kRegularUserID));
366 366
367 PolicyServiceImpl::Providers providers; 367 PolicyServiceImpl::Providers providers;
(...skipping 581 matching lines...) Expand 10 before | Expand all | Expand 10 after
949 949
950 EXPECT_TRUE(set_calls_.empty()); 950 EXPECT_TRUE(set_calls_.empty());
951 EXPECT_TRUE(cleared_calls_.empty()); 951 EXPECT_TRUE(cleared_calls_.empty());
952 ASSERT_EQ(1u, fetched_calls_.size()); 952 ASSERT_EQ(1u, fetched_calls_.size());
953 EXPECT_EQ(kRegularUserID, fetched_calls_.front().first); 953 EXPECT_EQ(kRegularUserID, fetched_calls_.front().first);
954 EXPECT_EQ(avatar_policy_2_data_, fetched_calls_.front().second); 954 EXPECT_EQ(avatar_policy_2_data_, fetched_calls_.front().second);
955 ClearObservations(); 955 ClearObservations();
956 } 956 }
957 957
958 } // namespace policy 958 } // namespace policy
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698