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

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

Issue 2301523007: Add trace events for some of telemetry internal functions (Closed)
Patch Set: Address Petr's nits Created 4 years, 3 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: telemetry/telemetry/page/legacy_page_test.py
diff --git a/telemetry/telemetry/page/legacy_page_test.py b/telemetry/telemetry/page/legacy_page_test.py
index 1c26c3658240b3fe5bd5890c9fea4af0599080b4..f8147f2996c369e7158f5b063cb800e62b0d00b0 100644
--- a/telemetry/telemetry/page/legacy_page_test.py
+++ b/telemetry/telemetry/page/legacy_page_test.py
@@ -4,6 +4,8 @@
import logging
+from py_trace_event import trace_event
+
from telemetry.core import exceptions
from telemetry.internal.actions import action_runner as action_runner_module
@@ -11,6 +13,7 @@ from telemetry.internal.actions import action_runner as action_runner_module
from telemetry.web_perf.story_test import Failure
petrcermak 2016/09/07 09:38:24 nit: there should be only 2 blank lines (remove on
nednguyen 2016/09/07 17:26:12 Done.
+
class TestNotSupportedOnPlatformError(Exception):
"""LegacyPageTest Exception raised when a required feature is unavailable.
@@ -53,6 +56,8 @@ class LegacyPageTest(object):
page, 'body_children', 'count', body_child_count))
"""
+ __metaclass__ = trace_event.TracedMetaClass
+
def __init__(self,
needs_browser_restart_after_each_page=False,
clear_cache_before_each_run=False):

Powered by Google App Engine
This is Rietveld 408576698