[Sync] Removed passphrase helper methods, removed ((n)) pattern, and fixed lint violations.

chrome/browser/sync/test/integration/sync_test.cc

Index: chrome/browser/sync/test/integration/sync_test.cc
diff --git a/chrome/browser/sync/test/integration/sync_test.cc b/chrome/browser/sync/test/integration/sync_test.cc
index 82cde03c4e42e03e68017b47e931c26da79b249d..d429a9ee10071259543feebed7ed480047c507fe 100644
--- a/chrome/browser/sync/test/integration/sync_test.cc
+++ b/chrome/browser/sync/test/integration/sync_test.cc
@@ -1021,6 +1021,18 @@ bool SyncTest::AwaitEncryptionComplete(int index) {
   return !checker.TimedOut();
 }
 
+void SyncTest::SetEncryptionPassphrase(

[maxbogue, 2016/09/23 17:39:37]

Honestly I'm not even sure these methods are necessary; you can just do
GetSyncService(i)->SetEncryptionPassphrase(passphrase, type)...

[skym, 2016/09/23 17:55:20]

Done.

I was thinking it'd make more sense once I combined these accessors with the
checkers... But I still haven't decided where I think the checkers should live.

+    int index,
+    const std::string& passphrase,
+    ProfileSyncService::PassphraseType type) {
+  GetClient(index)->service()->SetEncryptionPassphrase(passphrase, type);
+}
+
+bool SyncTest::SetDecryptionPassphrase(int index,
+                                       const std::string& passphrase) {
+  return GetClient(index)->service()->SetDecryptionPassphrase(passphrase);
+}
+
 bool SyncTest::AwaitQuiescence() {
   return ProfileSyncServiceHarness::AwaitQuiescence(clients());
 }