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

Unified Diff: chrome/browser/sync/test/integration/two_client_passwords_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, 3 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/sync/test/integration/two_client_passwords_sync_test.cc
diff --git a/chrome/browser/sync/test/integration/two_client_passwords_sync_test.cc b/chrome/browser/sync/test/integration/two_client_passwords_sync_test.cc
index aebe553e77e77d422ccb70e1f48e46c2bcdf8b1b..1e8a44752249976cd8c76f4253979de3f49be63c 100644
--- a/chrome/browser/sync/test/integration/two_client_passwords_sync_test.cc
+++ b/chrome/browser/sync/test/integration/two_client_passwords_sync_test.cc
@@ -31,8 +31,6 @@ using passwords_helper::GetVerifierPasswordCount;
using passwords_helper::GetVerifierPasswordStore;
using passwords_helper::RemoveLogin;
using passwords_helper::RemoveLogins;
-using passwords_helper::SetDecryptionPassphrase;
-using passwords_helper::SetEncryptionPassphrase;
using passwords_helper::UpdateLogin;
using sync_integration_test_util::AwaitPassphraseAccepted;
using sync_integration_test_util::AwaitPassphraseRequired;
@@ -83,13 +81,13 @@ IN_PROC_BROWSER_TEST_F(TwoClientPasswordsSyncTest,
E2E_ENABLED(SetPassphraseAndAddPassword)) {
ASSERT_TRUE(SetupSync()) << "SetupSync() failed.";
- SetEncryptionPassphrase(0, kValidPassphrase,
- browser_sync::ProfileSyncService::EXPLICIT);
- ASSERT_TRUE(AwaitPassphraseAccepted(GetSyncService((0))));
+ GetSyncService(0)->SetEncryptionPassphrase(
+ kValidPassphrase, browser_sync::ProfileSyncService::EXPLICIT);
+ ASSERT_TRUE(AwaitPassphraseAccepted(GetSyncService(0)));
- ASSERT_TRUE(AwaitPassphraseRequired(GetSyncService((1))));
- ASSERT_TRUE(SetDecryptionPassphrase(1, kValidPassphrase));
- ASSERT_TRUE(AwaitPassphraseAccepted(GetSyncService((1))));
+ ASSERT_TRUE(AwaitPassphraseRequired(GetSyncService(1)));
+ ASSERT_TRUE(GetSyncService(1)->SetDecryptionPassphrase(kValidPassphrase));
+ ASSERT_TRUE(AwaitPassphraseAccepted(GetSyncService(1)));
PasswordForm form = CreateTestPasswordForm(0);
AddLogin(GetPasswordStore(0), form);
@@ -149,8 +147,8 @@ IN_PROC_BROWSER_TEST_F(TwoClientPasswordsSyncTest,
ASSERT_TRUE(SetupClients());
ASSERT_TRUE(GetClient(0)->SetupSync());
- SetEncryptionPassphrase(0, kValidPassphrase,
- browser_sync::ProfileSyncService::EXPLICIT);
+ GetSyncService(0)->SetEncryptionPassphrase(
+ kValidPassphrase, browser_sync::ProfileSyncService::EXPLICIT);
ASSERT_TRUE(AwaitPassphraseAccepted(GetSyncService(0)));
// When client 1 hits a passphrase required state, we can infer that
@@ -159,7 +157,7 @@ IN_PROC_BROWSER_TEST_F(TwoClientPasswordsSyncTest,
ASSERT_TRUE(AwaitPassphraseRequired(GetSyncService(1)));
// Get client 1 out of the passphrase required state.
- ASSERT_TRUE(SetDecryptionPassphrase(1, kValidPassphrase));
+ ASSERT_TRUE(GetSyncService(1)->SetDecryptionPassphrase(kValidPassphrase));
ASSERT_TRUE(AwaitPassphraseAccepted(GetSyncService(1)));
// We must mark the setup complete now, since we just entered the passphrase

Powered by Google App Engine
This is Rietveld 408576698