[tools/perf] Enable smoke coverage for all telemetry benchmarks

tools/perf/benchmarks/benchmark_smoke_unittest.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.
 
 """Run the first page of one benchmark for every module.
 
 Only benchmarks that have a composable measurement are included.
 Ideally this test would be comprehensive, however, running one page
 of every benchmark would run impractically long.
 """
@@ skipping 66 matching lines @@
       self.skipTest('Benchmark %s has ShouldDisable return True' %
                     SinglePageBenchmark.Name())
 
     self.assertEqual(0, SinglePageBenchmark().Run(options),
                      msg='Failed: %s' % benchmark)
 
   return BenchmarkSmokeTest
 
 
 # The list of benchmark modules to be excluded from our smoke tests.
+# TODO(nednguyen): remove black listing by module & change this to black listing
+# by specific benchmark.
 _BLACK_LIST_TEST_MODULES = {
     image_decoding,  # Always fails on Mac10.9 Tests builder.
     indexeddb_perf,  # Always fails on Win7 & Android Tests builder.
     octane,  # Often fails & take long time to timeout on cq bot.
     rasterize_and_record_micro,  # Always fails on cq bot.
     repaint,  # Often fails & takes long time to timeout on cq bot.
     spaceport,  # Takes 451 seconds.
     speedometer,  # Takes 101 seconds.
     jetstream,  # Take 206 seconds.
     text_selection,  # Always fails on cq bot.
     memory  # Flaky on bots, crbug.com/513767.
 }
 
 # Some smoke benchmark tests that run quickly on desktop platform can be very
 # slow on Android. So we create a separate set of black list only for Android.
 _ANDROID_BLACK_LIST_MODULES = {
     kraken,  # Takes 275 seconds on Android.
     sunspider,  # Takes 163 seconds on Android.
     tracing,  # Failed on Android bot, crbug.com/600989.
 }
 
 
 def load_tests(loader, standard_tests, pattern):
   del loader, standard_tests, pattern  # unused
   suite = progress_reporter.TestSuite()
 
   benchmarks_dir = os.path.dirname(__file__)
   top_level_dir = os.path.dirname(benchmarks_dir)
 
-  # Using the default of |index_by_class_name=False| means that if a module
-  # has multiple benchmarks, only the last one is returned.
   all_benchmarks = discover.DiscoverClasses(
-      benchmarks_dir, top_level_dir, benchmark_module.Benchmark,
-      index_by_class_name=False).values()
+      benchmarks_dir, top_level_dir, benchmark_module.Benchmark).values()
   for benchmark in all_benchmarks:
     if sys.modules[benchmark.__module__] in _BLACK_LIST_TEST_MODULES:
       continue
-    # TODO(tonyg): Smoke doesn't work with session_restore yet.
-    if (benchmark.Name().startswith('session_restore') or
-        benchmark.Name().startswith('skpicture_printer')):
-      continue
 
     if hasattr(benchmark, 'generated_profile_archive'):
       # We'd like to test these, but don't know how yet.
       continue
 
     class BenchmarkSmokeTest(unittest.TestCase):
       pass
 
     method = SmokeTestGenerator(benchmark)
 
@@ skipping 18 matching lines @@
 
     # TODO(bashi): Remove once crrev.com/1266833004 is landed.
     if benchmark.Name() == 'memory.blink_memory_mobile':
       method._disabled_strings.add('android')
 
     setattr(BenchmarkSmokeTest, benchmark.Name(), method)
 
     suite.addTest(BenchmarkSmokeTest(benchmark.Name()))
 
   return suite