Fix OperationSystemVersion() on Android.

Fix OperationSystemVersion() on Android.

Right now it returns the underlying kernal version instead of the Android version.

However, OperatingSystemVersionNumbers returns the Android version.

Per discussion with Android folks, we agree the Android version should be returned.

This CL copies what we do for Mac version.

This piece of information could be very useful in defining driver bug workarounds in a finer grid on Android.

BUG=225582
TEST=OperationSystemVersion() returns android version, see about:gpu
R=brettw

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=198629

[Zhenyao Mo, 2013-05-06 22:36:06]

Brett, Daniel, please take a look.

[brettw, 2013-05-06 22:56:24]

lgtm

https://codereview.chromium.org/14631004/diff/13001/base/sys_info_android.cc
File base/sys_info_android.cc (right):

https://codereview.chromium.org/14631004/diff/13001/base/sys_info_android.cc#...
base/sys_info_android.cc:141: return base::StringPrintf("%d.%d.%d", major,
minor, bugfix);
Don't need "base::" here (you're already in base).

[Zhenyao Mo, 2013-05-06 23:07:07]

https://codereview.chromium.org/14631004/diff/13001/base/sys_info_android.cc
File base/sys_info_android.cc (right):

https://codereview.chromium.org/14631004/diff/13001/base/sys_info_android.cc#...
base/sys_info_android.cc:141: return base::StringPrintf("%d.%d.%d", major,
minor, bugfix);
On 2013/05/06 22:56:24, brettw wrote:
> Don't need "base::" here (you're already in base).

Done.

[no sievers, 2013-05-06 23:10:21]

lgtm. thanks!

[commit-bot: I haz the power, 2013-05-06 23:11:28]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/zmo@chromium.org/14631004/22001

[commit-bot: I haz the power, 2013-05-07 04:52:47]

Change committed as 198629