Adds a couple more measures to the benchmarking app

apps/benchmark/benchmark_app.cc

 // Copyright 2015 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.
 
 #include <memory>
 #include <set>
 #include <string>
 #include <vector>
 
 #include "apps/benchmark/event.h"
@@ skipping 98 matching lines @@
     if (!GetEvents(trace_data, &events)) {
       LOG(ERROR) << "Failed to parse the trace data";
       mojo::ApplicationImpl::Terminate();
       return;
     }
 
     // Calculate and print the results.
     bool succeeded = true;
     Measurements measurements(events, time_origin_);
     for (const Measurement& measurement : args_.measurements) {
-      double result = measurements.Measure(measurement);
-      if (result >= 0.0) {
-        printf("measurement: %s %lf\n", measurement.spec.c_str(), result);
-      } else {
-        succeeded = false;
-        printf("measurement: %s FAILED\n", measurement.spec.c_str());
+      std::vector<double> results;
+      measurements.Measure(measurement, &results);
+      printf("measurement: %s", measurement.spec.c_str());
+      for (double result : results) {

[ppi, 2015/10/08 22:29:47]

As discussed offline, I think it would be good to stick with one result per
measurement. We could have the measurement take a percentile to compute as
parameter: percentile_duration/category/event_name/0.9.

[zra, 2015/10/09 14:51:15]

Done.

+        if (result >= 0.0) {
+          printf(" %lf", result);
+        } else {
+          succeeded = false;
+          printf(" FAILED");
+        }
       }
+      printf("\n");
     }
 
     // Scripts that run benchmarks can pick this up as a signal that the run
     // succeeded, as shell exit code is 0 even if an app exits early due to an
     // error.
     if (succeeded) {
       printf("benchmark succeeded\n");
     } else {
       printf("some measurements failed\n");
     }
@@ skipping 10 matching lines @@
 };
 }  // namespace
 }  // namespace benchmark
 
 MojoResult MojoMain(MojoHandle application_request) {
   mojo::ApplicationRunnerChromium runner(new benchmark::BenchmarkApp);
   auto ret = runner.Run(application_request);
   fflush(nullptr);
   return ret;
 }