[Sensors] Consider maximum supported frequency

third_party/WebKit/Source/modules/sensor/AmbientLightSensor.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/AmbientLightSensor.h"
 
 #include "bindings/core/v8/ScriptPromise.h"
 #include "bindings/core/v8/ScriptPromiseResolver.h"
 #include "modules/sensor/AmbientLightSensorReading.h"
 
@@ skipping 22 matching lines @@
 AmbientLightSensorReading* AmbientLightSensor::reading() const {
   return static_cast<AmbientLightSensorReading*>(Sensor::reading());
 }
 
 SensorReading* AmbientLightSensor::createSensorReading(SensorProxy* proxy) {
   return AmbientLightSensorReading::create(proxy);
 }
 
 auto AmbientLightSensor::createSensorConfig(
     const SensorOptions& options,
-    const SensorConfiguration& defaultConfig) -> SensorConfigurationPtr {
+    const SensorConfiguration& defaultConfig,
+    double maxSupportedFrequency) -> SensorConfigurationPtr {
   auto result = device::mojom::blink::SensorConfiguration::New();
-  result->frequency =
+  double frequency =
       options.hasFrequency() ? options.frequency() : defaultConfig.frequency;
+
+  if (maxSupportedFrequency && frequency > maxSupportedFrequency)

[shalamov, 2016/11/02 14:42:45]

Can this be moved to Sensor or another place, so that all sensors can reuse this
code?

+    frequency = maxSupportedFrequency;
+
+  result->frequency = frequency;
   return result;
 }
 
 DEFINE_TRACE(AmbientLightSensor) {
   Sensor::trace(visitor);
 }
 
 }  // namespace blink