Adding <keygen> Content Setting (Blink)

chrome/renderer/content_settings_observer.h

Index: chrome/renderer/content_settings_observer.h
diff --git a/chrome/renderer/content_settings_observer.h b/chrome/renderer/content_settings_observer.h
index 37a70438c0c6657565877e219ac6202eb6b463ff..f3c8b5ce85e49e3ba384de8bbbb96368a22791a6 100644
--- a/chrome/renderer/content_settings_observer.h
+++ b/chrome/renderer/content_settings_observer.h
@@ -42,8 +42,8 @@ class ContentSettingsObserver
   ~ContentSettingsObserver() override;
 
   // Sets the content setting rules which back |AllowImage()|, |AllowScript()|,
-  // and |AllowScriptFromSource()|. |content_setting_rules| must outlive this
-  // |ContentSettingsObserver|.
+  // |AllowScriptFromSource()|, and |AllowKeygen()|. |content_setting_rules|
+  // must outlive this |ContentSettingsObserver|.
   void SetContentSettingRules(
       const RendererContentSettingRules* content_setting_rules);
 
@@ -67,6 +67,7 @@ class ContentSettingsObserver
                   const blink::WebURL& image_url) override;
   bool allowIndexedDB(const blink::WebString& name,
                       const blink::WebSecurityOrigin& origin) override;
+  bool allowKeygen(bool enabled_per_settings) override;
   bool allowPlugins(bool enabled_per_settings) override;
   bool allowScript(bool enabled_per_settings) override;
   bool allowScriptFromSource(bool enabled_per_settings,
@@ -75,6 +76,7 @@ class ContentSettingsObserver
   bool allowReadFromClipboard(bool default_value) override;
   bool allowWriteToClipboard(bool default_value) override;
   bool allowMutationEvents(bool default_value) override;
+  void didNotAllowKeygen() override;
   void didNotAllowPlugins() override;
   void didNotAllowScript() override;
   bool allowDisplayingInsecureContent(bool allowed_per_settings,
@@ -143,7 +145,7 @@ class ContentSettingsObserver
   // |SetContentSettingRules|.
   const RendererContentSettingRules* content_setting_rules_;
 
-  // Stores if images, scripts, and plugins have actually been blocked.
+  // Stores if images, scripts, keygen, and plugins have actually been blocked.
   std::map<ContentSettingsType, bool> content_blocked_;
 
   // Caches the result of AllowStorage.
@@ -153,6 +155,9 @@ class ContentSettingsObserver
   // Caches the result of |AllowScript|.
   std::map<blink::WebFrame*, bool> cached_script_permissions_;
 
+  // Caches the result of |AllowKeygen|.

[Bernhard Bauer, 2015/11/05 11:12:13]

Nit: method names don't go into pipe symbols.

[svaldez, 2015/11/05 16:35:19]

Done.

+  std::map<blink::WebFrame*, bool> cached_keygen_permissions_;
+
   std::set<std::string> temporarily_allowed_plugins_;
   bool is_interstitial_page_;
   bool npapi_plugins_blocked_;