Add android_stats.py script

Add android_stats.py script

Dumps out information about all connected devices:
- Battery
- Temperature
- Model name

...in JSON format.

BUG=skia:4606
Committed: https://skia.googlesource.com/buildbot/+/ca966f9ca59b8dc9e880b129aa3768eb69965f8c

[borenet, 2015-12-01 16:19:26]

borenet@google.com changed reviewers:
+ rmistry@google.com

[rmistry, 2015-12-01 16:33:17]

https://codereview.chromium.org/1491603002/diff/20001/scripts/android_stats.py
File scripts/android_stats.py (right):

https://codereview.chromium.org/1491603002/diff/20001/scripts/android_stats.p...
scripts/android_stats.py:43: cmd = ['adb', '-s', serial, 'shell', 'dumpsys',
'batteryproperties']
I ran this on N5 and it said:
Can't find service: batteryproperties.

https://codereview.chromium.org/1491603002/diff/20001/scripts/android_stats.p...
scripts/android_stats.py:54: temp_file =
'/sys/devices/virtual/thermal/thermal_zone0/temp'
What about the other zones?
On the N5 it is showing 0-16 zones.

https://codereview.chromium.org/1491603002/diff/20001/scripts/android_stats.p...
scripts/android_stats.py:63: temp /= 10
Why is this necessary? do some devices output 30C as 300C ?

[borenet, 2015-12-01 20:00:06]

borenet@google.com changed reviewers:
+ djsollen@google.com

[borenet, 2015-12-01 20:00:07]

+djsollen for advice on better ways to get this information.

https://codereview.chromium.org/1491603002/diff/20001/scripts/android_stats.py
File scripts/android_stats.py (right):

https://codereview.chromium.org/1491603002/diff/20001/scripts/android_stats.p...
scripts/android_stats.py:43: cmd = ['adb', '-s', serial, 'shell', 'dumpsys',
'batteryproperties']
On 2015/12/01 16:33:17, rmistry wrote:
> I ran this on N5 and it said:
> Can't find service: batteryproperties.

Yeah, the services are apparently different between OS versions.

https://codereview.chromium.org/1491603002/diff/20001/scripts/android_stats.p...
scripts/android_stats.py:54: temp_file =
'/sys/devices/virtual/thermal/thermal_zone0/temp'
On 2015/12/01 16:33:17, rmistry wrote:
> What about the other zones?
> On the N5 it is showing 0-16 zones.

Dunno.  I'm still hoping to find a better way to do this.

https://codereview.chromium.org/1491603002/diff/20001/scripts/android_stats.p...
scripts/android_stats.py:63: temp /= 10
On 2015/12/01 16:33:17, rmistry wrote:
> Why is this necessary? do some devices output 30C as 300C ?

Yes.  The two devices on my desk report 2780 and 31.

[djsollen, 2015-12-02 15:46:04]

lgtm

https://codereview.chromium.org/1491603002/diff/20001/scripts/android_stats.py
File scripts/android_stats.py (right):

https://codereview.chromium.org/1491603002/diff/20001/scripts/android_stats.p...
scripts/android_stats.py:43: cmd = ['adb', '-s', serial, 'shell', 'dumpsys',
'batteryproperties']
On 2015/12/01 20:00:07, borenet wrote:
> On 2015/12/01 16:33:17, rmistry wrote:
> > I ran this on N5 and it said:
> > Can't find service: batteryproperties.
> 
> Yeah, the services are apparently different between OS versions.

I think that batteryproperties should be available in 5.0 and beyond.

https://codereview.chromium.org/1491603002/diff/20001/scripts/android_stats.p...
scripts/android_stats.py:54: temp_file =
'/sys/devices/virtual/thermal/thermal_zone0/temp'
On 2015/12/01 20:00:07, borenet wrote:
> On 2015/12/01 16:33:17, rmistry wrote:
> > What about the other zones?
> > On the N5 it is showing 0-16 zones.
> 
> Dunno.  I'm still hoping to find a better way to do this.

You could loop through all the zones and print the type and temp of each and
return the max.

[borenet, 2015-12-02 15:59:26]

https://codereview.chromium.org/1491603002/diff/20001/scripts/android_stats.py
File scripts/android_stats.py (right):

https://codereview.chromium.org/1491603002/diff/20001/scripts/android_stats.p...
scripts/android_stats.py:54: temp_file =
'/sys/devices/virtual/thermal/thermal_zone0/temp'
On 2015/12/02 15:46:03, djsollen wrote:
> On 2015/12/01 20:00:07, borenet wrote:
> > On 2015/12/01 16:33:17, rmistry wrote:
> > > What about the other zones?
> > > On the N5 it is showing 0-16 zones.
> > 
> > Dunno.  I'm still hoping to find a better way to do this.
> 
> You could loop through all the zones and print the type and temp of each and
> return the max.

I think I'll leave as-is for now and update if/when we want
more/better/different data.

[borenet, 2015-12-02 15:59:29]

The CQ bit was checked by borenet@google.com

[commit-bot: I haz the power, 2015-12-02 15:59:34]

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

[commit-bot: I haz the power, 2015-12-02 16:03:48]

Description was changed from

==========
Add android_stats.py script

Dumps out information about all connected devices:
- Battery
- Temperature
- Model name

...in JSON format.

BUG=skia:4606
==========

to

==========
Add android_stats.py script

Dumps out information about all connected devices:
- Battery
- Temperature
- Model name

...in JSON format.

BUG=skia:4606

Committed:
https://skia.googlesource.com/buildbot/+/ca966f9ca59b8dc9e880b129aa3768eb6996...
==========

[commit-bot: I haz the power, 2015-12-02 16:03:50]

Committed patchset #2 (id:20001) as
https://skia.googlesource.com/buildbot/+/ca966f9ca59b8dc9e880b129aa3768eb6996...