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..63668c4efe23f9f15de938bff5c8929080923c81 100644
--- a/chrome/browser/supervised_user/supervised_user_service.h
+++ b/chrome/browser/supervised_user/supervised_user_service.h
@@ -187,7 +187,10 @@ class SupervisedUserService : public KeyedService,
   friend class SupervisedUserServiceExtensionTestBase;
   friend class SupervisedUserServiceFactory;
   FRIEND_TEST_ALL_PREFIXES(SupervisedUserServiceTest, ClearOmitOnRegistration);
-
+  FRIEND_TEST_ALL_PREFIXES(SupervisedUserServiceTest,
+                           ChangesIncludedSessionOnChangedSettings);
+  FRIEND_TEST_ALL_PREFIXES(SupervisedUserServiceTest,
+                           ChangesSyncSessionStateOnChangedSettings);
   // 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 +275,23 @@ class SupervisedUserService : public KeyedService,
   // Returns the human readable name of the supervised user.
   std::string GetSupervisedUserName() const;
 
+  // Subscribes to the SupervisedUserPrefStore, refreshes
+  // |includes_sync_sessions_| and triggers reconfiguring the
+  // ProfileSyncService.
+  void OnHistoryRecordingStateChanged();
+
+  // Reads the state of |includes_sync_sessions_| from prefrences. The

[Marc Treib, 2014/08/20 14:14:26]

Typo: preferences

[fhorschig, 2014/08/20 14:48:04]

Done.

+  // returned value indicates if |includes_sync_sessions_| has changed.
+  bool FetchNewSessionSyncState();
+
+  // Returns true if the syncer::SESSIONS should be included on Sync.
+  bool IncludesSyncSessions() const;

[Marc Treib, 2014/08/20 14:14:26]

add "type" after SESSIONS
included IN sync

[fhorschig, 2014/08/20 14:48:03]

Done.

+
+  // The option a custodian sets to either record or prevent recording the
+  // supervised user's history. Set by |FetchNewSessionSyncState()| and
+  // defaults to true.
+  bool includes_sync_sessions_;
+
   // Owns us via the KeyedService mechanism.
   Profile* profile_;