Fix and cleanup Mojo and Mandoline python scripts.

mojo/tools/mopy/gtest.py

 # Copyright 2014 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.
 
 import logging
 import os
 import re
 import sys
 
 from mopy import test_util
 from mopy.print_process_error import print_process_error
 
 
 _logger = logging.getLogger()
 
 
 def set_color():
   """Run gtests with color if we're on a TTY (and we're not being told
   explicitly what to do)."""
   if sys.stdout.isatty() and "GTEST_COLOR" not in os.environ:
     _logger.debug("Setting GTEST_COLOR=yes")
     os.environ["GTEST_COLOR"] = "yes"
 
 # TODO(vtl): The return value is bizarre. Should just make it either return
-# True/False, or a list of failing fixtures. But the dart_apptest runner would
-# also need to be updated in the same way.
+# True/False, or a list of failing fixtures.
 def run_fixtures(config, shell, apptest_dict, apptest, isolate, test_args,
                  shell_args):
   """Run the gtest fixtures in isolation."""
 
   if not isolate:
     if not RunApptestInShell(config, shell, apptest, test_args, shell_args):
       return "Failed test(s) in %r" % apptest_dict
     return "Succeeded"
 
   # List the apptest fixtures so they can be run independently for isolation.
@@ skipping 86 matching lines @@
     if line[0] != " ":
       suite = line.strip()
       continue
     test_list.append(suite + line.strip())
 
   return test_list
 
 
 def RunApptestInShell(config, shell, application, application_args, shell_args):
   return run_test(config, shell, shell_args, {application: application_args})