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 1226 matching lines @@
 
 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)) {
     errors->AddError(kPolicyJavascriptEnabled,
                      IDS_POLICY_TYPE_ERROR,
                      ValueTypeToString(Value::TYPE_BOOLEAN));
-    javascript_enabled = NULL;
   }
 
   if (default_setting && !default_setting->IsType(Value::TYPE_INTEGER)) {
     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,
+  if (javascript_enabled && default_setting) {
+    errors->AddError(kPolicyJavascriptEnabled,
                      IDS_POLICY_OVERRIDDEN,
-                     GetPolicyName(kPolicyJavascriptEnabled));
+                     GetPolicyName(kPolicyDefaultJavaScriptSetting));
   }
+
   return true;
 }
 
 void JavascriptPolicyHandler::ApplyPolicySettings(const PolicyMap& policies,
                                                   PrefValueMap* prefs) {
-  const Value* javascript_enabled = policies.Get(kPolicyJavascriptEnabled);
+  int setting = CONTENT_SETTING_DEFAULT;
   const Value* default_setting = policies.Get(kPolicyDefaultJavaScriptSetting);
 
-  int setting = CONTENT_SETTING_DEFAULT;
-  if (default_setting)
+  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)
+  } else {
+    const Value* javascript_enabled = policies.Get(kPolicyJavascriptEnabled);
+    bool enabled = true;
+    if (javascript_enabled &&
+        javascript_enabled->GetAsBoolean(&enabled) &&
+        !enabled) {
       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