Run mojo_benchmark on the perf bot.

mojo/tools/get_test_list.py

 #!/usr/bin/env python
 # 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.
 
 """Central list of tests to run (as appropriate for a given config). Add tests
 to run by modifying this file.
 
 Note that this file is both imported (by mojob.py) and run directly (via a
 recipe)."""
@@ skipping 194 matching lines @@
       if config.values.get("builder_name"):
         command += ["--builder-name", config.values["builder_name"]]
       if config.values.get("build_number"):
         command += ["--build-number", config.values["build_number"]]
       if config.values.get("master_name"):
         command += ["--master-name", config.values["master_name"]]
       command += [os.path.join(build_dir, test_name)]
 
       AddEntry(test_name, command)
 
+  # Benchmarks -----------------------------------------------------------------
+
+  if target_os == Config.OS_LINUX and ShouldRunTest(Config.TEST_TYPE_PERF):

[viettrungluu, 2015/10/01 15:22:06]

Seems kind of pointless to run them and not record results.

[ppi, 2015/10/01 15:25:04]

This is meant to catch breakages, e.g. benchmarks no longer producing results.

Before we record results we need to teach mojo_benchmark about the compatible
format, I was planning to do that separately.

[viettrungluu, 2015/10/01 19:38:39]

This suggests to me that you need actual tests for the benchmarks. Just running
this here will only catch relatively trivial errors. Moreover, this isn't run on
the trybots, etc.

+    command = ([os.path.join("mojo", "devtools", "common", "mojo_benchmark"),
+               os.path.join("mojo", "tools", "data", "benchmarks")] +
+               verbose_flags)
+    if not config.is_debug:
+      command.append("--release")
+
+    AddEntry("benchmarks", command)
+
   # Integration tests ----------------------------------------------------------
 
   if target_os == Config.OS_ANDROID and ShouldRunTest(
       Config.TEST_TYPE_DEFAULT, Config.TEST_TYPE_INTEGRATION):
     AddEntry("Integration test (MojoTest)",
              ["python",
               os.path.join("build", "android", "test_runner.py"),
               "instrumentation",
               "--test-apk=MojoTest",
               "--output-directory=%s" % build_dir,
@@ skipping 16 matching lines @@
   config = Config(**json.load(args.config_file))
   test_list = GetTestList(config)
   json.dump(test_list, args.test_list_file, indent=2)
   args.test_list_file.write("\n")
 
   return 0
 
 
 if __name__ == "__main__":
   sys.exit(main())