Add migration code for the EnabledPlugins and DisabledPlugins policies.

chrome/browser/plugins/plugin_policy_handler.h

Index: chrome/browser/plugins/plugin_policy_handler.h
diff --git a/chrome/browser/plugins/plugin_policy_handler.h b/chrome/browser/plugins/plugin_policy_handler.h
new file mode 100644
index 0000000000000000000000000000000000000000..4d6fcc0b89556987fee384e88f8e2a985bbc8dae
--- /dev/null
+++ b/chrome/browser/plugins/plugin_policy_handler.h
@@ -0,0 +1,47 @@
+// Copyright (c) 2016 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.
+
+#ifndef CHROME_BROWSER_PLUGINS_PLUGIN_POLICY_HANDLER_H_
+#define CHROME_BROWSER_PLUGINS_PLUGIN_POLICY_HANDLER_H_
+
+#include <map>
+#include <set>
+#include <vector>
+
+#include "base/macros.h"
+#include "chrome/browser/plugins/plugin_prefs.h"
+#include "components/content_settings/core/common/content_settings.h"
+#include "components/policy/core/browser/configuration_policy_handler.h"
+#include "components/policy/policy_constants.h"
+#include "components/prefs/pref_value_map.h"
+
+// Migrates values of deprecated plugin policies into their new counterpars.
+// It reads the set of EnabledPlugins and DisabledPlugins and updates the Flash
+// default behavior and PDF plugin behavior. All other values in the list are
+// ignored. The string matching rules are not trying to catch edge cases like

[Bernhard Bauer, 2016/10/14 11:49:46]

This comments needs updating.

[pastarmovj, 2016/10/14 12:38:59]

Yes. No need for the explanation anymore.

+// matching "Shockwave Flash Player" by "????????*Player". Only the direct
+// mentioning of "PDF" and "FLASH" or the wildcard "*" element are migrated to
+// the new policies.
+class PluginPolicyHandler : public policy::ConfigurationPolicyHandler {
+ public:
+  PluginPolicyHandler();
+  ~PluginPolicyHandler() override;

[Bernhard Bauer, 2016/10/14 11:49:46]

If you have a protected section anyway, move the destructor to it? This handler
will be owned by ConfigurationPolicyHandlerList, which will own (and therefore
destroy it) as the base class.

[pastarmovj, 2016/10/14 12:38:59]

Nope. VC doesn't like it 

ninja -t msvc -e environment.x64 --
"C:\src\depot_tools\win_toolchain\vs_files\d5dc33b15d1b2c086f2f6632e2fd15882f80dbd3\VC\bin\amd64/cl.exe"
/nologo /showIncludes /FC
@obj/chrome/browser/browser/configuration_policy_handler_list_factory.obj.rsp /c
../../chrome/browser/policy/configuration_policy_handler_list_factory.cc
/Foobj/chrome/browser/browser/configuration_policy_handler_list_factory.obj
/Fd"obj/chrome/browser/browser_cc.pdb"
c:\src\depot_tools\win_toolchain\vs_files\d5dc33b15d1b2c086f2f6632e2fd15882f80dbd3\vc\include\memory(1195):
error C2248: 'PluginPolicyHandler::~PluginPolicyHandler': cannot access
protected member declared in class 'PluginPolicyHandler'

[Bernhard Bauer, 2016/10/14 15:27:05]

Oh right. For posteriority, I think this is because the compiler will generate
code that would destroy the unique_ptr, even though that code will never run
because we will have passed the unique_ptr to somewhere else.

+
+ protected:
+  bool CheckPolicySettings(const policy::PolicyMap& policies,
+                           policy::PolicyErrorMap* errors) override;
+  void ApplyPolicySettings(const policy::PolicyMap& policies,
+                           PrefValueMap* prefs) override;
+
+ private:
+  void ProcessPolicy(const policy::PolicyMap& policies,
+                     PrefValueMap* prefs,
+                     const std::string& policy,
+                     bool disable_pdf_plugin,
+                     ContentSetting flash_content_setting);
+
+  DISALLOW_COPY_AND_ASSIGN(PluginPolicyHandler);
+};
+
+#endif  // CHROME_BROWSER_PLUGINS_PLUGIN_POLICY_HANDLER_H_