Convert AutoBlocker static class to KeyedService.

chrome/browser/permissions/permission_util.h

 // Copyright 2015 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.
 
 #ifndef CHROME_BROWSER_PERMISSIONS_PERMISSION_UTIL_H_
 #define CHROME_BROWSER_PERMISSIONS_PERMISSION_UTIL_H_
 
 #include <string>
 
 #include "base/macros.h"
 #include "chrome/browser/permissions/permission_request.h"
 #include "components/content_settings/core/common/content_settings.h"
 #include "components/content_settings/core/common/content_settings_types.h"
 #include "url/gurl.h"
 
 class Profile;
 
+// Anonymous namespace for internal constants.
+namespace {
+  const char kGeolocationPermissionName[] = "Geolocation";
+  const char kNotificationPermissionName[] = "Notifications";
+  const char kMidiSysExnPermissionName[] = "MidiSysEx";
+  const char kPushMessagingPermissionName[] = "PushMessaging";
+  const char kDurableStoragePermissionName[] = "DurableStorage";
+  const char kProtectedMediaPermissionName[] = "ProtectedMediaIdentifier";
+  const char kAudioCapturePermissionName[] = "AudioCapture";
+  const char kVideoCapturePermissionName[] = "VideoCapture";
+  const char kMidiPermissionName[] = "Midi";
+  const char kBackgroundSyncPermissionName[] = "BackgroundSync";
+  const char kFlashPermissionName[] = "Flash";

[raymes, 2017/01/24 05:15:17]

Is it necessary to define these as constants?

[meredithl, 2017/01/24 23:20:21]

Not since I've changed the conversion to return a Safe Browsing string rather
than the PermissionType.

+} // namespace
+
 namespace content {
 enum class PermissionType;
 }  // namespace content
 
 enum class PermissionSourceUI;
 
 // This enum backs a UMA histogram, so it must be treated as append-only.
 enum PermissionAction {
   GRANTED = 0,
   DENIED = 1,
@@ skipping 10 matching lines @@
 struct PermissionTypeHash {
   std::size_t operator()(const content::PermissionType& type) const;
 };
 
 // A utility class for permissions.
 class PermissionUtil {
  public:
   // Returns the permission string for the given PermissionType.
   static std::string GetPermissionString(content::PermissionType permission);
 
+  // Converts between the stringified PermissionType enum that Safe Browsing
+  // returns to a content::PermissionType.
+  static content::PermissionType ConvertSafeBrowsingNameToPermissionType(
+      const std::string& sb_name);
+
   // Returns the request type corresponding to a permission type.
   static PermissionRequestType GetRequestType(content::PermissionType type);
 
   // Returns the gesture type corresponding to whether a permission request is
   // made with or without a user gesture.
   static PermissionRequestGestureType GetGestureType(bool user_gesture);
 
   // Limited conversion of ContentSettingsType to PermissionType. Intended for
   // recording Permission UMA metrics from areas of the codebase which have not
   // yet been converted to PermissionType. Returns true if the conversion was
@@ skipping 31 matching lines @@
     ContentSettingsType content_type_;
     PermissionSourceUI source_ui_;
     bool is_initially_allowed_;
   };
 
  private:
   DISALLOW_IMPLICIT_CONSTRUCTORS(PermissionUtil);
 };
 
 #endif  // CHROME_BROWSER_PERMISSIONS_PERMISSION_UTIL_H_