Disable the Developer Tools when Javascript is disabled.

chrome/browser/policy/configuration_policy_handler.cc

 // Copyright (c) 2011 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.h"
 
 #include <string>
 #include <vector>
 
 #include "base/file_path.h"
@@ skipping 1222 matching lines @@
 }
 
 JavascriptPolicyHandler::~JavascriptPolicyHandler() {
 }
 
 bool JavascriptPolicyHandler::CheckPolicySettings(const PolicyMap* policies,
                                                   PolicyErrorMap* errors) {
   const Value* javascript_enabled = policies->Get(kPolicyJavascriptEnabled);
   const Value* default_setting = policies->Get(kPolicyDefaultJavaScriptSetting);
 
-  if (javascript_enabled && !javascript_enabled->IsType(Value::TYPE_BOOLEAN)) {
+  bool enabled = true;
+  if (javascript_enabled && !javascript_enabled->GetAsBoolean(&enabled)) {
     errors->AddError(kPolicyJavascriptEnabled,
                      IDS_POLICY_TYPE_ERROR,
                      ValueTypeToString(Value::TYPE_BOOLEAN));
     javascript_enabled = NULL;
   }
 
-  if (default_setting && !default_setting->IsType(Value::TYPE_INTEGER)) {
+  int setting = CONTENT_SETTING_DEFAULT;
+  if (default_setting && !default_setting->GetAsInteger(&setting)) {
     errors->AddError(kPolicyDefaultJavaScriptSetting,
                      IDS_POLICY_TYPE_ERROR,
                      ValueTypeToString(Value::TYPE_INTEGER));
     default_setting = NULL;
   }
 
-  bool enabled;
-  int setting;
   if (javascript_enabled &&
       default_setting &&
-      javascript_enabled->GetAsBoolean(&enabled) &&
-      default_setting->GetAsInteger(&setting) &&
       !enabled &&
       setting != CONTENT_SETTING_BLOCK) {
     errors->AddError(kPolicyDefaultJavaScriptSetting,
                      IDS_POLICY_OVERRIDDEN,
                      GetPolicyName(kPolicyJavascriptEnabled));
   }
   return true;
 }
 
 void JavascriptPolicyHandler::ApplyPolicySettings(const PolicyMap* policies,
                                                   PrefValueMap* prefs) {
   const Value* javascript_enabled = policies->Get(kPolicyJavascriptEnabled);
   const Value* default_setting = policies->Get(kPolicyDefaultJavaScriptSetting);
 
   int setting = CONTENT_SETTING_DEFAULT;
   if (default_setting)
     default_setting->GetAsInteger(&setting);
 
   bool enabled = true;
-  if (javascript_enabled && javascript_enabled->GetAsBoolean(&enabled)) {
-    prefs->SetValue(prefs::kWebKitJavascriptEnabled,
-                    javascript_enabled->DeepCopy());
-    // Force the javascript content setting to BLOCK when this policy disables
-    // javascript globally.
-    if (!enabled)
-      setting = CONTENT_SETTING_BLOCK;
+  if (javascript_enabled &&
+      javascript_enabled->GetAsBoolean(&enabled) &&
+      !enabled) {
+    // This policy is deprecated, and currently overrides the content setting.

[Mattias Nissler (ping if slow), 2011/11/03 18:42:02]

I would expect this to be the other way round. Is there a good reason to have
something that is deprecated be overriden by the supported way of doing things?

[Joao da Silva, 2011/11/03 20:46:55]

No particular reason for it. The other deprecated policies also override the
deprecated policy in favor of the "new" one, so this has been changed as
suggested.

+    setting = CONTENT_SETTING_BLOCK;
   }
 
-  if (setting != CONTENT_SETTING_DEFAULT)
+  if (setting != CONTENT_SETTING_DEFAULT) {
     prefs->SetValue(prefs::kManagedDefaultJavaScriptSetting,
                     Value::CreateIntegerValue(setting));
+  }
 }
 
 
 }  // namespace
 
 
 // ConfigurationPolicyHandler implementation -----------------------------------
 
 // static
 ConfigurationPolicyHandler::HandlerList*
@@ skipping 17 matching lines @@
   list->push_back(new SyncPolicyHandler());
 
 #if !defined(OS_CHROMEOS)
   list->push_back(new DownloadDirPolicyHandler());
 #endif  // !defined(OS_CHROME0S)
 
   return list;
 }
 
 }  // namespace policy