DeviceUtils: Fix a couple of corner-cases when wrapping commands.

DeviceUtils: Fix a couple of corner-cases when wrapping commands.

DeviceUtils.RunShellCommand() was missing ()s when wrapping, causing
commands containing ;s to not work properly.

AdbWrapper.py was missing a space after ()s, causing an error when the
wrapped command begins with a (. This is because (( != ( (.

BUG=

Committed: https://crrev.com/c4d22ef711dee64e24e805eb6fe30c1121fc7178
Cr-Commit-Position: refs/heads/master@{#351412}

[agrieve, 2015-09-29 17:19:18]

agrieve@chromium.org changed reviewers:
+ jbudorick@chromium.org

[agrieve, 2015-09-29 17:23:03]

On 2015/09/29 17:19:18, agrieve wrote:
> mailto:agrieve@chromium.org changed reviewers:
> + mailto:jbudorick@chromium.org

please review.

[jbudorick, 2015-09-29 17:24:33]

lgtm

https://codereview.chromium.org/1372873003/diff/1/build/android/devil/android...
File build/android/devil/android/device_utils.py (right):

https://codereview.chromium.org/1372873003/diff/1/build/android/devil/android...
build/android/devil/android/device_utils.py:755: cmd = '( %s )>%s' % (cmd,
large_output_file.name)
nice catch here.

[agrieve, 2015-09-29 20:14:33]

The CQ bit was checked by agrieve@chromium.org

[agrieve, 2015-09-29 20:14:33]

The patchset sent to the CQ was uploaded after l-g-t-m from
jbudorick@chromium.org
Link to the patchset: https://codereview.chromium.org/1372873003/#ps20001
(title: "rebase")

[commit-bot: I haz the power, 2015-09-29 20:14:45]

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

[commit-bot: I haz the power, 2015-09-29 22:22:35]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-09-29 22:23:46]

Patchset 2 (id:??) landed as
https://crrev.com/c4d22ef711dee64e24e805eb6fe30c1121fc7178
Cr-Commit-Position: refs/heads/master@{#351412}