[sensors][permission] Add new permission types in permission module.

chrome/browser/permissions/permission_manager.cc

Index: chrome/browser/permissions/permission_manager.cc
diff --git a/chrome/browser/permissions/permission_manager.cc b/chrome/browser/permissions/permission_manager.cc
index bc068fc197fd3efacf50652d8bc8dbabe86966dd..d39294dcaa9ddbdc4bb93e06fb10037ad39df0a7 100644
--- a/chrome/browser/permissions/permission_manager.cc
+++ b/chrome/browser/permissions/permission_manager.cc
@@ -104,6 +104,15 @@ ContentSettingsType PermissionTypeToContentSetting(PermissionType permission) {
       return CONTENT_SETTINGS_TYPE_BACKGROUND_SYNC;
     case PermissionType::FLASH:
       return CONTENT_SETTINGS_TYPE_PLUGINS;
+    // TODO(riju): Add ContentSetting entries for sensors later.

[raymes, 2017/07/19 02:42:10]

We probably only want to add 1 ContentSettingsType and have all these map to the
same.

[Reilly Grant (use Gerrit), 2017/07/19 21:30:27]

I'm tempted to say that even adding multiple PermissionType enum values for the
different sensors adds unnecessary complexity so all we need is to support
multiple string names in ParsePermission().

[riju_, 2017/07/20 11:07:40]

Done.

+    case PermissionType::AMBIENT_LIGHT_SENSOR:
+      return CONTENT_SETTINGS_TYPE_DEFAULT;
+    case PermissionType::ACCELEROMETER:
+      return CONTENT_SETTINGS_TYPE_DEFAULT;
+    case PermissionType::GYROSCOPE:
+      return CONTENT_SETTINGS_TYPE_DEFAULT;
+    case PermissionType::MAGNETOMETER:
+      return CONTENT_SETTINGS_TYPE_DEFAULT;
     case PermissionType::NUM:
       // This will hit the NOTREACHED below.
       break;