Stopping the history recording for a supervised user

chrome/browser/supervised_user/supervised_user_service.h

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROME_BROWSER_SUPERVISED_USER_SUPERVISED_USER_SERVICE_H_
 #define CHROME_BROWSER_SUPERVISED_USER_SUPERVISED_USER_SERVICE_H_
 
 #include <set>
 #include <vector>
 
@@ skipping 169 matching lines @@
   // ProfileSyncServiceObserver implementation:
   virtual void OnStateChanged() OVERRIDE;
 
   // chrome::BrowserListObserver implementation:
   virtual void OnBrowserSetLastActive(Browser* browser) OVERRIDE;
 
  private:
   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.
   class URLFilterContext {
    public:
     URLFilterContext();
     ~URLFilterContext();
 
     SupervisedUserURLFilter* ui_url_filter() const;
     SupervisedUserURLFilter* io_url_filter() const;
@@ skipping 64 matching lines @@
   // corresponding preference is changed.
   void UpdateManualHosts();
 
   // Updates the manual overrides for URLs in the URL filters when the
   // corresponding preference is changed.
   void UpdateManualURLs();
 
   // 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_;
 
   bool active_;
 
   Delegate* delegate_;
 
 #if defined(ENABLE_EXTENSIONS)
   ScopedObserver<extensions::ExtensionRegistry,
                  extensions::ExtensionRegistryObserver>
@@ skipping 22 matching lines @@
   // Used to create permission requests.
   scoped_ptr<PermissionRequestCreator> permissions_creator_;
 
   // True iff we are waiting for a permission request to be issued.
   bool waiting_for_permissions_;
 
   base::WeakPtrFactory<SupervisedUserService> weak_ptr_factory_;
 };
 
 #endif  // CHROME_BROWSER_SUPERVISED_USER_SUPERVISED_USER_SERVICE_H_