Add print of all system properties after provisioning.

Add print of all system properties after provisioning.

BUG=

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

[zty, 2014-04-25 00:50:48]

lgtm

[navabi, 2014-04-25 01:44:43]

The CQ bit was checked by navabi@google.com

[commit-bot: I haz the power, 2014-04-25 02:03:28]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/navabi@google.com/257743002/10001

[commit-bot: I haz the power, 2014-04-25 02:03:29]

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

[commit-bot: I haz the power, 2014-04-25 02:03:30]

No LGTM from a valid reviewer yet. Only full committers are accepted.
Even if an LGTM may have been provided, it was from a non-committer or
a lowly provisional committer, _not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[tonyg, 2014-04-26 00:41:30]

https://codereview.chromium.org/257743002/diff/10001/build/android/pylib/andr...
File build/android/pylib/android_commands.py (right):

https://codereview.chromium.org/257743002/diff/10001/build/android/pylib/andr...
build/android/pylib/android_commands.py:639: 'su': 'RunShellCommandWithSU',
We should be able to add your method name here instead of adding skip_check. Or
else, perhaps we should get rid of whitelisted_callers and have
RunShellCommandWithSU pass skip_check.

[navabi, 2014-04-26 22:52:06]

https://codereview.chromium.org/257743002/diff/10001/build/android/pylib/andr...
File build/android/pylib/android_commands.py (right):

https://codereview.chromium.org/257743002/diff/10001/build/android/pylib/andr...
build/android/pylib/android_commands.py:639: 'su': 'RunShellCommandWithSU',
On 2014/04/26 00:41:31, tonyg wrote:
> We should be able to add your method name here instead of adding skip_check.
Or
> else, perhaps we should get rid of whitelisted_callers and have
> RunShellCommandWithSU pass skip_check.

Done.

[navabi, 2014-04-26 22:54:40]

https://codereview.chromium.org/257743002/diff/30001/build/android/pylib/andr...
File build/android/pylib/android_commands.py (right):

https://codereview.chromium.org/257743002/diff/30001/build/android/pylib/andr...
build/android/pylib/android_commands.py:643: base_command =
shlex.split(command)[0].strip(';')
without this call to strip the call would actually not raise this error, but for
the wrong reason: i.e. because GetShellCommandStatusAndOutput adds a ';'
character to 'getprop' so it could then echo the output, and that makes the
check base_command in preferred_apis return false.

With this strip, it will catch things like this, making it necessary to
whitelest getprop. I whitelisted calls from ProvisionDevices function.

[tonyg, 2014-04-27 19:03:57]

lgtm

[commit-bot: I haz the power, 2014-04-28 05:54:10]

Change committed as 266433