Stopping the history recording for a supervised user

chrome/browser/supervised_user/supervised_user_service_unittest.cc

 // 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.
 
 #include "base/path_service.h"
 #include "base/prefs/pref_service.h"
 #include "base/prefs/scoped_user_pref_update.h"
 #include "base/strings/utf_string_conversions.h"
 #include "chrome/browser/chrome_notification_types.h"
 #include "chrome/browser/extensions/extension_service.h"
 #include "chrome/browser/extensions/extension_service_test_base.h"
 #include "chrome/browser/extensions/unpacked_installer.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/signin/fake_profile_oauth2_token_service.h"
 #include "chrome/browser/signin/fake_profile_oauth2_token_service_builder.h"
 #include "chrome/browser/signin/profile_oauth2_token_service_factory.h"
 #include "chrome/browser/supervised_user/custodian_profile_downloader_service.h"
 #include "chrome/browser/supervised_user/custodian_profile_downloader_service_factory.h"
+#include "chrome/browser/supervised_user/supervised_user_constants.h"

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

I don't think this is necessary anymore.

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

Done.

 #include "chrome/browser/supervised_user/supervised_user_service.h"
 #include "chrome/browser/supervised_user/supervised_user_service_factory.h"
 #include "chrome/browser/ui/browser_list.h"
 #include "chrome/common/chrome_paths.h"
 #include "chrome/common/extensions/features/feature_channel.h"
 #include "chrome/common/pref_names.h"
 #include "chrome/test/base/testing_profile.h"
 #include "content/public/test/test_browser_thread_bundle.h"
 #include "content/public/test/test_utils.h"
 #include "extensions/common/extension.h"
@@ skipping 118 matching lines @@
             supervised_user_service_->GetManualBehaviorForURL(kExampleBarURL));
   EXPECT_EQ(SupervisedUserService::MANUAL_ALLOW,
             supervised_user_service_->GetManualBehaviorForURL(
                 kExampleFooNoWWWURL));
   EXPECT_EQ(SupervisedUserService::MANUAL_ALLOW,
             supervised_user_service_->GetManualBehaviorForURL(kBlurpURL));
   EXPECT_EQ(SupervisedUserService::MANUAL_NONE,
             supervised_user_service_->GetManualBehaviorForURL(kMooseURL));
 }
 
+TEST_F(SupervisedUserServiceTest, ChangesIncludedSessionOnChangedSettings) {
+  EXPECT_TRUE(supervised_user_service_->IncludesSyncSessions());
+  profile_->GetPrefs()->SetBoolean(prefs::kRecordHistory, false);
+
+  supervised_user_service_->OnHistoryRecordingStateChanged();

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

You shouldn't need to call this explicitly, since the SUService is listening for
changes in the pref. (Or maybe that's not set up correctly in a unit test? Then
please fix that, or at least add a comment why you need to call this explicitly
here.)

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

Done.

+
+  EXPECT_FALSE(supervised_user_service_->IncludesSyncSessions());
+}
+
+TEST_F(SupervisedUserServiceTest, ChangesSyncSessionStateOnChangedSettings) {
+  profile_->GetPrefs()->SetBoolean(prefs::kRecordHistory, false);
+  supervised_user_service_->OnHistoryRecordingStateChanged();

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

^^

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

Done.

+  profile_->GetPrefs()->SetBoolean(prefs::kRecordHistory, true);
+
+  bool changed_state = supervised_user_service_->FetchNewSessionSyncState();
+
+  EXPECT_TRUE(changed_state);
+}
+
 // Ensure that the CustodianProfileDownloaderService shuts down cleanly. If no
 // DCHECK is hit when the service is destroyed, this test passed.
 TEST_F(SupervisedUserServiceTest, ShutDownCustodianProfileDownloader) {
   CustodianProfileDownloaderService* downloader_service =
       CustodianProfileDownloaderServiceFactory::GetForProfile(profile_.get());
 
   // Emulate being logged in, then start to download a profile so a
   // ProfileDownloader gets created.
   profile_->GetPrefs()->SetString(prefs::kGoogleServicesUsername, "Logged In");
   downloader_service->DownloadProfile(base::Bind(&OnProfileDownloadedFail));
@@ skipping 247 matching lines @@
   site_lists[0]->GetSites(&sites);
   ASSERT_EQ(1u, sites.size());
   EXPECT_EQ(base::ASCIIToUTF16("Moose"), sites[0].name);
 
   EXPECT_EQ(SupervisedUserURLFilter::WARN,
             url_filter->GetFilteringBehaviorForURL(example_url));
   EXPECT_EQ(SupervisedUserURLFilter::ALLOW,
             url_filter->GetFilteringBehaviorForURL(moose_url));
 }
 #endif  // !defined(OS_ANDROID)