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

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

Issue 2772673002: mash: switch to the new pref service (Closed)
Patch Set: Address review comments 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 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 0e4a8649c4be3072144397ddc292664febc13187..4838e36a4643e1d812e6abb40a582c8f25b83346 100644
--- a/chrome/browser/chromeos/preferences_chromeos_browsertest.cc
+++ b/chrome/browser/chromeos/preferences_chromeos_browsertest.cc
@@ -283,43 +283,4 @@ IN_PROC_BROWSER_TEST_F(PreferencesTest, MultiProfiles) {
CheckLocalStateCorrespondsToPrefs(prefs1);
}
-IN_PROC_BROWSER_TEST_F(PreferencesServiceBrowserTest, Basic) {
- constexpr int kInitialValue = 1;
- PrefService* pref_service = browser()->profile()->GetPrefs();
- pref_service->SetInteger(prefs::kMouseSensitivity, kInitialValue);
-
- prefs::mojom::PreferencesServiceFactoryPtr factory_a;
- connector()->BindInterface(prefs::mojom::kServiceName, &factory_a);
- scoped_refptr<preferences::PrefClientStore> pref_store_a =
- new preferences::PrefClientStore(std::move(factory_a));
- pref_store_a->Subscribe({prefs::kMouseSensitivity});
- WaitForPrefChange(pref_store_a.get(), prefs::kMouseSensitivity);
-
- prefs::mojom::PreferencesServiceFactoryPtr factory_b;
- connector()->BindInterface(prefs::mojom::kServiceName, &factory_b);
- scoped_refptr<preferences::PrefClientStore> pref_store_b =
- new preferences::PrefClientStore(std::move(factory_b));
- pref_store_b->Subscribe({prefs::kMouseSensitivity});
- WaitForPrefChange(pref_store_b.get(), prefs::kMouseSensitivity);
-
- int value_a = 0;
- ASSERT_TRUE(GetIntegerPrefValue(pref_store_a.get(), prefs::kMouseSensitivity,
- &value_a));
- EXPECT_EQ(kInitialValue, value_a);
-
- int value_b = 0;
- ASSERT_TRUE(GetIntegerPrefValue(pref_store_b.get(), prefs::kMouseSensitivity,
- &value_b));
- EXPECT_EQ(kInitialValue, value_b);
-
- const int kTestValue = 42;
- pref_store_a->SetValue(prefs::kMouseSensitivity,
- base::MakeUnique<base::Value>(kTestValue),
- WriteablePrefStore::DEFAULT_PREF_WRITE_FLAGS);
- WaitForPrefChange(pref_store_b.get(), prefs::kMouseSensitivity);
- ASSERT_TRUE(GetIntegerPrefValue(pref_store_b.get(), prefs::kMouseSensitivity,
- &value_b));
- EXPECT_EQ(kTestValue, value_b);
-}
-
} // namespace chromeos
« no previous file with comments | « chrome/browser/chrome_content_browser_manifest_overlay.json ('k') | chrome/browser/prefs/active_profile_pref_service.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698