Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(8234)

Unified Diff: chrome/browser/supervised_user/supervised_user_service.h

Issue 480513004: Stopping the history recording for a supervised user (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Made documentation clearer. Created 6 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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 55e3be269487cd242b5a759ecdee46b5c354e687..e4e282931ae4db9feb4758e6ac6dc82cbc8f8afe 100644
--- a/chrome/browser/supervised_user/supervised_user_service.h
+++ b/chrome/browser/supervised_user/supervised_user_service.h
@@ -206,7 +206,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.
@@ -309,6 +312,19 @@ 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_type_| and triggers reconfiguring the
+ // ProfileSyncService.
+ void OnHistoryRecordingStateChanged();
+
+ // Returns true if the syncer::SESSIONS type should be included in Sync.
+ bool IncludesSyncSessionsType() const;
+
+ // 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_type_;
+
// Owns us via the KeyedService mechanism.
Profile* profile_;

Powered by Google App Engine
This is Rietveld 408576698