Plumbing for login apps device policy to extensions.

chrome/browser/extensions/policy_handlers.h

Index: chrome/browser/extensions/policy_handlers.h
diff --git a/chrome/browser/extensions/policy_handlers.h b/chrome/browser/extensions/policy_handlers.h
index cd7e5d99cc722a8c678b4ea9be0ab9ea08e2de6e..207f20a914b4cb627741a32b11964d33ddd016f5 100644
--- a/chrome/browser/extensions/policy_handlers.h
+++ b/chrome/browser/extensions/policy_handlers.h
@@ -49,27 +49,54 @@ class ExtensionListPolicyHandler
   DISALLOW_COPY_AND_ASSIGN(ExtensionListPolicyHandler);
 };
 
-class ExtensionInstallForcelistPolicyHandler
+class ExtensionInstallListPolicyHandler
     : public policy::TypeCheckingPolicyHandler {
  public:
-  ExtensionInstallForcelistPolicyHandler();
-  ~ExtensionInstallForcelistPolicyHandler() override;
-
   // ConfigurationPolicyHandler methods:
   bool CheckPolicySettings(const policy::PolicyMap& policies,
                            policy::PolicyErrorMap* errors) override;
   void ApplyPolicySettings(const policy::PolicyMap& policies,
                            PrefValueMap* prefs) override;
 
+ protected:
+  ExtensionInstallListPolicyHandler(const char* policy_name,
+                                    const char* pref_name);
+
+  ~ExtensionInstallListPolicyHandler() override;
+
+  const char* pref_name() const { return pref_name_; }
+
  private:
   // Parses the data in |policy_value| and writes them to |extension_dict|.
   bool ParseList(const base::Value* policy_value,
                  base::DictionaryValue* extension_dict,
                  policy::PolicyErrorMap* errors);
 
+  const char* const pref_name_;
+
+  DISALLOW_COPY_AND_ASSIGN(ExtensionInstallListPolicyHandler);
+};
+
+class ExtensionInstallForcelistPolicyHandler
+    : public ExtensionInstallListPolicyHandler {
+ public:
+  ExtensionInstallForcelistPolicyHandler();
+  ~ExtensionInstallForcelistPolicyHandler() override;

[emaxx, 2016/07/15 13:22:31]

I guess, defining the destructor explicitly isn't necessary in these cases
(which just result in calling parent's destructor).

[achuithb, 2016/07/21 00:16:05]

I'd like to keep this for consistency with the rest of the classes in this file.

+
+ private:
   DISALLOW_COPY_AND_ASSIGN(ExtensionInstallForcelistPolicyHandler);
 };
 
+class ExtensionInstallLoginlistPolicyHandler
+    : public ExtensionInstallListPolicyHandler {
+ public:
+  ExtensionInstallLoginlistPolicyHandler();
+  ~ExtensionInstallLoginlistPolicyHandler() override;
+
+ private:
+  DISALLOW_COPY_AND_ASSIGN(ExtensionInstallLoginlistPolicyHandler);
+};
+
 // Implements additional checks for policies that are lists of extension
 // URLPatterns.
 class ExtensionURLPatternListPolicyHandler