[sensors][win] Use angular velocity values for gyroscope sensor

[sensors][win] Use angular velocity values for gyroscope sensor

This CL aligns implementation of gyroscope sensor with the
specification and uses angular velocity reading values instead
of angular acceleration. Unit tests updated to verify changes.

BUG=669765
Review-Url: https://codereview.chromium.org/2648423005
Cr-Commit-Position: refs/heads/master@{#445974}
Committed: https://chromium.googlesource.com/chromium/src/+/988f519584a682a2191c6ca72f91380f91d8166a

[shalamov, 2017-01-24 14:04:31]

The CQ bit was checked by alexander.shalamov@intel.com to run a CQ dry run

[commit-bot: I haz the power, 2017-01-24 14:04:49]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[shalamov, 2017-01-24 14:22:25]

alexander.shalamov@intel.com changed reviewers:
+ mikhail.pozdnyakov@intel.com, reillyg@chromium.org

[shalamov, 2017-01-24 14:22:25]

PTAL

[Mikhail, 2017-01-24 14:24:48]

lgtm

https://codereview.chromium.org/2648423005/diff/1/device/generic_sensor/platf...
File device/generic_sensor/platform_sensor_reader_win.cc (right):

https://codereview.chromium.org/2648423005/diff/1/device/generic_sensor/platf...
device/generic_sensor/platform_sensor_reader_win.cc:126: reading.values[0] = -x
* kRadiansInDegreesPerSecond;
s/kRadiansInDegreesPerSecond/kRadiansInDegrees

[shalamov, 2017-01-24 14:47:05]

The CQ bit was checked by alexander.shalamov@intel.com to run a CQ dry run

[commit-bot: I haz the power, 2017-01-24 14:47:20]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-01-24 15:35:57]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-01-24 15:35:58]

Dry run: This issue passed the CQ dry run.

[Reilly Grant (use Gerrit), 2017-01-24 20:05:15]

lgtm but please check that you have the right issue number in the BUG= line. The
current one does not seem related.

[shalamov, 2017-01-25 07:20:35]

On 2017/01/24 20:05:15, Reilly Grant wrote:
> lgtm but please check that you have the right issue number in the BUG= line.
The
> current one does not seem related.

It is a correct bug, unfortunately, description is cryptic and talks about
ambient light tests, so, I couldn't figure out what to fix until I talked to
reporter.

[shalamov, 2017-01-25 07:20:45]

The CQ bit was checked by alexander.shalamov@intel.com

[shalamov, 2017-01-25 07:20:45]

The patchset sent to the CQ was uploaded after l-g-t-m from
mikhail.pozdnyakov@intel.com
Link to the patchset: https://codereview.chromium.org/2648423005/#ps20001
(title: "Rename constant")

[commit-bot: I haz the power, 2017-01-25 07:20:58]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-01-25 07:25:35]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1485328845093550,
"parent_rev": "28e80d9b1e987a61d52b7a934391bbe89c6ed71f", "commit_rev":
"988f519584a682a2191c6ca72f91380f91d8166a"}

[commit-bot: I haz the power, 2017-01-25 07:26:10]

Description was changed from

==========
[sensors][win] Use angular velocity values for gyroscope sensor

This CL aligns implementation of gyroscope sensor with the
specification and uses angular velocity reading values instead
of angular acceleration. Unit tests updated to verify changes.

BUG=669765
==========

to

==========
[sensors][win] Use angular velocity values for gyroscope sensor

This CL aligns implementation of gyroscope sensor with the
specification and uses angular velocity reading values instead
of angular acceleration. Unit tests updated to verify changes.

BUG=669765

Review-Url: https://codereview.chromium.org/2648423005
Cr-Commit-Position: refs/heads/master@{#445974}
Committed:
https://chromium.googlesource.com/chromium/src/+/988f519584a682a2191c6ca72f91...
==========

[commit-bot: I haz the power, 2017-01-25 07:26:11]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/988f519584a682a2191c6ca72f91...