Adding <keygen> Content Setting (Blink)

chrome/renderer/content_settings_observer.cc

 // Copyright (c) 2012 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/renderer/content_settings_observer.h"
 
 #include "base/command_line.h"
 #include "base/metrics/histogram.h"
 #include "components/content_settings/content/common/content_settings_messages.h"
 #include "content/public/common/url_constants.h"
@@ skipping 575 matching lines @@
     SendInsecureContentSignal(INSECURE_CONTENT_RUN_SWF);
 
   if (!allow_running_insecure_content_ && !allowed_per_settings) {
     DidBlockContentType(CONTENT_SETTINGS_TYPE_MIXEDSCRIPT, origin.host());
     return false;
   }
 
   return true;
 }
 
+void ContentSettingsObserver::didUseKeygen() {
+  Send(new ChromeViewHostMsg_DidUseKeygen(routing_id()));

[jochen (gone - plz use gerrit), 2015/11/25 08:54:34]

i'd include the frame url here, so you can show what sites want to use keygen in
the UI. I'd also store it somewhere in tab specific content settings

[svaldez, 2015/11/25 14:56:55]

We're currently using the top-level origin for the content-setting, and we
already display that in the UI through ContentSettingImage/Bubble. 

+}
+
 void ContentSettingsObserver::didNotAllowPlugins() {
   DidBlockContentType(CONTENT_SETTINGS_TYPE_PLUGINS);
 }
 
 void ContentSettingsObserver::didNotAllowScript() {
   DidBlockContentType(CONTENT_SETTINGS_TYPE_JAVASCRIPT);
 }
 
 bool ContentSettingsObserver::AreNPAPIPluginsBlocked() const {
   return npapi_plugins_blocked_;
@@ skipping 118 matching lines @@
 
   // If the scheme is file:, an empty file name indicates a directory listing,
   // which requires JavaScript to function properly.
   if (base::EqualsASCII(protocol, url::kFileScheme)) {
     return document_url.SchemeIs(url::kFileScheme) &&
            document_url.ExtractFileName().empty();
   }
 
   return false;
 }