[Sync] Refactor GetEncryptedTypes usage.

sync/test/fake_sync_encryption_handler.cc

Index: sync/test/fake_sync_encryption_handler.cc
diff --git a/sync/test/fake_sync_encryption_handler.cc b/sync/test/fake_sync_encryption_handler.cc
index b4908622540a7a1b343e7f8ce8086613b226ba43..c589558c4e406eff8f30574dc5bc1bf41dd893db 100644
--- a/sync/test/fake_sync_encryption_handler.cc
+++ b/sync/test/fake_sync_encryption_handler.cc
@@ -6,7 +6,6 @@
 
 #include "sync/protocol/nigori_specifics.pb.h"
 #include "sync/syncable/nigori_util.h"
-#include "sync/util/cryptographer.h"
 
 namespace syncer {
 
@@ -14,7 +13,7 @@ FakeSyncEncryptionHandler::FakeSyncEncryptionHandler()
     : encrypted_types_(SensitiveTypes()),
       encrypt_everything_(false),
       explicit_passphrase_(false),
-      cryptographer_(NULL) {
+      cryptographer_(&encryptor_) {
 }
 FakeSyncEncryptionHandler::~FakeSyncEncryptionHandler() {}
 
@@ -30,21 +29,18 @@ void FakeSyncEncryptionHandler::ApplyNigoriUpdate(
   if (nigori.using_explicit_passphrase())
     explicit_passphrase_ = true;
 
-  if (!cryptographer_)
-    return;
-
-  if (cryptographer_->CanDecrypt(nigori.encrypted()))
-    cryptographer_->InstallKeys(nigori.encrypted());
-  else
-    cryptographer_->SetPendingKeys(nigori.encrypted());
+  if (cryptographer_.CanDecrypt(nigori.encrypted()))
+    cryptographer_.InstallKeys(nigori.encrypted());
+  else if (nigori.has_encrypted())
+    cryptographer_.SetPendingKeys(nigori.encrypted());
 
-  if (cryptographer_->has_pending_keys()) {
+  if (cryptographer_.has_pending_keys()) {
     DVLOG(1) << "OnPassPhraseRequired Sent";
-    sync_pb::EncryptedData pending_keys = cryptographer_->GetPendingKeys();
+    sync_pb::EncryptedData pending_keys = cryptographer_.GetPendingKeys();
     FOR_EACH_OBSERVER(SyncEncryptionHandler::Observer, observers_,
                       OnPassphraseRequired(REASON_DECRYPTION,
                                            pending_keys));
-  } else if (!cryptographer_->is_ready()) {
+  } else if (!cryptographer_.is_ready()) {
     DVLOG(1) << "OnPassphraseRequired sent because cryptographer is not "
              << "ready";
     FOR_EACH_OBSERVER(SyncEncryptionHandler::Observer, observers_,
@@ -61,6 +57,11 @@ void FakeSyncEncryptionHandler::UpdateNigoriFromEncryptedTypes(
                                            nigori);
 }
 
+ModelTypeSet FakeSyncEncryptionHandler::GetEncryptedTypes(
+    syncable::BaseTransaction* const trans) const {
+  return encrypted_types_;
+}
+
 void FakeSyncEncryptionHandler::AddObserver(Observer* observer) {
   observers_.AddObserver(observer);
 }
@@ -95,10 +96,6 @@ bool FakeSyncEncryptionHandler::EncryptEverythingEnabled() const {
   return encrypt_everything_;
 }
 
-ModelTypeSet FakeSyncEncryptionHandler::GetEncryptedTypes() const {
-  return encrypted_types_;
-}
-
 bool FakeSyncEncryptionHandler::IsUsingExplicitPassphrase() const {
   return explicit_passphrase_;
 }