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

Side by Side Diff: device/generic_sensor/public/cpp/platform_sensor_configuration.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/platform_sensor_configuration.h" 5 #include "device/generic_sensor/public/cpp/platform_sensor_configuration.h"
6 6
7 #include "device/generic_sensor/public/interfaces/sensor.mojom.h"
8
7 namespace device { 9 namespace device {
8 10
9 PlatformSensorConfiguration::PlatformSensorConfiguration(double frequency) 11 PlatformSensorConfiguration::PlatformSensorConfiguration(double frequency)
10 : frequency_(frequency) { 12 : frequency_(frequency) {
11 DCHECK(frequency_ <= kMaxAllowedFrequency && frequency_ > 0.0); 13 DCHECK(frequency_ <= mojom::SensorConfiguration::kMaxAllowedFrequency &&
14 frequency_ > 0.0);
12 } 15 }
13 16
14 PlatformSensorConfiguration::PlatformSensorConfiguration() = default; 17 PlatformSensorConfiguration::PlatformSensorConfiguration() = default;
15 PlatformSensorConfiguration::~PlatformSensorConfiguration() = default; 18 PlatformSensorConfiguration::~PlatformSensorConfiguration() = default;
16 19
20 void PlatformSensorConfiguration::set_frequency(double frequency) {
21 DCHECK(frequency_ <= mojom::SensorConfiguration::kMaxAllowedFrequency &&
22 frequency_ > 0.0);
23 frequency_ = frequency;
24 }
25
17 bool PlatformSensorConfiguration::operator==( 26 bool PlatformSensorConfiguration::operator==(
18 const PlatformSensorConfiguration& other) const { 27 const PlatformSensorConfiguration& other) const {
19 return frequency_ == other.frequency(); 28 return frequency_ == other.frequency();
20 } 29 }
21 30
22 bool PlatformSensorConfiguration::operator>( 31 bool PlatformSensorConfiguration::operator>(
23 const PlatformSensorConfiguration& other) const { 32 const PlatformSensorConfiguration& other) const {
24 return frequency() > other.frequency(); 33 return frequency() > other.frequency();
25 } 34 }
26 35
27 } // namespace device 36 } // namespace device
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698