Remove CONTENT_SETTINGS_NUM_TYPES (part 2)

chrome/browser/ui/content_settings/content_setting_bubble_model.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/browser/ui/content_settings/content_setting_bubble_model.h"
 
 #include "base/command_line.h"
+#include "base/lazy_instance.h"
 #include "base/prefs/pref_service.h"
+#include "base/stl_util.h"
 #include "base/strings/utf_string_conversions.h"
 #include "chrome/browser/chrome_notification_types.h"
 #include "chrome/browser/content_settings/chrome_content_settings_utils.h"
 #include "chrome/browser/content_settings/cookie_settings_factory.h"
 #include "chrome/browser/content_settings/tab_specific_content_settings.h"
 #include "chrome/browser/custom_handlers/protocol_handler_registry.h"
 #include "chrome/browser/custom_handlers/protocol_handler_registry_factory.h"
 #include "chrome/browser/infobars/infobar_service.h"
 #include "chrome/browser/media/media_capture_devices_dispatcher.h"
 #include "chrome/browser/plugins/chrome_plugin_service_filter.h"
@@ skipping 29 matching lines @@
 using content::WebContents;
 using content_settings::SettingInfo;
 using content_settings::SettingSource;
 using content_settings::SETTING_SOURCE_USER;
 using content_settings::SETTING_SOURCE_NONE;
 
 namespace {
 
 const int kAllowButtonIndex = 0;
 
+// static
+const ContentSettingsType kSupportedBubbleTypes[] = {
+    CONTENT_SETTINGS_TYPE_COOKIES,
+    CONTENT_SETTINGS_TYPE_IMAGES,
+    CONTENT_SETTINGS_TYPE_JAVASCRIPT,
+    CONTENT_SETTINGS_TYPE_PPAPI_BROKER,
+    CONTENT_SETTINGS_TYPE_PLUGINS,
+    CONTENT_SETTINGS_TYPE_POPUPS,
+    CONTENT_SETTINGS_TYPE_GEOLOCATION,
+    CONTENT_SETTINGS_TYPE_MIXEDSCRIPT,
+    CONTENT_SETTINGS_TYPE_PROTOCOL_HANDLERS,
+    CONTENT_SETTINGS_TYPE_MEDIASTREAM,
+    CONTENT_SETTINGS_TYPE_MEDIASTREAM_MIC,
+    CONTENT_SETTINGS_TYPE_MEDIASTREAM_CAMERA,
+    CONTENT_SETTINGS_TYPE_AUTOMATIC_DOWNLOADS,
+    CONTENT_SETTINGS_TYPE_MIDI_SYSEX,
+};
+
 // These states must match the order of appearance of the radio buttons
 // in the XIB file for the Mac port.
 enum RPHState {
   RPH_ALLOW = 0,
   RPH_BLOCK,
   RPH_IGNORE,
 };
 
 struct ContentSettingsTypeIdEntry {
   ContentSettingsType type;
@@ skipping 1188 matching lines @@
 
   for (const std::pair<GURL, ContentSetting>& map_entry : state_map) {
     settings_map->SetContentSetting(
         ContentSettingsPattern::FromURLNoWildcard(map_entry.first),
         ContentSettingsPattern::FromURLNoWildcard(embedder_url),
         CONTENT_SETTINGS_TYPE_MIDI_SYSEX, std::string(),
         CONTENT_SETTING_DEFAULT);
   }
 }
 
+base::LazyInstance<std::set<ContentSettingsType>> g_supported_bubble_types =

[tapted, 2015/09/03 06:36:38]

either:

anonymous namespace + base::LazyInstance<std::set<ContentSettingsType>>::Leaky
(otherwise it registers an atexit handler to destroy it)

or (probably better)

in GetSupportedBubbleTypes, do

CR_DEFINE_STATIC_LOCAL(
    const std::set<ContentSettingsType>,
    supported_bubble_types,
    (kSupportedBubbleTypes,
         kSupportedBubbleTypes + arraysize(kSupportedBubbleTypes)));
return supported_bubble_types;

[raymes, 2015/09/07 04:29:27]

Ah you're right - I had this before but changed it but now I can do what you
suggested :)

+    LAZY_INSTANCE_INITIALIZER;
+
+// static
+const std::set<ContentSettingsType>&
+ContentSettingBubbleModel::GetSupportedBubbleTypes() {
+  std::set<ContentSettingsType>& types = g_supported_bubble_types.Get();
+  if (types.empty()) {
+    types.insert(kSupportedBubbleTypes,
+                 kSupportedBubbleTypes + arraysize(kSupportedBubbleTypes));
+  }
+  return types;
+}
+
 // static
 ContentSettingBubbleModel*
     ContentSettingBubbleModel::CreateContentSettingBubbleModel(
         Delegate* delegate,
         WebContents* web_contents,
         Profile* profile,
         ContentSettingsType content_type) {
+  DCHECK(ContainsKey(GetSupportedBubbleTypes(), content_type));

[tapted, 2015/09/03 06:36:38]

I've never really understood ContainsKey.... if you don't want stl_util.h, you
can just do

DCHECK(GetSupportedBubbleTypes().count(content_type));

but I guess not everyone sees set::count as being an alias for (non-existent)
set::contains

[raymes, 2015/09/07 04:29:27]

I agree with you - I used to do that but I've had other people tell me to do the
opposite :( I'll leave it as is for now.

   if (content_type == CONTENT_SETTINGS_TYPE_COOKIES) {
     return new ContentSettingCookiesBubbleModel(delegate, web_contents,
                                                 profile);
   }
   if (content_type == CONTENT_SETTINGS_TYPE_POPUPS) {
     return new ContentSettingPopupBubbleModel(delegate, web_contents, profile);
   }
   if (content_type == CONTENT_SETTINGS_TYPE_GEOLOCATION) {
     return new ContentSettingDomainListBubbleModel(delegate, web_contents,
                                                    profile, content_type);
@@ skipping 66 matching lines @@
   if (type == content::NOTIFICATION_WEB_CONTENTS_DESTROYED) {
     DCHECK_EQ(web_contents_,
               content::Source<WebContents>(source).ptr());
     web_contents_ = NULL;
   } else {
     DCHECK_EQ(chrome::NOTIFICATION_PROFILE_DESTROYED, type);
     DCHECK_EQ(profile_, content::Source<Profile>(source).ptr());
     profile_ = NULL;
   }
 }