Stopping the history recording for a supervised user

chrome/browser/supervised_user/supervised_user_service.h

Index: chrome/browser/supervised_user/supervised_user_service.h
diff --git a/chrome/browser/supervised_user/supervised_user_service.h b/chrome/browser/supervised_user/supervised_user_service.h
index 7bbdfedefab7adc3b972c779ed10fd6ed1ed7fbe..359373c9ff05259378eea42bfd173736181fe2f1 100644
--- a/chrome/browser/supervised_user/supervised_user_service.h
+++ b/chrome/browser/supervised_user/supervised_user_service.h
@@ -187,7 +187,8 @@ class SupervisedUserService : public KeyedService,
   friend class SupervisedUserServiceExtensionTestBase;
   friend class SupervisedUserServiceFactory;
   FRIEND_TEST_ALL_PREFIXES(SupervisedUserServiceTest, ClearOmitOnRegistration);
-
+  FRIEND_TEST_ALL_PREFIXES(SupervisedUserServiceTest,
+                           ChangesSessionSyncOnChangedSettings);
   // A bridge from the UI thread to the SupervisedUserURLFilters, one of which
   // lives on the IO thread. This class mediates access to them and makes sure
   // they are kept in sync.
@@ -272,6 +273,27 @@ class SupervisedUserService : public KeyedService,
   // Returns the human readable name of the supervised user.
   std::string GetSupervisedUserName() const;
 
+  // Gets attached to the SupervisedUserSettingsService and writes into
+  // may_session_sync_ if the history may be recorded.
+  void OnNewSettingsAvailable(const base::DictionaryValue* settings);
+
+  // Reads the state of |may_session_sync_| from given SU settings. The returned
+  // value indicates if |may_session_sync_| has changed.
+  bool FetchNewSessionSyncState(const base::DictionaryValue* settings);
+
+  // Reinitializing this Service means reloading the Sync settings. This method
+  // thereby ensures that changes in Sync settings take effect immediately.
+  void ReinitializeProfileSyncService();

[Marc Treib, 2014/08/19 14:00:27]

Reconfigure?

+
+  // Reinitializing this Service means reloading the Sync settings. This method
+  // thereby ensures that changes in Sync settings take effect immediately.
+  bool MaySyncSessions() const;

[Marc Treib, 2014/08/19 14:00:28]

"May" isn't the right word here, since the SU doesn't have a choice in the
matter. Either "Must", or just "SyncSessions"?

+
+  // The option a custodian sets to either recored or prevent recording the
+  // supervised user's history. Set by |FetchNewSessionSyncState()| and
+  // Defaults to true;

[Marc Treib, 2014/08/19 14:00:28]

nit: don't capitalize "Defaults"; end with "."

+  bool may_session_sync_;
+
   // Owns us via the KeyedService mechanism.
   Profile* profile_;