Remove unused PermissionRequestType::MEDIA_STREAM/UNUSED_PERMISSION
Change-Id: I219a0dfc48e834e55b35e376378fd3775fff6ee0
Reviewed-on: https://chromium-review.googlesource.com/821331
Reviewed-by: Raymes Khoury <[email protected]>
Commit-Queue: Timothy Loh <[email protected]>
Cr-Commit-Position: refs/heads/master@{#523355}diff --git a/chrome/browser/permissions/permission_request.h b/chrome/browser/permissions/permission_request.h
index 06509ec..f4473c3f 100644
--- a/chrome/browser/permissions/permission_request.h
+++ b/chrome/browser/permissions/permission_request.h
@@ -24,10 +24,10 @@
enum class PermissionRequestType {
UNKNOWN = 0,
MULTIPLE = 1,
- UNUSED_PERMISSION = 2,
+ // UNUSED_PERMISSION = 2,
QUOTA = 3,
DOWNLOAD = 4,
- MEDIA_STREAM = 5,
+ // MEDIA_STREAM = 5,
REGISTER_PROTOCOL_HANDLER = 6,
PERMISSION_GEOLOCATION = 7,
PERMISSION_MIDI_SYSEX = 8,
diff --git a/chrome/browser/permissions/permission_uma_util.cc b/chrome/browser/permissions/permission_uma_util.cc
index d8bc3d6..9c3832c 100644
--- a/chrome/browser/permissions/permission_uma_util.cc
+++ b/chrome/browser/permissions/permission_uma_util.cc
@@ -105,11 +105,6 @@
if (requests.size() > 1)
type = PermissionRequestType::MULTIPLE;
- // This is only hit if kUsePermissionManagerForMediaRequests is off, since it
- // is now on by default we'll just silenty drop this.
- if (type == PermissionRequestType::MEDIA_STREAM)
- return;
-
DCHECK(action == "Accepted" || action == "Denied" || action == "Dismissed" ||
action == "Ignored");
std::string name = "Permissions.Engagement." + action + '.' +
