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

Unified Diff: chrome/browser/chromeos/policy/auto_enrollment_client_unittest.cc

Issue 2666093002: Remove base::FundamentalValue (Closed)
Patch Set: Rebase Created 3 years, 10 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/chromeos/policy/auto_enrollment_client_unittest.cc
diff --git a/chrome/browser/chromeos/policy/auto_enrollment_client_unittest.cc b/chrome/browser/chromeos/policy/auto_enrollment_client_unittest.cc
index 3c30b365ceca541b1b498f8f4d60e51c9942790c..47d4a46b6cf52314cf025036cd992b8956d21d79 100644
--- a/chrome/browser/chromeos/policy/auto_enrollment_client_unittest.cc
+++ b/chrome/browser/chromeos/policy/auto_enrollment_client_unittest.cc
@@ -141,8 +141,8 @@ class AutoEnrollmentClientTest : public testing::Test {
}
void VerifyCachedResult(bool should_enroll, int power_limit) {
- base::FundamentalValue value_should_enroll(should_enroll);
- base::FundamentalValue value_power_limit(power_limit);
+ base::Value value_should_enroll(should_enroll);
+ base::Value value_power_limit(power_limit);
EXPECT_TRUE(base::Value::Equals(
&value_should_enroll,
local_state_->GetUserPref(prefs::kShouldAutoEnroll)));
@@ -426,10 +426,9 @@ TEST_F(AutoEnrollmentClientTest, ReuseCachedDecision) {
EXPECT_CALL(*service_,
CreateJob(DeviceManagementRequestJob::TYPE_AUTO_ENROLLMENT, _))
.Times(0);
- local_state_->SetUserPref(prefs::kShouldAutoEnroll,
- new base::FundamentalValue(true));
+ local_state_->SetUserPref(prefs::kShouldAutoEnroll, new base::Value(true));
local_state_->SetUserPref(prefs::kAutoEnrollmentPowerLimit,
- new base::FundamentalValue(8));
+ new base::Value(8));
// Note that device state will be retrieved every time, regardless of any
// cached information. This is intentional, the idea is that device state on
@@ -447,10 +446,9 @@ TEST_F(AutoEnrollmentClientTest, ReuseCachedDecision) {
}
TEST_F(AutoEnrollmentClientTest, RetryIfPowerLargerThanCached) {
- local_state_->SetUserPref(prefs::kShouldAutoEnroll,
- new base::FundamentalValue(false));
+ local_state_->SetUserPref(prefs::kShouldAutoEnroll, new base::Value(false));
local_state_->SetUserPref(prefs::kAutoEnrollmentPowerLimit,
- new base::FundamentalValue(8));
+ new base::Value(8));
CreateClient(kStateKey, 5, 10);
ServerWillReply(-1, true, true);
ServerWillSendState(

Powered by Google App Engine
This is Rietveld 408576698