Revert of Discover should return a list of classes, not a dictionary. (patchset ...
commiteae6808489d37aaeede466164d89ce3ecb65d8bd
authorwittman <wittman@chromium.org>
Fri, 26 Jun 2015 01:51:56 +0000 (25 18:51 -0700)
committerCommit bot <commit-bot@chromium.org>
Fri, 26 Jun 2015 01:52:34 +0000 (26 01:52 +0000)
tree31e15a18c8dcfdbc35318e78ffd61c2df1ac52ca
parent05810c666d13e5a7f6f29be8ee038bd2fc980613
Revert of Discover should return a list of classes, not a dictionary. (patchset #4 id:120001 of https://codereview.chromium.org/1194723002/)

Reason for revert:
Appears to break benchmarks.benchmark_smoke_unittest.BenchmarkSmokeTest.startup.cold.blank_page

[1/1] benchmarks.benchmark_smoke_unittest.BenchmarkSmokeTest.startup.cold.blank_page failed unexpectedly 0.1250s:
  Pages: []
  RESULT telemetry_page_measurement_results: num_failed= 1 count
  RESULT telemetry_page_measurement_results: num_errored= 0 count

  View result at file://E:\b\build\slave\XP_Tests__1_\build\src\tools\telemetry\telemetry\unittest_util\results.html
  Benchmark startup.cold.blank_page run takes 0 seconds

  Traceback (most recent call last):
    Run at tools\telemetry\telemetry\benchmark.py:194
      max_failures=self._max_failures)
    Run at tools\telemetry\telemetry\internal\story_runner.py:216
      expectations, user_story, results, state)
    _RunUserStoryAndProcessErrorIfNeeded at tools\telemetry\telemetry\internal\story_runner.py:78
      state.WillRunUserStory(user_story)
    WillRunUserStory at tools\telemetry\telemetry\page\shared_page_state.py:161
      self.browser = self._possible_browser.Create(self._finder_options)
    Create at tools\telemetry\telemetry\internal\backends\chrome\desktop_browser_finder.py:67
      browser_backend, self._platform_backend, self._credentials_path)
    __init__ at tools\telemetry\telemetry\core\browser.py:47
      self._browser_backend.profile_directory)
    FlushSystemCacheForDirectory at tools\telemetry\telemetry\core\platform\__init__.py:159
      return self._platform_backend.FlushSystemCacheForDirectory(directory)
    FlushSystemCacheForDirectory at tools\telemetry\telemetry\core\platform\desktop_platform_backend.py:23
      assert flush_command, 'You must build clear_system_cache first'
  AssertionError: You must build clear_system_cache first

  Locals:
    directory     : 'c:\\docume~1\\chrome~2\\locals~1\\temp\\tmpmxy3b0'
    flush_command : None

  Traceback (most recent call last):
    File "E:\b\build\slave\XP_Tests__1_\build\src\tools\telemetry\telemetry\decorators.py", line 118, in wrapper
      func(*args, **kwargs)
    File "E:\b\build\slave\XP_Tests__1_\build\src\tools\perf\benchmarks\benchmark_smoke_unittest.py", line 75, in BenchmarkSmokeTest
      msg='Failed: %s' % benchmark)
  AssertionError: Failed: <class 'benchmarks.startup.StartupColdBlankPage'>

https://build.chromium.org/p/chromium.win/builders/XP%20Tests%20%281%29/builds/38411

Original issue's description:
> Discover should return a list of classes, not a dictionary.
> Add classes_util.
>
> BUG=498968
> CQ_EXTRA_TRYBOTS=tryserver.chromium.perf:linux_perf_bisect;tryserver.chromium.perf:mac_perf_bisect;tryserver.chromium.perf:win_perf_bisect;tryserver.chromium.perf:android_nexus5_perf_bisect
>
> Committed: https://crrev.com/57fa7b351455df84db4507f370417c4808101e99
> Cr-Commit-Position: refs/heads/master@{#336283}

TBR=dtu@chromium.org,nednguyen@google.com,kbr@chromium.org,bengr@chromium.org,aiolos@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=498968

Review URL: https://codereview.chromium.org/1216433004

Cr-Commit-Position: refs/heads/master@{#336320}
20 files changed:
content/test/gpu/page_sets/__init__.py
tools/chrome_proxy/integration_tests/chrome_proxy_pagesets/__init__.py
tools/chrome_proxy/live_tests/pagesets/__init__.py
tools/perf/benchmarks/benchmark_smoke_unittest.py
tools/perf/benchmarks/benchmark_unittest.py
tools/perf/benchmarks/skpicture_printer.py
tools/perf/measurements/measurement_smoke_test.py
tools/perf/page_sets/__init__.py
tools/perf/profile_creators/profile_generator.py
tools/telemetry/telemetry/benchmark_runner.py
tools/telemetry/telemetry/core/discover.py
tools/telemetry/telemetry/core/discover_unittest.py
tools/telemetry/telemetry/core/platform/__init__.py
tools/telemetry/telemetry/core/platform/profiler/profiler_finder.py
tools/telemetry/telemetry/core/platform/tracing_controller_backend.py
tools/telemetry/telemetry/internal/util/find_dependencies.py
tools/telemetry/telemetry/record_wpr.py
tools/telemetry/telemetry/unittest_util/story_set_smoke_test.py
tools/telemetry/telemetry/util/classes_util.py [deleted file]
tools/telemetry/telemetry/value/__init__.py