[Telemetry] Update the code that detect windows version to use CurrentMajorVersionNumber (Reland)

[Telemetry] Update the code that detect windows version to use CurrentMajorVersionNumber (Reland)

Since Windows 8, platform.uname()'s third field starts with 6.2, hence to detect
Windows 10, we have to use CurrentMajorVersionNumber from registry.

This is a reland of https://codereview.chromium.org/1830253003/
Fix is added in patch 2.

BUG=chromium:596640
Committed: https://chromium.googlesource.com/external/github.com/catapult-project/catapult/+/02c0e106d409492a00606015e02a566943708bc8

[nednguyen, 2016-03-28 18:17:18]

Description was changed from

==========
[Telemetry] Update the code that detect windows version to use
CurrentMajorVersionNumber (Reland)

Since Windows 8, platform.uname()'s third field starts with 6.2, hence to detect
Windows 10, we have to use CurrentMajorVersionNumber from registry.

BUG=chromium:596640
==========

to

==========
[Telemetry] Update the code that detect windows version to use
CurrentMajorVersionNumber (Reland)

Since Windows 8, platform.uname()'s third field starts with 6.2, hence to detect
Windows 10, we have to use CurrentMajorVersionNumber from registry.

This is a reland of https://codereview.chromium.org/1830253003/
Fix is added in patch 2.

BUG=chromium:596640
==========

[nednguyen, 2016-03-28 18:17:50]

nednguyen@google.com changed reviewers:
+ brucedawson@chromium.org

[nednguyen, 2016-03-28 18:17:58]

The CQ bit was checked by nednguyen@google.com to run a CQ dry run

[commit-bot: I haz the power, 2016-03-28 18:18:05]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1839713002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1839713002/20001

[commit-bot: I haz the power, 2016-03-28 18:53:59]

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

[commit-bot: I haz the power, 2016-03-28 18:54:00]

Dry run: This issue passed the CQ dry run.

[brucedawson, 2016-03-29 00:53:26]

lgtm

[nednguyen, 2016-03-29 01:18:25]

The CQ bit was checked by nednguyen@google.com

[commit-bot: I haz the power, 2016-03-29 01:18:31]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1839713002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1839713002/20001

[commit-bot: I haz the power, 2016-03-29 01:20:08]

Description was changed from

==========
[Telemetry] Update the code that detect windows version to use
CurrentMajorVersionNumber (Reland)

Since Windows 8, platform.uname()'s third field starts with 6.2, hence to detect
Windows 10, we have to use CurrentMajorVersionNumber from registry.

This is a reland of https://codereview.chromium.org/1830253003/
Fix is added in patch 2.

BUG=chromium:596640
==========

to

==========
[Telemetry] Update the code that detect windows version to use
CurrentMajorVersionNumber (Reland)

Since Windows 8, platform.uname()'s third field starts with 6.2, hence to detect
Windows 10, we have to use CurrentMajorVersionNumber from registry.

This is a reland of https://codereview.chromium.org/1830253003/
Fix is added in patch 2.

BUG=chromium:596640

Committed:
https://chromium.googlesource.com/external/github.com/catapult-project/catapu...
==========

[commit-bot: I haz the power, 2016-03-29 01:20:09]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/external/github.com/catapult-project/catapu...