Some python test runner files are mixing 2 and 4 space indentation levels, I'm pretty sure we're settled on 4 as the convention.

rs=me to just forcibly re-indent whole files wherever you want.

In bug 1127376 I used autopep8 [1] to not only fix whitespace, but other pep8 violations. I'd be down with running all our code through this, two space or not. Though some people will dislike how certain things get styled, at least it enforces a convention. From bug 1127376 comment 7 > To replicate this patch: > > $ pip install --upgrade autopep8 > $ autopep8 -i -a -a -r testing/mochitest --exclude 'testing/mochitest/pywebsocket/*' [1] https://github.com/hhatto/autopep8

Consistency is better than the current situation, and PEP8 is generally considered canonical in the Python world, so I'm in agreement for this for all of our testing code.

Enforced by lint now, I believe.