[sensors] Add Permission guard to the generic sensor apis.

third_party/WebKit/Source/modules/sensor/SensorProxy.cpp

 // Copyright 2016 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 "modules/sensor/SensorProxy.h"
 
 #include "core/frame/LocalFrame.h"
+#include "modules/permissions/PermissionUtils.h"
 #include "modules/sensor/SensorProviderProxy.h"
 #include "modules/sensor/SensorReading.h"
+#include "platform/UserGestureIndicator.h"
 #include "platform/mojo/MojoHelper.h"
+#include "platform/weborigin/SecurityOrigin.h"
 #include "public/platform/Platform.h"
 
 using namespace device::mojom::blink;
 
 namespace blink {
 
+using mojom::blink::PermissionName;
+using mojom::blink::PermissionService;
+using mojom::blink::PermissionStatus;
+
 SensorProxy::SensorProxy(SensorType sensorType,
+                         PermissionService* permissionService,
+                         RefPtr<SecurityOrigin> origin,
                          SensorProviderProxy* provider,
                          std::unique_ptr<SensorReadingFactory> readingFactory)
     : m_type(sensorType),
       m_mode(ReportingMode::CONTINUOUS),
       m_provider(provider),
       m_clientBinding(this),
       m_state(SensorProxy::Uninitialized),
       m_suspended(false),
       m_readingFactory(std::move(readingFactory)),
-      m_maximumFrequency(0.0) {}
+      m_maximumFrequency(0.0),
+      m_permissionStatus(PermissionStatus::ASK),
+      m_permissionService(permissionService),
+      m_securityOrigin(std::move(origin)) {}
+
+void SensorProxy::onPermissionUpdate(PermissionStatus status) {
+  if (m_permissionStatus != status) {
+    m_permissionStatus = status;
+    SensorPermissionChanged();
+  }
+
+  // Keep listening to changes.
+  m_permissionService->GetNextPermissionChange(
+      createPermissionDescriptor(PermissionName::SENSORS), m_securityOrigin,
+      m_permissionStatus,
+      convertToBaseCallback(
+          WTF::bind(&SensorProxy::onPermissionUpdate, wrapPersistent(this))));

[Mikhail, 2016/11/16 13:04:28]

wrapPersistent will hold SensorProxy from deletion until the callback is
delivered, is it intentional?

[riju_, 2016/11/18 09:29:35]

Done.

+}
 
 SensorProxy::~SensorProxy() {}
 
 void SensorProxy::dispose() {
   m_clientBinding.Close();
 }
 
 DEFINE_TRACE(SensorProxy) {
   visitor->trace(m_reading);
   visitor->trace(m_observers);
@@ skipping 11 matching lines @@
 
 void SensorProxy::initialize() {
   if (m_state != Uninitialized)
     return;
 
   if (!m_provider->sensorProvider()) {
     handleSensorError();
     return;
   }
 
-  m_state = Initializing;
-  auto callback = convertToBaseCallback(
-      WTF::bind(&SensorProxy::onSensorCreated, wrapWeakPersistent(this)));
-  m_provider->sensorProvider()->GetSensor(m_type, mojo::GetProxy(&m_sensor),
-                                          callback);
+  // Request permission.
+  if (m_permissionStatus == PermissionStatus::ASK) {

[Mikhail, 2016/11/16 13:04:28]

nit: switch .. case .. would look better

[riju_, 2016/11/18 09:29:35]

Done.

+    m_permissionService->RequestPermission(
+        createPermissionDescriptor(PermissionName::SENSORS), m_securityOrigin,
+        UserGestureIndicator::processingUserGesture(),
+        convertToBaseCallback(
+            WTF::bind(&SensorProxy::onPermissionUpdate, wrapPersistent(this))));
+  } else if (m_permissionStatus == PermissionStatus::DENIED) {
+    handleSensorError(SecurityError, "Permission not granted.");

[Mikhail, 2016/11/16 13:04:28]

NotAllowedError

[riju_, 2016/11/18 09:29:35]

Done.

+  }

[Mikhail, 2016/11/16 13:04:28]

case for granted permissions?

[riju_, 2016/11/18 09:29:35]

Done.

 }
 
 void SensorProxy::addConfiguration(
     SensorConfigurationPtr configuration,
     std::unique_ptr<Function<void(bool)>> callback) {
   DCHECK(isInitialized());
   m_sensor->AddConfiguration(std::move(configuration),
                              convertToBaseCallback(std::move(callback)));
 }
 
@@ skipping 47 matching lines @@
 
 void SensorProxy::RaiseError() {
   handleSensorError();
 }
 
 void SensorProxy::SensorReadingChanged() {
   for (Observer* observer : m_observers)
     observer->onSensorReadingChanged();
 }
 
+void SensorProxy::SensorPermissionChanged() {

[Mikhail, 2016/11/16 13:04:28]

if proxy is not initialized, it should ignore this notification

[riju_, 2016/11/18 09:29:35]

Done.

+  // If Permission is revoked throw error.
+  if (m_permissionStatus != PermissionStatus::GRANTED) {
+    handleSensorError(SecurityError, "Permission not granted.");
+  } else {
+    if (!isInitialized()) {
+      m_state = Initializing;

[Mikhail, 2016/11/16 13:04:28]

why? 'Initializing' state should be cause *only* from 'initialize()' method
call.

[riju_, 2016/11/18 09:29:35]

Done.

+      auto callback = convertToBaseCallback(
+          WTF::bind(&SensorProxy::onSensorCreated, wrapWeakPersistent(this)));
+      m_provider->sensorProvider()->GetSensor(m_type, mojo::GetProxy(&m_sensor),
+                                              callback);
+    }
+  }
+}
+
 void SensorProxy::handleSensorError(ExceptionCode code,
                                     String sanitizedMessage,
                                     String unsanitizedMessage) {
   if (!Platform::current()) {
     // TODO(rockot): Remove this hack once renderer shutdown sequence is fixed.
     return;
   }
 
   m_state = Uninitialized;
   // The m_sensor.reset() will release all callbacks and its bound parameters,
@@ skipping 61 matching lines @@
   const device::OneWriterSeqLock& seqlock = buffer->seqlock.value();
   auto version = seqlock.ReadBegin();
   auto readingData = buffer->reading;
   if (seqlock.ReadRetry(version))
     return false;
   result = readingData;
   return true;
 }
 
 }  // namespace blink