[Sensors] Consider maximum supported frequency

device/generic_sensor/platform_sensor.h

 // 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.
 
 #ifndef DEVICE_GENERIC_SENSOR_PLATFORM_SENSOR_H_
 #define DEVICE_GENERIC_SENSOR_PLATFORM_SENSOR_H_
 
 #include <list>
 #include <map>
 
@@ skipping 23 matching lines @@
     virtual void OnSensorError() = 0;
     virtual bool IsNotificationSuspended() = 0;
 
    protected:
     virtual ~Client() {}
   };
 
   virtual mojom::ReportingMode GetReportingMode() = 0;
   virtual PlatformSensorConfiguration GetDefaultConfiguration() = 0;
 
+  // Can be overriden to return the sensor maximum sampling frequency
+  // value obtained from the platfrom if it is available. If platfrom

[timvolodine, 2016/11/03 17:32:56]

platfrom->platform

[Mikhail, 2016/11/04 21:17:44]

Done.

+  // does not provide maximum sampling frequency this method must
+  // return default frequency.
+  // The default implementation returns default frequency.
+  virtual double GetMaximumSupportedFrequency();

[timvolodine, 2016/11/03 17:32:56]

const? (or is that too restrictive?)

[Mikhail, 2016/11/04 21:17:44]

would not like to put such a limit for implementation, other virtual getters are
non-const as well.

+
   mojom::SensorType GetType() const;
 
   bool StartListening(Client* client,
                       const PlatformSensorConfiguration& config);
   bool StopListening(Client* client, const PlatformSensorConfiguration& config);
 
   void UpdateSensor();
 
   void AddClient(Client*);
   void RemoveClient(Client*);
@@ skipping 36 matching lines @@
   base::ObserverList<Client, true> clients_;
   ConfigMap config_map_;
   PlatformSensorProvider* provider_;
   base::WeakPtrFactory<PlatformSensor> weak_factory_;
   DISALLOW_COPY_AND_ASSIGN(PlatformSensor);
 };
 
 }  // namespace device
 
 #endif  // DEVICE_GENERIC_SENSOR_PLATFORM_SENSOR_H_