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

Unified Diff: chrome/browser/sync/test/integration/two_client_themes_sync_test.cc

Issue 244893004: Improve some naming (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: mac and win Created 6 years, 7 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
« no previous file with comments | « chrome/browser/sync/test/integration/themes_helper.cc ('k') | chrome/browser/themes/theme_service.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/sync/test/integration/two_client_themes_sync_test.cc
diff --git a/chrome/browser/sync/test/integration/two_client_themes_sync_test.cc b/chrome/browser/sync/test/integration/two_client_themes_sync_test.cc
index 04c437fc512861e0696ba48b6c9b8f97736c3dd2..bd06991c7c317a3f36e5a516c19349005e669ba9 100644
--- a/chrome/browser/sync/test/integration/two_client_themes_sync_test.cc
+++ b/chrome/browser/sync/test/integration/two_client_themes_sync_test.cc
@@ -15,10 +15,10 @@ using themes_helper::HasOrWillHaveCustomTheme;
using themes_helper::ThemeIsPendingInstall;
using themes_helper::UseCustomTheme;
using themes_helper::UseDefaultTheme;
-using themes_helper::UseNativeTheme;
+using themes_helper::UseSystemTheme;
using themes_helper::UsingCustomTheme;
using themes_helper::UsingDefaultTheme;
-using themes_helper::UsingNativeTheme;
+using themes_helper::UsingSystemTheme;
class TwoClientThemesSyncTest : public SyncTest {
public:
@@ -82,16 +82,16 @@ IN_PROC_BROWSER_TEST_F(TwoClientThemesSyncTest, NativeTheme) {
ASSERT_TRUE(AwaitQuiescence());
- UseNativeTheme(GetProfile(0));
- UseNativeTheme(verifier());
- ASSERT_TRUE(UsingNativeTheme(GetProfile(0)));
- ASSERT_TRUE(UsingNativeTheme(verifier()));
+ UseSystemTheme(GetProfile(0));
+ UseSystemTheme(verifier());
+ ASSERT_TRUE(UsingSystemTheme(GetProfile(0)));
+ ASSERT_TRUE(UsingSystemTheme(verifier()));
ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
- ASSERT_TRUE(UsingNativeTheme(GetProfile(0)));
- ASSERT_TRUE(UsingNativeTheme(GetProfile(1)));
- ASSERT_TRUE(UsingNativeTheme(verifier()));
+ ASSERT_TRUE(UsingSystemTheme(GetProfile(0)));
+ ASSERT_TRUE(UsingSystemTheme(GetProfile(1)));
+ ASSERT_TRUE(UsingSystemTheme(verifier()));
}
// TCM ID - 7247455.
@@ -126,9 +126,9 @@ IN_PROC_BROWSER_TEST_F(TwoClientThemesSyncTest, NativeDefaultRace) {
#endif // OS_CHROMEOS
ASSERT_TRUE(SetupSync()) << "SetupSync() failed.";
- UseNativeTheme(GetProfile(0));
+ UseSystemTheme(GetProfile(0));
UseDefaultTheme(GetProfile(1));
- ASSERT_TRUE(UsingNativeTheme(GetProfile(0)));
+ ASSERT_TRUE(UsingSystemTheme(GetProfile(0)));
ASSERT_TRUE(UsingDefaultTheme(GetProfile(1)));
ASSERT_TRUE(AwaitQuiescence());
@@ -136,8 +136,8 @@ IN_PROC_BROWSER_TEST_F(TwoClientThemesSyncTest, NativeDefaultRace) {
// TODO(akalin): Add function that compares two profiles to see if
// they're at the same state.
- ASSERT_EQ(UsingNativeTheme(GetProfile(0)),
- UsingNativeTheme(GetProfile(1)));
+ ASSERT_EQ(UsingSystemTheme(GetProfile(0)),
+ UsingSystemTheme(GetProfile(1)));
ASSERT_EQ(UsingDefaultTheme(GetProfile(0)),
UsingDefaultTheme(GetProfile(1)));
}
@@ -152,9 +152,9 @@ IN_PROC_BROWSER_TEST_F(TwoClientThemesSyncTest, CustomNativeRace) {
ASSERT_TRUE(SetupSync()) << "SetupSync() failed.";
UseCustomTheme(GetProfile(0), 0);
- UseNativeTheme(GetProfile(1));
+ UseSystemTheme(GetProfile(1));
ASSERT_EQ(GetCustomTheme(0), GetThemeID(GetProfile(0)));
- ASSERT_TRUE(UsingNativeTheme(GetProfile(1)));
+ ASSERT_TRUE(UsingSystemTheme(GetProfile(1)));
ASSERT_TRUE(AwaitQuiescence());
« no previous file with comments | « chrome/browser/sync/test/integration/themes_helper.cc ('k') | chrome/browser/themes/theme_service.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698