[Sync] Cache encrypted types info in ProfileSyncService

chrome/browser/sync/profile_sync_service.h

Index: chrome/browser/sync/profile_sync_service.h
diff --git a/chrome/browser/sync/profile_sync_service.h b/chrome/browser/sync/profile_sync_service.h
index 20214e1409e0311f57a9c0ef55b09b1f8092dffe..5d5154f077b4018c1bf494204de69408140f4fdc 100644
--- a/chrome/browser/sync/profile_sync_service.h
+++ b/chrome/browser/sync/profile_sync_service.h
@@ -209,8 +209,10 @@ class ProfileSyncService : public browser_sync::SyncFrontend,
   virtual void OnPassphraseRequired(
       sync_api::PassphraseRequiredReason reason) OVERRIDE;
   virtual void OnPassphraseAccepted() OVERRIDE;
-  virtual void OnEncryptionComplete(
-      const syncable::ModelTypeSet& encrypted_types) OVERRIDE;
+  virtual void OnEncryptedTypesChanged(
+      const syncable::ModelTypeSet& enncrypted_types,
+      bool encrypt_everything) OVERRIDE;
+  virtual void OnEncryptionComplete() OVERRIDE;
   virtual void OnMigrationNeededForTypes(
       const syncable::ModelTypeSet& types) OVERRIDE;
   virtual void OnDataTypesChanged(
@@ -670,6 +672,12 @@ class ProfileSyncService : public browser_sync::SyncFrontend,
   // is reworked to allow one-shot commands like clearing server data.
   base::OneShotTimer<ProfileSyncService> clear_server_data_timer_;
 
+  // The set of encrypted types and whether or not we want to encrypt
+  // everything.  Initialized on OnBackendInitialized and updated by

[Nicolas Zea, 2011/10/21 14:29:07]

Perhaps include a comment that the Cryptographer::SensitiveTypes will always be
part of the set of encrypted types.

[akalin, 2011/10/22 03:28:38]

Done.

+  // OnEncryptionComplete.
+  syncable::ModelTypeSet encrypted_types_;
+  bool encrypt_everything_;
+
   // Whether we're waiting for an attempt to encryption all sync data to
   // complete. We track this at this layer in order to allow the user to cancel
   // if they e.g. don't remember their explicit passphrase.