Added support for improvement_direction to relevant values, which is propogated to chartjson.

tools/perf/benchmarks/service_worker.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 collections
 import page_sets
 import re
 
 from measurements import timeline_controller
 from metrics import speedindex
@@ skipping 11 matching lines @@
     for counter_name, counter in process.counters.iteritems():
       if not counter_filter.search(counter_name):
         continue
 
       total = sum(counter.totals)
 
       # Results objects cannot contain the '.' character, so remove that here.
       sanitized_counter_name = counter_name.replace('.', '_')
 
       results.AddValue(scalar.ScalarValue(
-          results.current_page, sanitized_counter_name, 'count', total))
+          results.current_page, sanitized_counter_name, 'count', total,
+          higher_is_better=False))
       results.AddValue(scalar.ScalarValue(
           results.current_page, sanitized_counter_name + '_avg', 'count',
-          total / float(len(counter.totals))))
+          total / float(len(counter.totals)), higher_is_better=False))
 
   def AddResultsOfEvents(
       self, process, thread_regex_string, event_regex_string, results):
     thread_filter = re.compile(thread_regex_string)
     event_filter = re.compile(event_regex_string)
 
     for thread in process.threads.itervalues():
       thread_name = thread.name.replace('/', '_')
       if not thread_filter.search(thread_name):
         continue
@@ skipping 22 matching lines @@
 
   def _AddResultOfEvent(self, thread_name, event_name, times, results):
     total = sum(times)
     biggest_jank = max(times)
 
     # Results objects cannot contain the '.' character, so remove that here.
     sanitized_event_name = event_name.replace('.', '_')
 
     full_name = thread_name + '|' + sanitized_event_name
     results.AddValue(scalar.ScalarValue(
-        results.current_page, full_name, 'ms', total))
+        results.current_page, full_name, 'ms', total, higher_is_better=False))
     results.AddValue(scalar.ScalarValue(
-        results.current_page, full_name + '_max', 'ms', biggest_jank))
+        results.current_page, full_name + '_max', 'ms', biggest_jank,
+        higher_is_better=False))
     results.AddValue(scalar.ScalarValue(
-        results.current_page, full_name + '_avg', 'ms', total / len(times)))
+        results.current_page, full_name + '_avg', 'ms', total / len(times),
+        higher_is_better=False))
 
 
 class _ServiceWorkerMeasurement(page_test.PageTest):
   """Measure Speed Index and TRACE_EVENTs"""
 
   def __init__(self, *args, **kwargs):
     super(_ServiceWorkerMeasurement, self).__init__(*args, **kwargs)
     self._timeline_controller = timeline_controller.TimelineController()
     self._speed_index = speedindex.SpeedIndexMetric()
     self._page_open_times = collections.defaultdict(int)
@@ skipping 85 matching lines @@
   test = _ServiceWorkerMeasurement
   page_set = page_sets.ServiceWorkerPageSet
 
 
 # Disabled due to redness on the tree. crbug.com/442752
 @benchmark.Disabled('reference')
 class ServiceWorkerMicroBenchmarkPerfTest(benchmark.Benchmark):
   """Service Worker performance test using a micro benchmark page set"""
   test = _ServiceWorkerMicroBenchmarkMeasurement
   page_set = page_sets.ServiceWorkerMicroBenchmarkPageSet