Index: tools/telemetry/telemetry/benchmark.py |
diff --git a/tools/telemetry/telemetry/benchmark.py b/tools/telemetry/telemetry/benchmark.py |
index 7f3f1aefb0dc5ed6ac1420da3ca693b912cdfe32..0c94fc1ef004f68d64c1afb5cf71864ed229e2cf 100644 |
--- a/tools/telemetry/telemetry/benchmark.py |
+++ b/tools/telemetry/telemetry/benchmark.py |
@@ -11,7 +11,6 @@ from telemetry.internal.results import results_options |
from telemetry.internal import story_runner |
from telemetry.internal.util import exception_formatter |
from telemetry import page |
-from telemetry.page import page_set |
from telemetry.page import page_test |
from telemetry.page import test_expectations |
from telemetry.web_perf import timeline_based_measurement |
@@ -242,24 +241,17 @@ class Benchmark(command_line.Command): |
self.SetupTraceRerunOptions(options, opts) |
return timeline_based_measurement.TimelineBasedMeasurement(opts) |
- def CreatePageSet(self, options): # pylint: disable=unused-argument |
- """Get the page set this test will run on. |
- |
- By default, it will create a page set from the this test's page_set |
- attribute. Override to generate a custom page set. |
- """ |
- if not hasattr(self, 'page_set'): |
- raise NotImplementedError('This test has no "page_set" attribute.') |
- if not issubclass(self.page_set, page_set.PageSet): |
- raise TypeError('"%s" is not a PageSet.' % self.page_set.__name__) |
- return self.page_set() |
- |
def CreateStorySet(self, options): |
"""Creates the instance of StorySet used to run the benchmark. |
Can be overridden by subclasses. |
""" |
- return self.CreatePageSet(options) |
+ del options # unused |
+ # TODO(aiolos, nednguyen, eakufner): replace class attribute page_set with |
+ # story_set. |
+ if not hasattr(self, 'page_set'): |
+ raise NotImplementedError('This test has no "page_set" attribute.') |
+ return self.page_set() |
@classmethod |
def CreateExpectations(cls): |