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

chrome/browser/permissions/permission_manager.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_manager.h"
 
 #include <memory>
 #include <utility>
 
 #include "base/callback.h"
@@ skipping 86 matching lines @@
     case PermissionType::DURABLE_STORAGE:
       return CONTENT_SETTINGS_TYPE_DURABLE_STORAGE;
     case PermissionType::AUDIO_CAPTURE:
       return CONTENT_SETTINGS_TYPE_MEDIASTREAM_MIC;
     case PermissionType::VIDEO_CAPTURE:
       return CONTENT_SETTINGS_TYPE_MEDIASTREAM_CAMERA;
     case PermissionType::BACKGROUND_SYNC:
       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;
   }
 
   NOTREACHED() << "Unknown content setting for permission "
                << static_cast<int>(permission);
   return CONTENT_SETTINGS_TYPE_DEFAULT;
 }
 
@@ skipping 459 matching lines @@
     const GURL& embedding_origin) {
   PermissionContextBase* context = GetPermissionContext(permission);
   PermissionResult result = context->GetPermissionStatus(
       render_frame_host, requesting_origin.GetOrigin(),
       embedding_origin.GetOrigin());
   DCHECK(result.content_setting == CONTENT_SETTING_ALLOW ||
          result.content_setting == CONTENT_SETTING_ASK ||
          result.content_setting == CONTENT_SETTING_BLOCK);
   return result;
 }