Replace PermissionType in chrome/ with ContentSettingsType

chrome/browser/permissions/permission_util.cc

 // 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.
 
 #include "chrome/browser/permissions/permission_util.h"
 
 #include "build/build_config.h"
 #include "base/feature_list.h"
 #include "base/logging.h"
 #include "chrome/browser/content_settings/host_content_settings_map_factory.h"
 #include "chrome/browser/permissions/permission_uma_util.h"
 #include "chrome/common/chrome_features.h"
 #include "components/content_settings/core/browser/host_content_settings_map.h"
 #include "content/public/browser/permission_type.h"
 
 using content::PermissionType;
 
-std::size_t PermissionTypeHash::operator()(
-    const content::PermissionType& type) const {
-  return static_cast<size_t>(type);
+std::string PermissionUtil::GetPermissionString(
+    ContentSettingsType content_type) {
+  PermissionType permission_type;
+  CHECK(PermissionUtil::GetPermissionType(content_type, &permission_type));

[raymes, 2017/02/09 00:39:57]

nit: here and elsewhere. We tend not to use CHECK in chrome very often and it
seems fairly safe not to here. We might just want to DCHECK the result.

[Timothy Loh, 2017/02/10 07:25:38]

Done here and in permission_uma_util.cc

+  return GetPermissionString(permission_type);
 }
 
 // The returned strings must match the RAPPOR metrics in rappor.xml,
 // and any Field Trial configs for the Permissions kill switch e.g.
 // Permissions.Action.Geolocation etc..
-std::string PermissionUtil::GetPermissionString(
-    content::PermissionType permission) {
+std::string PermissionUtil::GetPermissionString(PermissionType permission) {
   switch (permission) {
-    case content::PermissionType::GEOLOCATION:
+    case PermissionType::GEOLOCATION:
       return "Geolocation";
-    case content::PermissionType::NOTIFICATIONS:
+    case PermissionType::NOTIFICATIONS:
       return "Notifications";
-    case content::PermissionType::MIDI_SYSEX:
+    case PermissionType::MIDI_SYSEX:
       return "MidiSysEx";
-    case content::PermissionType::PUSH_MESSAGING:
+    case PermissionType::PUSH_MESSAGING:
       return "PushMessaging";
-    case content::PermissionType::DURABLE_STORAGE:
+    case PermissionType::DURABLE_STORAGE:
       return "DurableStorage";
-    case content::PermissionType::PROTECTED_MEDIA_IDENTIFIER:
+    case PermissionType::PROTECTED_MEDIA_IDENTIFIER:
       return "ProtectedMediaIdentifier";
-    case content::PermissionType::AUDIO_CAPTURE:
+    case PermissionType::AUDIO_CAPTURE:
       return "AudioCapture";
-    case content::PermissionType::VIDEO_CAPTURE:
+    case PermissionType::VIDEO_CAPTURE:
       return "VideoCapture";
-    case content::PermissionType::MIDI:
+    case PermissionType::MIDI:
       return "Midi";
-    case content::PermissionType::BACKGROUND_SYNC:
+    case PermissionType::BACKGROUND_SYNC:
       return "BackgroundSync";
-    case content::PermissionType::FLASH:
+    case PermissionType::FLASH:
       return "Flash";
-    case content::PermissionType::NUM:
+    default:
       break;
   }
   NOTREACHED();
   return std::string();
 }
 
-std::string PermissionUtil::ConvertPermissionTypeToSafeBrowsingName(
-    const content::PermissionType& permission_type) {
+std::string PermissionUtil::ConvertContentSettingsTypeToSafeBrowsingName(
+    ContentSettingsType permission_type) {
   switch (permission_type) {
-    case content::PermissionType::GEOLOCATION:
+    case CONTENT_SETTINGS_TYPE_GEOLOCATION:
       return "GEOLOCATION";
-    case content::PermissionType::NOTIFICATIONS:
+    case CONTENT_SETTINGS_TYPE_NOTIFICATIONS:
       return "NOTIFICATIONS";
-    case content::PermissionType::MIDI_SYSEX:
+    case CONTENT_SETTINGS_TYPE_MIDI_SYSEX:
       return "MIDI_SYSEX";
-    case content::PermissionType::PUSH_MESSAGING:
+    case CONTENT_SETTINGS_TYPE_PUSH_MESSAGING:
       return "PUSH_MESSAGING";
-    case content::PermissionType::DURABLE_STORAGE:
+    case CONTENT_SETTINGS_TYPE_DURABLE_STORAGE:
       return "DURABLE_STORAGE";
-    case content::PermissionType::PROTECTED_MEDIA_IDENTIFIER:
+    case CONTENT_SETTINGS_TYPE_PROTECTED_MEDIA_IDENTIFIER:
       return "PROTECTED_MEDIA_IDENTIFIER";
-    case content::PermissionType::AUDIO_CAPTURE:
+    case CONTENT_SETTINGS_TYPE_MEDIASTREAM_MIC:
       return "AUDIO_CAPTURE";
-    case content::PermissionType::VIDEO_CAPTURE:
+    case CONTENT_SETTINGS_TYPE_MEDIASTREAM_CAMERA:
       return "VIDEO_CAPTURE";
-    case content::PermissionType::MIDI:
-      return "MIDI";
-    case content::PermissionType::BACKGROUND_SYNC:
+    case CONTENT_SETTINGS_TYPE_BACKGROUND_SYNC:
       return "BACKGROUND_SYNC";
-    case content::PermissionType::FLASH:
+    case CONTENT_SETTINGS_TYPE_PLUGINS:
       return "FLASH";
-    case content::PermissionType::NUM:
+    default:
       break;
   }
   NOTREACHED();
   return std::string();
 }
 
-PermissionRequestType PermissionUtil::GetRequestType(
-    content::PermissionType type) {
+PermissionRequestType PermissionUtil::GetRequestType(ContentSettingsType type) {
   switch (type) {
-    case content::PermissionType::GEOLOCATION:
+    case CONTENT_SETTINGS_TYPE_GEOLOCATION:
       return PermissionRequestType::PERMISSION_GEOLOCATION;
-    case content::PermissionType::NOTIFICATIONS:
+    case CONTENT_SETTINGS_TYPE_NOTIFICATIONS:
       return PermissionRequestType::PERMISSION_NOTIFICATIONS;
-    case content::PermissionType::MIDI_SYSEX:
+    case CONTENT_SETTINGS_TYPE_MIDI_SYSEX:
       return PermissionRequestType::PERMISSION_MIDI_SYSEX;
-    case content::PermissionType::PUSH_MESSAGING:
+    case CONTENT_SETTINGS_TYPE_PUSH_MESSAGING:
       return PermissionRequestType::PERMISSION_PUSH_MESSAGING;
-    case content::PermissionType::PROTECTED_MEDIA_IDENTIFIER:
+    case CONTENT_SETTINGS_TYPE_PROTECTED_MEDIA_IDENTIFIER:
       return PermissionRequestType::PERMISSION_PROTECTED_MEDIA_IDENTIFIER;
-    case content::PermissionType::FLASH:
+    case CONTENT_SETTINGS_TYPE_PLUGINS:
       return PermissionRequestType::PERMISSION_FLASH;
     default:
       NOTREACHED();
       return PermissionRequestType::UNKNOWN;
   }
 }
 
 PermissionRequestGestureType PermissionUtil::GetGestureType(bool user_gesture) {
   return user_gesture ? PermissionRequestGestureType::GESTURE
                       : PermissionRequestGestureType::NO_GESTURE;
 }
 
 bool PermissionUtil::GetPermissionType(ContentSettingsType type,
                                        PermissionType* out) {
   if (type == CONTENT_SETTINGS_TYPE_GEOLOCATION) {
     *out = PermissionType::GEOLOCATION;
   } else if (type == CONTENT_SETTINGS_TYPE_NOTIFICATIONS) {
     *out = PermissionType::NOTIFICATIONS;
+  } else if (type == CONTENT_SETTINGS_TYPE_PUSH_MESSAGING) {
+    *out = PermissionType::PUSH_MESSAGING;
   } else if (type == CONTENT_SETTINGS_TYPE_MIDI_SYSEX) {
     *out = PermissionType::MIDI_SYSEX;
   } else if (type == CONTENT_SETTINGS_TYPE_DURABLE_STORAGE) {
     *out = PermissionType::DURABLE_STORAGE;
   } else if (type == CONTENT_SETTINGS_TYPE_MEDIASTREAM_CAMERA) {
     *out = PermissionType::VIDEO_CAPTURE;
   } else if (type == CONTENT_SETTINGS_TYPE_MEDIASTREAM_MIC) {
     *out = PermissionType::AUDIO_CAPTURE;
   } else if (type == CONTENT_SETTINGS_TYPE_BACKGROUND_SYNC) {
     *out = PermissionType::BACKGROUND_SYNC;
+  } else if (type == CONTENT_SETTINGS_TYPE_PLUGINS) {
+    *out = PermissionType::FLASH;
 #if defined(OS_ANDROID) || defined(OS_CHROMEOS)
   } else if (type == CONTENT_SETTINGS_TYPE_PROTECTED_MEDIA_IDENTIFIER) {
     *out = PermissionType::PROTECTED_MEDIA_IDENTIFIER;
 #endif
   } else {
     return false;
   }
   return true;
 }
 
@@ skipping 43 matching lines @@
 PermissionUtil::ScopedRevocationReporter::~ScopedRevocationReporter() {
   if (!is_initially_allowed_)
     return;
   HostContentSettingsMap* settings_map =
       HostContentSettingsMapFactory::GetForProfile(profile_);
   ContentSetting final_content_setting = settings_map->GetContentSetting(
       primary_url_, secondary_url_, content_type_, std::string());
   if (final_content_setting != CONTENT_SETTING_ALLOW) {
     PermissionType permission_type;
     if (PermissionUtil::GetPermissionType(content_type_, &permission_type)) {
-      PermissionUmaUtil::PermissionRevoked(permission_type, source_ui_,
+      PermissionUmaUtil::PermissionRevoked(content_type_, source_ui_,
                                            primary_url_, profile_);
     }
   }
 }