[Password manager tests] Fix indentation bug & use subprocesses to run tests.

[Password manager tests] Fix bugs & use subprocesses to run tests.

Fixes indentation bug in test runner and switches from using os to
subprocesses.

R=vabr@chromium.org
BUG=435249
Committed: https://crrev.com/5fa5a2a1173354c137769ad146f3a78fa922e3ed
Cr-Commit-Position: refs/heads/master@{#313711}

[melandory, 2015-01-29 09:33:13]

Patchset #1 (id:1) has been deleted

[vabr (Chromium), 2015-01-29 14:21:44]

LGTM!
Thanks a lot for these improvements!

(Just out of curiosity, what bugs were there apart from the bad indentation and
use of os.system()?)

Cheers,
Vaclav

[melandory, 2015-01-29 14:23:04]

On 2015/01/29 14:21:44, vabr (Chromium) wrote:
> LGTM!
> Thanks a lot for these improvements!
> 
> (Just out of curiosity, what bugs were there apart from the bad indentation
and
> use of os.system()?)
> 
> Cheers,
> Vaclav

Hm, you right. Plural is wrong here. I'll change it before submitting the code.

[melandory, 2015-01-29 14:23:30]

The CQ bit was checked by melandory@chromium.org

[commit-bot: I haz the power, 2015-01-29 14:24:37]

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

[commit-bot: I haz the power, 2015-01-29 15:38:13]

Committed patchset #1 (id:20001)

[commit-bot: I haz the power, 2015-01-29 15:39:12]

Patchset 1 (id:??) landed as
https://crrev.com/5fa5a2a1173354c137769ad146f3a78fa922e3ed
Cr-Commit-Position: refs/heads/master@{#313711}