diff options
author | aiolos <aiolos@chromium.org> | 2015-08-03 14:41:10 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2015-08-03 21:41:44 +0000 |
commit | fd9cce5babbe6f4341793c2cd21753a9fe773244 (patch) | |
tree | 1c43ccce0741616a3c59c8f4aff971374832a016 /tools/chrome_proxy | |
parent | 4a71a24c0d566117eeec1e20968ab6f345a3c73e (diff) | |
download | chromium_src-fd9cce5babbe6f4341793c2cd21753a9fe773244.zip chromium_src-fd9cce5babbe6f4341793c2cd21753a9fe773244.tar.gz chromium_src-fd9cce5babbe6f4341793c2cd21753a9fe773244.tar.bz2 |
Revert of Create classes_util API, change discover to return a list instead of a dict. (patchset #1 id:1 of https://codereview.chromium.org/1262623007/)
Reason for revert:
Landed even though the patch failed.
Original issue's description:
> Reland of Create classes_util API, change discover to return a list instead of a dict. (patchset #1 id:1 of https://codereview.chromium.org/1263063003/)
>
> Reason for revert:
> The telemetry tests aren't broken. The perf_unittests are because they don't pick tests in a deterministic way. Reverting after a change to make them deterministic.
>
> Original issue's description:
> > Revert of Create classes_util API, change discover to return a list instead of a dict. (patchset #5 id:120001 of https://codereview.chromium.org/1244223002/)
> >
> > Reason for revert:
> > This appears to break the XP telemetry tests. Details on the bug.
> >
> > Original issue's description:
> > > Create classes_util API, change discover to return a list instead of a dict.
> > >
> > > 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/e6cbec4747aa7fe3c96b1cdb89de21ae77b30ab0
> > > Cr-Commit-Position: refs/heads/master@{#341129}
> >
> > TBR=dtu@chromium.org,bengr@chromium.org,aiolos@chromium.org
> > NOPRESUBMIT=true
> > NOTREECHECKS=true
> > NOTRY=true
> > BUG=498968
> >
> > Committed: https://crrev.com/c85e3e1af7e170d5f8ec2b012df6e9337d2352bb
> > Cr-Commit-Position: refs/heads/master@{#341218}
>
> TBR=dtu@chromium.org,bengr@chromium.org,avi@chromium.org
> BUG=498968
>
> Committed: https://crrev.com/2c1531af216c026be0b4aed672dc89d264f95a72
> Cr-Commit-Position: refs/heads/master@{#341603}
TBR=dtu@chromium.org,bengr@chromium.org,avi@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=498968
Review URL: https://codereview.chromium.org/1272433002
Cr-Commit-Position: refs/heads/master@{#341621}
Diffstat (limited to 'tools/chrome_proxy')
-rw-r--r-- | tools/chrome_proxy/integration_tests/chrome_proxy_pagesets/__init__.py | 6 | ||||
-rw-r--r-- | tools/chrome_proxy/live_tests/pagesets/__init__.py | 5 |
2 files changed, 6 insertions, 5 deletions
diff --git a/tools/chrome_proxy/integration_tests/chrome_proxy_pagesets/__init__.py b/tools/chrome_proxy/integration_tests/chrome_proxy_pagesets/__init__.py index c5699ad..799c246 100644 --- a/tools/chrome_proxy/integration_tests/chrome_proxy_pagesets/__init__.py +++ b/tools/chrome_proxy/integration_tests/chrome_proxy_pagesets/__init__.py @@ -6,9 +6,8 @@ import inspect import os import sys +from telemetry.core import discover from telemetry import story -from telemetry.util import classes_util - import video @@ -16,7 +15,8 @@ import video start_dir = os.path.dirname(os.path.abspath(__file__)) top_level_dir = os.path.abspath(os.path.join(start_dir, os.pardir, os.pardir)) base_class = story.StorySet -for cls in classes_util.DiscoverClasses(start_dir, top_level_dir, base_class): +for cls in discover.DiscoverClasses( + start_dir, top_level_dir, base_class).values(): setattr(sys.modules[__name__], cls.__name__, cls) # DiscoverClasses makes the assumption that there is exactly one matching diff --git a/tools/chrome_proxy/live_tests/pagesets/__init__.py b/tools/chrome_proxy/live_tests/pagesets/__init__.py index a67d12d..45ce4af 100644 --- a/tools/chrome_proxy/live_tests/pagesets/__init__.py +++ b/tools/chrome_proxy/live_tests/pagesets/__init__.py @@ -6,14 +6,15 @@ import inspect import os import sys +from telemetry.core import discover from telemetry import story -from telemetry.util import classes_util # Import all submodules' StorySet classes. start_dir = os.path.dirname(os.path.abspath(__file__)) top_level_dir = os.path.abspath(os.path.join(start_dir, os.pardir, os.pardir)) base_class = story.StorySet -for cls in classes_util.DiscoverClasses(start_dir, top_level_dir, base_class): +for cls in discover.DiscoverClasses( + start_dir, top_level_dir, base_class).values(): setattr(sys.modules[__name__], cls.__name__, cls) |