Made junit coverage-dir take directory, not file.

build/android/pylib/junit/test_runner.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 json
 import os
 import tempfile
 
 from devil.utils import cmd_helper
 from pylib import constants
@@ skipping 29 matching lines @@
         jar_args.extend(['-package-filter', self._package_filter])
       if self._runner_filter:
         jar_args.extend(['-runner-filter', self._runner_filter])
       if self._sdk_version:
         jar_args.extend(['-sdk-version', self._sdk_version])
       command.extend(['--jar-args', '"%s"' % ' '.join(jar_args)])
 
       # Add JVM arguments.
       jvm_args = []
       if self._coverage_dir:
-        jvm_args.append('-Demma.coverage.out.file=%s' % self._coverage_dir)
+        jvm_args.append('-Demma.coverage.out.file=%s' % os.path.join(

[jbudorick, 2016/05/24 12:35:18]

This should check that self._coverage_dir exists and should make it (via
https://docs.python.org/2/library/os.html#os.makedirs) if it doesn't.

[BigBossZhiling, 2016/05/24 17:41:33]

That's a good point, but I just tested on it and it seems like this emma
coverage will automatically create the directory.

[jbudorick, 2016/05/24 19:15:09]

I'd still prefer we create it here.

[BigBossZhiling, 2016/05/24 19:28:52]

Done.

+            self._coverage_dir, '%s.ec' % self._test_suite))
       if jvm_args:
         command.extend(['--jvm-args', '"%s"' % ' '.join(jvm_args)])
 
       return_code = cmd_helper.RunCmd(command)
       results_list = json_results.ParseResultsFromJson(
           json.loads(json_file.read()))
       return (results_list, return_code)
 
   def TearDown(self):
     pass