Update GTest Parsing Code

Update GTest parsing code to allow show timed out tests as failures (even if they are prefixed FAILS_)


BUG=128978
TEST=Tests that are marked as FAILS_ but timeout should now be listed as reason for step failure.


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=142713

[csharp, 2012-06-18 14:51:35]

Fix the issue of the timeout failure getting ignore. I'm now going to look into
why the other 3 ignore tests weren't ignored.

[M-A Ruel, 2012-06-18 14:55:57]

lgtm with nit

https://chromiumcodereview.appspot.com/10556042/diff/1/scripts/common/unittes...
File scripts/common/unittests/gtest_utils_test.py (right):

https://chromiumcodereview.appspot.com/10556042/diff/1/scripts/common/unittes...
scripts/common/unittests/gtest_utils_test.py:20: 'SomeOtherTest.FAILS_FooBar']
maybe renaming it to 
SomeOtherTest.FAILS_ThisTestTimesOut
would make the test case more self-descriptive.

[commit-bot: I haz the power, 2012-06-18 15:17:33]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/csharp@chromium.org/10556042/5001

[csharp, 2012-06-18 15:17:39]

https://chromiumcodereview.appspot.com/10556042/diff/1/scripts/common/unittes...
File scripts/common/unittests/gtest_utils_test.py (right):

https://chromiumcodereview.appspot.com/10556042/diff/1/scripts/common/unittes...
scripts/common/unittests/gtest_utils_test.py:20: 'SomeOtherTest.FAILS_FooBar']
On 2012/06/18 14:55:57, Marc-Antoine Ruel wrote:
> maybe renaming it to 
> SomeOtherTest.FAILS_ThisTestTimesOut
> would make the test case more self-descriptive.

Done.

[commit-bot: I haz the power, 2012-06-18 15:19:39]

Presubmit check for 10556042-5001 failed and returned exit status 1.

Running presubmit commit checks ...

** Presubmit Warnings **
Found lines longer than 80 characters (first 5 shown).
  build/scripts/common/unittests/gtest_utils_test.py, line 129, 114 chars \
  build/scripts/common/unittests/gtest_utils_test.py, line 240, 114 chars

Presubmit checks took 122.8s to calculate.

[commit-bot: I haz the power, 2012-06-18 15:32:23]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/csharp@chromium.org/10556042/8001

[commit-bot: I haz the power, 2012-06-18 15:34:29]

Change committed as 142713