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

Unified Diff: chrome/browser/chromeos/preferences_chromeos_browsertest.cc

Issue 2799143002: Improve profile stats performace. (Closed)
Patch Set: Always Created 3 years, 8 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/preferences_chromeos_browsertest.cc
diff --git a/chrome/browser/chromeos/preferences_chromeos_browsertest.cc b/chrome/browser/chromeos/preferences_chromeos_browsertest.cc
index fe2654de0fbe1243b07ed08819a90572dcd30cfa..d7aa1b8d1377de313da29ed33e4b1eebb3059278 100644
--- a/chrome/browser/chromeos/preferences_chromeos_browsertest.cc
+++ b/chrome/browser/chromeos/preferences_chromeos_browsertest.cc
@@ -246,23 +246,23 @@ IN_PROC_BROWSER_TEST_F(PreferencesTest, MultiProfiles) {
// Check that changing prefs of the active user doesn't affect prefs of the
// inactive user.
std::unique_ptr<base::DictionaryValue> prefs_backup =
- prefs1->GetPreferenceValues();
+ prefs1->GetPreferenceValues(true);
SetPrefs(prefs2, false);
CheckSettingsCorrespondToPrefs(prefs2);
- EXPECT_TRUE(prefs_backup->Equals(prefs1->GetPreferenceValues().get()));
+ EXPECT_TRUE(prefs_backup->Equals(prefs1->GetPreferenceValues(true).get()));
SetPrefs(prefs2, true);
CheckSettingsCorrespondToPrefs(prefs2);
- EXPECT_TRUE(prefs_backup->Equals(prefs1->GetPreferenceValues().get()));
+ EXPECT_TRUE(prefs_backup->Equals(prefs1->GetPreferenceValues(true).get()));
// Check that changing prefs of the inactive user doesn't affect prefs of the
// active user.
- prefs_backup = prefs2->GetPreferenceValues();
+ prefs_backup = prefs2->GetPreferenceValues(true);
SetPrefs(prefs1, true);
CheckSettingsCorrespondToPrefs(prefs2);
- EXPECT_TRUE(prefs_backup->Equals(prefs2->GetPreferenceValues().get()));
+ EXPECT_TRUE(prefs_backup->Equals(prefs2->GetPreferenceValues(true).get()));
SetPrefs(prefs1, false);
CheckSettingsCorrespondToPrefs(prefs2);
- EXPECT_TRUE(prefs_backup->Equals(prefs2->GetPreferenceValues().get()));
+ EXPECT_TRUE(prefs_backup->Equals(prefs2->GetPreferenceValues(true).get()));
// Check that changing non-owner prefs doesn't change corresponding local
// state prefs and vice versa.

Powered by Google App Engine
This is Rietveld 408576698