[Android] Fix type-parameterized gtest parsing.

build/android/pylib/gtest/test_package.py

 # Copyright (c) 2012 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 """Base class representing GTest test packages."""
 # pylint: disable=R0201
 
 
 class TestPackage(object):
 
@@ skipping 70 matching lines @@
       A list of all tests. For the above raw listing:
 
       [IPCChannelTest.SendMessageInChannelConnected, IPCSyncChannelTest.Simple,
        IPCSyncChannelTest.DISABLED_SendWithTimeoutMixedOKAndTimeout]
     """
     ret = []
     current = ''
     for test in raw_list:
       if not test:
         continue
-      if test[0] != ' ' and not test.endswith('.'):
-        # Ignore any lines with unexpected format.
-        continue
-      if test[0] != ' ' and test.endswith('.'):
-        current = test
+      if test[0] != ' ':
+        test_case = test.split(None, 1)[0]

[craigdh, 2014/08/05 20:59:31]

how is this different from test.split()[0]?

[jbudorick, 2014/08/05 21:04:10]

As before, functionally the same, but it only does one split.

I guess it doesn't really matter, so switching to the simpler one.

[craigdh, 2014/08/05 21:23:33]

Ah, I never saw your response to the other one. I like erring on the side of
readability for Python when it's as minor of a perf tradeoff as this (I had to
go to pydoc to remember the order of params for split :)

+        if test_case.endswith('.'):
+          current = test_case
         continue

[craigdh, 2014/08/05 20:59:31]

use an else instead

[jbudorick, 2014/08/05 21:04:10]

Done.

       if 'YOU HAVE' in test:
         break
       test_name = test.split(None, 1)[0]
       ret += [current + test_name]
     return ret