Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1196)

Side by Side Diff: device/generic_sensor/public/cpp/sensor_struct_traits.cc

Issue 2353493002: [Sensors] Allow Sensor API only on secure top-level browsing contexts and add frequency checks (Closed)
Patch Set: Comments from Tim Created 4 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "device/generic_sensor/public/cpp/sensor_struct_traits.h" 5 #include "device/generic_sensor/public/cpp/sensor_struct_traits.h"
6 6
7 namespace mojo { 7 namespace mojo {
8 8
9 // static 9 // static
10 bool StructTraits<device::mojom::SensorConfigurationDataView, 10 bool StructTraits<device::mojom::SensorConfigurationDataView,
11 device::PlatformSensorConfiguration>:: 11 device::PlatformSensorConfiguration>::
12 Read(device::mojom::SensorConfigurationDataView data, 12 Read(device::mojom::SensorConfigurationDataView data,
13 device::PlatformSensorConfiguration* out) { 13 device::PlatformSensorConfiguration* out) {
14 // Maximum allowed frequency is capped to 60Hz. 14 // Maximum allowed frequency is capped to 60Hz.
15 if (data.frequency() > 15 if (data.frequency() >
16 device::PlatformSensorConfiguration::kMaxAllowedFrequency || 16 device::mojom::SensorConfiguration::kMaxAllowedFrequency ||
17 data.frequency() <= 0.0) { 17 data.frequency() <= 0.0) {
18 return false; 18 return false;
19 } 19 }
20 20
21 out->set_frequency(data.frequency()); 21 out->set_frequency(data.frequency());
22 return true; 22 return true;
23 } 23 }
24 24
25 } // namespace mojo 25 } // namespace mojo
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698