Adding <keygen> Content Setting (UI)

components/content_settings/core/browser/host_content_settings_map.h

 // 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.
 
 // Maps hostnames to custom content settings.  Written on the UI thread and read
 // on any thread.  One instance per profile.
 
 #ifndef COMPONENTS_CONTENT_SETTINGS_CORE_BROWSER_HOST_CONTENT_SETTINGS_MAP_H_
 #define COMPONENTS_CONTENT_SETTINGS_CORE_BROWSER_HOST_CONTENT_SETTINGS_MAP_H_
 
 #include <map>
+#include <set>
 #include <string>
 #include <vector>
 
 #include "base/basictypes.h"
 #include "base/memory/ref_counted.h"
 #include "base/observer_list.h"
 #include "base/prefs/pref_change_registrar.h"
 #include "base/threading/platform_thread.h"
 #include "base/threading/thread_checker.h"
 #include "components/content_settings/core/browser/content_settings_observer.h"
@@ skipping 214 matching lines @@
                           const GURL& secondary_url,
                           ContentSettingsType content_type);
 
   // Returns the last time the pattern has requested permission for the
   // |content_type| setting.
   base::Time GetLastUsageByPattern(
       const ContentSettingsPattern& primary_pattern,
       const ContentSettingsPattern& secondary_pattern,
       ContentSettingsType content_type);
 
+  void SetShouldDisplay(const GURL& url, ContentSettingsType content_type);
+  bool ShouldDisplay(const GURL& url, ContentSettingsType content_type);
+
   // Adds/removes an observer for content settings changes.
   void AddObserver(content_settings::Observer* observer);
   void RemoveObserver(content_settings::Observer* observer);
 
   // Schedules any pending lossy website settings to be written to disk.
   void FlushLossyWebsiteSettings();
 
   // Passes ownership of |clock|.
   void SetPrefClockForTesting(scoped_ptr<base::Clock> clock);
 
@@ skipping 69 matching lines @@
   // immutable).
   mutable base::PlatformThreadId used_from_thread_id_;
 #endif
 
   // Weak; owned by the Profile.
   PrefService* prefs_;
 
   // Whether this settings map is for an OTR session.
   bool is_off_the_record_;
 
+  std::map<GURL, std::set<ContentSettingsType>> display_map_;

[Bernhard Bauer, 2015/12/17 11:56:03]

This should not live in the HostContentSettingsMap, because it's not a content
setting. Also, content settings are usually persistent, while this will stay
around only until the HCSM is destroyed, right? If what you want is to indicate
usage of a content type (allowed or blocked), TabSpecificContentSettings is the
right place for this.

[svaldez, 2015/12/17 15:25:42]

Done.

[Bernhard Bauer, 2015/12/17 15:51:05]

Sorry, I should have been a bit more detailed. TabSpecificContentSettings
already has code to store and find out whether a given content setting was
blocked, with OnContentBlockedWithDetail() and IsContentBlocked(). This is not
keyed on the URL, but as TabSpecificContentSettings has a 1:1 relation to
WebContents, it is stored per tab. Now, I don't know the exact specification in
this case, but for all other content settings this is the desired behavior, as
all the UI elements that indicate blocking are also per tab.

So, you would get by without modifications to TabSpecificContentSettings, and
just check IsContentBlocked() in WebsiteSettingsUI. (You might want to only do
that for the KEYGEN content setting type, just to make sure we don't run into
any weird edge cases with other blocked content setting types.)

+
   // Content setting providers. This is only modified at construction
   // time and by RegisterExtensionService, both of which should happen
   // before any other uses of it.
   ProviderMap content_settings_providers_;
 
   base::ThreadChecker thread_checker_;
 
   base::ObserverList<content_settings::Observer> observers_;
 
   DISALLOW_COPY_AND_ASSIGN(HostContentSettingsMap);
 };
 
 #endif  // COMPONENTS_CONTENT_SETTINGS_CORE_BROWSER_HOST_CONTENT_SETTINGS_MAP_H_