Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(214)

Unified Diff: tools/telemetry/telemetry/page.py

Issue 12088107: Revert 180117, it broke all telemetry tests. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 7 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: tools/telemetry/telemetry/page.py
===================================================================
--- tools/telemetry/telemetry/page.py (revision 180130)
+++ tools/telemetry/telemetry/page.py (working copy)
@@ -9,7 +9,7 @@
from telemetry import util
class Page(object):
- def __init__(self, url, page_set, attributes=None, base_dir=None):
+ def __init__(self, url, attributes=None, base_dir=None):
parsed_url = urlparse.urlparse(url)
if not parsed_url.scheme:
abspath = os.path.abspath(os.path.join(base_dir, parsed_url.path))
@@ -18,7 +18,6 @@
else:
raise Exception('URLs must be fully qualified: %s' % url)
self.url = url
- self.page_set = page_set
self.base_dir = base_dir
self.credentials = None
self.disabled = False
@@ -60,10 +59,6 @@
return os.path.split(self.url)[1]
return re.sub('https?://', '', self.url)
- @property
- def archive_path(self):
- return self.page_set.WprFilePathForPage(self)
-
def __str__(self):
return self.url
« no previous file with comments | « tools/telemetry/telemetry/multi_page_benchmark_unittest_base.py ('k') | tools/telemetry/telemetry/page_runner.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698