Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(4)

Side by Side Diff: chrome/browser/plugins/plugin_policy_handler.cc

Issue 2474713003: Migrate enabled state of plugins into specialized Flash and PDF prefs. (Closed)
Patch Set: Address comments. Created 4 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/plugins/plugin_policy_handler.h" 5 #include "chrome/browser/plugins/plugin_policy_handler.h"
6 6
7 #include <string> 7 #include <string>
8 8
9 #include "base/memory/ptr_util.h" 9 #include "base/memory/ptr_util.h"
10 #include "base/strings/pattern.h" 10 #include "base/strings/pattern.h"
11 #include "base/strings/string_util.h" 11 #include "base/strings/string_util.h"
12 #include "base/values.h" 12 #include "base/values.h"
13 #include "chrome/browser/plugins/plugin_prefs.h" 13 #include "chrome/browser/plugins/plugin_prefs.h"
14 #include "chrome/common/chrome_content_client.h" 14 #include "chrome/common/chrome_content_client.h"
15 #include "chrome/common/pref_names.h" 15 #include "chrome/common/pref_names.h"
16 #include "components/content_settings/core/common/pref_names.h" 16 #include "components/content_settings/core/common/pref_names.h"
17 #include "components/policy/core/browser/policy_error_map.h" 17 #include "components/policy/core/browser/policy_error_map.h"
18 #include "components/strings/grit/components_strings.h" 18 #include "components/strings/grit/components_strings.h"
19 #include "content/public/common/content_constants.h" 19 #include "content/public/common/content_constants.h"
20 20
21 namespace { 21 namespace {
22 22
23 const char kAdobeFlashPlayerName[] = "Adobe Flash Player";
24
25 // Retrieves a list typed policy or nullptr if not present or not a list. 23 // Retrieves a list typed policy or nullptr if not present or not a list.
26 const base::ListValue* GetListPolicy(const policy::PolicyMap& policies, 24 const base::ListValue* GetListPolicy(const policy::PolicyMap& policies,
27 const std::string& policy) { 25 const std::string& policy) {
28 const base::Value* value = policies.GetValue(policy); 26 const base::Value* value = policies.GetValue(policy);
29 if (!value) 27 if (!value)
30 return nullptr; 28 return nullptr;
31 29
32 const base::ListValue* policy_value = nullptr; 30 const base::ListValue* policy_value = nullptr;
33 value->GetAsList(&policy_value); 31 value->GetAsList(&policy_value);
34 return policy_value; 32 return policy_value;
(...skipping 18 matching lines...) Expand all
53 for (int i = 0; i < size; ++i) { 51 for (int i = 0; i < size; ++i) {
54 std::string plugin; 52 std::string plugin;
55 if (!plugins->GetString(i, &plugin)) 53 if (!plugins->GetString(i, &plugin))
56 continue; 54 continue;
57 if (base::MatchPattern(ChromeContentClient::kPDFPluginName, plugin) && 55 if (base::MatchPattern(ChromeContentClient::kPDFPluginName, plugin) &&
58 !policies.GetValue(policy::key::kAlwaysOpenPdfExternally)) { 56 !policies.GetValue(policy::key::kAlwaysOpenPdfExternally)) {
59 prefs->SetValue( 57 prefs->SetValue(
60 prefs::kPluginsAlwaysOpenPdfExternally, 58 prefs::kPluginsAlwaysOpenPdfExternally,
61 base::MakeUnique<base::FundamentalValue>(disable_pdf_plugin)); 59 base::MakeUnique<base::FundamentalValue>(disable_pdf_plugin));
62 } 60 }
63 if ((base::MatchPattern(kAdobeFlashPlayerName, plugin) || 61 if ((base::MatchPattern(content::kAdobeFlashPlayerPluginName, plugin) ||
64 base::MatchPattern(content::kFlashPluginName, plugin)) && 62 base::MatchPattern(content::kFlashPluginName, plugin)) &&
65 !policies.GetValue(policy::key::kDefaultPluginsSetting)) { 63 !policies.GetValue(policy::key::kDefaultPluginsSetting)) {
66 prefs->SetValue( 64 prefs->SetValue(
67 prefs::kManagedDefaultPluginsSetting, 65 prefs::kManagedDefaultPluginsSetting,
68 base::MakeUnique<base::FundamentalValue>(flash_content_setting)); 66 base::MakeUnique<base::FundamentalValue>(flash_content_setting));
69 } 67 }
70 } 68 }
71 } 69 }
72 70
73 bool PluginPolicyHandler::CheckPolicySettings(const policy::PolicyMap& policies, 71 bool PluginPolicyHandler::CheckPolicySettings(const policy::PolicyMap& policies,
(...skipping 29 matching lines...) Expand all
103 return; 101 return;
104 const int size = plugins->GetSize(); 102 const int size = plugins->GetSize();
105 for (int i = 0; i < size; ++i) { 103 for (int i = 0; i < size; ++i) {
106 std::string plugin; 104 std::string plugin;
107 if (!plugins->GetString(i, &plugin)) 105 if (!plugins->GetString(i, &plugin))
108 continue; 106 continue;
109 if (base::MatchPattern(ChromeContentClient::kPDFPluginName, plugin) && 107 if (base::MatchPattern(ChromeContentClient::kPDFPluginName, plugin) &&
110 !policies.GetValue(policy::key::kAlwaysOpenPdfExternally)) { 108 !policies.GetValue(policy::key::kAlwaysOpenPdfExternally)) {
111 prefs->RemoveValue(prefs::kPluginsAlwaysOpenPdfExternally); 109 prefs->RemoveValue(prefs::kPluginsAlwaysOpenPdfExternally);
112 } 110 }
113 if ((base::MatchPattern(kAdobeFlashPlayerName, plugin) || 111 if ((base::MatchPattern(content::kAdobeFlashPlayerPluginName, plugin) ||
114 base::MatchPattern(content::kFlashPluginName, plugin)) && 112 base::MatchPattern(content::kFlashPluginName, plugin)) &&
115 !policies.GetValue(policy::key::kDefaultPluginsSetting)) { 113 !policies.GetValue(policy::key::kDefaultPluginsSetting)) {
116 prefs->RemoveValue(prefs::kManagedDefaultPluginsSetting); 114 prefs->RemoveValue(prefs::kManagedDefaultPluginsSetting);
117 } 115 }
118 } 116 }
119 } 117 }
OLDNEW
« no previous file with comments | « no previous file | chrome/browser/plugins/plugin_prefs.cc » ('j') | chrome/browser/plugins/plugin_prefs.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698