Separate state of basic and advanced tab in CBD dialog

chrome/browser/policy/configuration_policy_handler_list_factory.cc

 // Copyright 2013 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 "chrome/browser/policy/configuration_policy_handler_list_factory.h"
 
 #include <limits.h>
 #include <stddef.h>
 
 #include <utility>
@@ skipping 715 matching lines @@
 
   void ApplyPolicySettings(const PolicyMap& policies,
                            PrefValueMap* prefs) override {
     const base::Value* value = policies.GetValue(policy_name());
     bool deleting_history_allowed;
     if (value && value->GetAsBoolean(&deleting_history_allowed) &&
         !deleting_history_allowed) {
       prefs->SetBoolean(
           browsing_data::prefs::kDeleteBrowsingHistory, false);
       prefs->SetBoolean(
-          browsing_data::prefs::kDeleteDownloadHistory, false);
+          browsing_data::prefs::kDeleteBrowsingHistoryBasic, false);

[msramek, 2017/02/08 10:55:56]

Good catch!

[dullweber, 2017/02/08 23:03:18]

thanks, I tried to find all usages of kDeleteBrowsingHistory and the other
preferences. I hope I have all of them :D

+      prefs->SetBoolean(browsing_data::prefs::kDeleteDownloadHistory, false);
     }
   }
 };
 
 #if BUILDFLAG(ENABLE_EXTENSIONS)
 void GetExtensionAllowedTypesMap(
     std::vector<std::unique_ptr<StringMappingListPolicyHandler::MappingEntry>>*
         result) {
   // Mapping from extension type names to Manifest::Type.
   for (size_t index = 0;
@@ skipping 259 matching lines @@
 #endif  // defined(OS_CHROMEOS)
 
 #if BUILDFLAG(ENABLE_PLUGINS)
   handlers->AddHandler(base::MakeUnique<PluginPolicyHandler>());
 #endif  // BUILDFLAG(ENABLE_PLUGINS)
 
   return handlers;
 }
 
 }  // namespace policy