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

Side by Side Diff: chrome/browser/sync/test/integration/single_client_themes_sync_test.cc

Issue 2365763002: [Sync] Removed passphrase helper methods, removed ((n)) pattern, and fixed lint violations. (Closed)
Patch Set: Removed useless pass-through accessors. Created 4 years, 2 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 (c) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 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 "base/macros.h" 5 #include "base/macros.h"
6 #include "build/build_config.h" 6 #include "build/build_config.h"
7 #include "chrome/browser/sync/test/integration/sync_integration_test_util.h" 7 #include "chrome/browser/sync/test/integration/sync_integration_test_util.h"
8 #include "chrome/browser/sync/test/integration/sync_test.h" 8 #include "chrome/browser/sync/test/integration/sync_test.h"
9 #include "chrome/browser/sync/test/integration/themes_helper.h" 9 #include "chrome/browser/sync/test/integration/themes_helper.h"
10 #include "components/browser_sync/profile_sync_service.h" 10 #include "components/browser_sync/profile_sync_service.h"
(...skipping 25 matching lines...) Expand all
36 ASSERT_TRUE(SetupSync()) << "SetupSync() failed."; 36 ASSERT_TRUE(SetupSync()) << "SetupSync() failed.";
37 37
38 ASSERT_FALSE(UsingCustomTheme(GetProfile(0))); 38 ASSERT_FALSE(UsingCustomTheme(GetProfile(0)));
39 ASSERT_FALSE(UsingCustomTheme(verifier())); 39 ASSERT_FALSE(UsingCustomTheme(verifier()));
40 40
41 UseCustomTheme(GetProfile(0), 0); 41 UseCustomTheme(GetProfile(0), 0);
42 UseCustomTheme(verifier(), 0); 42 UseCustomTheme(verifier(), 0);
43 ASSERT_EQ(GetCustomTheme(0), GetThemeID(GetProfile(0))); 43 ASSERT_EQ(GetCustomTheme(0), GetThemeID(GetProfile(0)));
44 ASSERT_EQ(GetCustomTheme(0), GetThemeID(verifier())); 44 ASSERT_EQ(GetCustomTheme(0), GetThemeID(verifier()));
45 45
46 ASSERT_TRUE(AwaitCommitActivityCompletion(GetSyncService((0)))); 46 ASSERT_TRUE(AwaitCommitActivityCompletion(GetSyncService(0)));
47 47
48 ASSERT_EQ(GetCustomTheme(0), GetThemeID(GetProfile(0))); 48 ASSERT_EQ(GetCustomTheme(0), GetThemeID(GetProfile(0)));
49 ASSERT_EQ(GetCustomTheme(0), GetThemeID(verifier())); 49 ASSERT_EQ(GetCustomTheme(0), GetThemeID(verifier()));
50 } 50 }
51 51
52 // TODO(sync): Fails on Chrome OS. See http://crbug.com/84575. 52 // TODO(sync): Fails on Chrome OS. See http://crbug.com/84575.
53 #if defined(OS_CHROMEOS) 53 #if defined(OS_CHROMEOS)
54 IN_PROC_BROWSER_TEST_F(SingleClientThemesSyncTest, DISABLED_NativeTheme) { 54 IN_PROC_BROWSER_TEST_F(SingleClientThemesSyncTest, DISABLED_NativeTheme) {
55 #else 55 #else
56 IN_PROC_BROWSER_TEST_F(SingleClientThemesSyncTest, NativeTheme) { 56 IN_PROC_BROWSER_TEST_F(SingleClientThemesSyncTest, NativeTheme) {
57 #endif // OS_CHROMEOS 57 #endif // OS_CHROMEOS
58 ASSERT_TRUE(SetupSync()) << "SetupSync() failed."; 58 ASSERT_TRUE(SetupSync()) << "SetupSync() failed.";
59 59
60 UseCustomTheme(GetProfile(0), 0); 60 UseCustomTheme(GetProfile(0), 0);
61 UseCustomTheme(verifier(), 0); 61 UseCustomTheme(verifier(), 0);
62 ASSERT_FALSE(UsingSystemTheme(GetProfile(0))); 62 ASSERT_FALSE(UsingSystemTheme(GetProfile(0)));
63 ASSERT_FALSE(UsingSystemTheme(verifier())); 63 ASSERT_FALSE(UsingSystemTheme(verifier()));
64 64
65 ASSERT_TRUE(AwaitCommitActivityCompletion(GetSyncService((0)))); 65 ASSERT_TRUE(AwaitCommitActivityCompletion(GetSyncService(0)));
66 66
67 UseSystemTheme(GetProfile(0)); 67 UseSystemTheme(GetProfile(0));
68 UseSystemTheme(verifier()); 68 UseSystemTheme(verifier());
69 ASSERT_TRUE(UsingSystemTheme(GetProfile(0))); 69 ASSERT_TRUE(UsingSystemTheme(GetProfile(0)));
70 ASSERT_TRUE(UsingSystemTheme(verifier())); 70 ASSERT_TRUE(UsingSystemTheme(verifier()));
71 71
72 ASSERT_TRUE(AwaitCommitActivityCompletion(GetSyncService((0)))); 72 ASSERT_TRUE(AwaitCommitActivityCompletion(GetSyncService(0)));
73 73
74 ASSERT_TRUE(UsingSystemTheme(GetProfile(0))); 74 ASSERT_TRUE(UsingSystemTheme(GetProfile(0)));
75 ASSERT_TRUE(UsingSystemTheme(verifier())); 75 ASSERT_TRUE(UsingSystemTheme(verifier()));
76 } 76 }
77 77
78 IN_PROC_BROWSER_TEST_F(SingleClientThemesSyncTest, DefaultTheme) { 78 IN_PROC_BROWSER_TEST_F(SingleClientThemesSyncTest, DefaultTheme) {
79 ASSERT_TRUE(SetupSync()) << "SetupSync() failed."; 79 ASSERT_TRUE(SetupSync()) << "SetupSync() failed.";
80 80
81 UseCustomTheme(GetProfile(0), 0); 81 UseCustomTheme(GetProfile(0), 0);
82 UseCustomTheme(verifier(), 0); 82 UseCustomTheme(verifier(), 0);
83 ASSERT_FALSE(UsingDefaultTheme(GetProfile(0))); 83 ASSERT_FALSE(UsingDefaultTheme(GetProfile(0)));
84 ASSERT_FALSE(UsingDefaultTheme(verifier())); 84 ASSERT_FALSE(UsingDefaultTheme(verifier()));
85 85
86 ASSERT_TRUE(AwaitCommitActivityCompletion(GetSyncService((0)))); 86 ASSERT_TRUE(AwaitCommitActivityCompletion(GetSyncService(0)));
87 87
88 UseDefaultTheme(GetProfile(0)); 88 UseDefaultTheme(GetProfile(0));
89 UseDefaultTheme(verifier()); 89 UseDefaultTheme(verifier());
90 ASSERT_TRUE(UsingDefaultTheme(GetProfile(0))); 90 ASSERT_TRUE(UsingDefaultTheme(GetProfile(0)));
91 ASSERT_TRUE(UsingDefaultTheme(verifier())); 91 ASSERT_TRUE(UsingDefaultTheme(verifier()));
92 92
93 ASSERT_TRUE(AwaitCommitActivityCompletion(GetSyncService((0)))); 93 ASSERT_TRUE(AwaitCommitActivityCompletion(GetSyncService(0)));
94 94
95 ASSERT_TRUE(UsingDefaultTheme(GetProfile(0))); 95 ASSERT_TRUE(UsingDefaultTheme(GetProfile(0)));
96 ASSERT_TRUE(UsingDefaultTheme(verifier())); 96 ASSERT_TRUE(UsingDefaultTheme(verifier()));
97 } 97 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698