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

Unified Diff: tools/telemetry/telemetry/value/trace.py

Issue 663023005: [Telemetry] Add file handle & trace value. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Ready to land ^^ Created 6 years, 2 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/value/trace.py
diff --git a/tools/telemetry/telemetry/value/trace.py b/tools/telemetry/telemetry/value/trace.py
new file mode 100644
index 0000000000000000000000000000000000000000..34aae1abbe45212a7cfc816bb54f3f90aeb4dccf
--- /dev/null
+++ b/tools/telemetry/telemetry/value/trace.py
@@ -0,0 +1,79 @@
+# Copyright 2014 The Chromium Authors. All rights reserved.
+# Use of this source code is governed by a BSD-style license that can be
+# found in the LICENSE file.
+
+import tempfile
+
+from telemetry import value as value_module
+from telemetry.util import file_handle
+import telemetry.web_components # pylint: disable=W0611
+from trace_viewer.build import trace2html
+
+class TraceValue(value_module.Value):
+ def __init__(self, page, tracing_timeline_data, important=False,
+ description=None):
+ """A value that contains a TracingTimelineData object and knows how to
+ output it.
+
+ Adding TraceValues and outputting as JSON will produce a directory full of
+ HTML files called trace_files. Outputting as chart JSON will also produce
+ an index, files.html, linking to each of these files.
+ """
+ super(TraceValue, self).__init__(
+ page, name='trace', units='', important=important,
+ description=description)
+
+ tf = tempfile.NamedTemporaryFile(delete=False, suffix='.html')
achuithb 2014/11/20 21:40:50 Is this file cleaned up?
+ if page:
+ title = page.display_name
+ else:
+ title = ''
+ trace2html.WriteHTMLForTraceDataToFile(
+ [tracing_timeline_data.EventData()], title, tf)
+ tf.close()
+
+ self._file_handle = file_handle.FromTempFile(tf)
+
+ def GetAssociatedFileHandle(self):
+ return self._file_handle
+
+ def __repr__(self):
+ if self.page:
+ page_name = self.page.url
+ else:
+ page_name = None
+ return 'TraceValue(%s, %s)' % (page_name, self.name)
+
+ def GetBuildbotDataType(self, output_context):
+ return None
+
+ def GetBuildbotValue(self):
+ return None
+
+ def GetRepresentativeNumber(self):
+ return None
+
+ def GetRepresentativeString(self):
+ return None
+
+ @staticmethod
+ def GetJSONTypeName():
+ return 'trace'
+
+ @classmethod
+ def MergeLikeValuesFromSamePage(cls, values):
+ # TODO(eakuefner): Implement a MultiTraceValue: a Polymer-based,
+ # componentized, MultiTraceViwer-backed representation of more than one
+ # trace.
+ assert len(values) > 0
+ return values[0]
+
+ @classmethod
+ def MergeLikeValuesFromDifferentPages(cls, values,
+ group_by_name_suffix=False):
+ return None
+
+ def AsDict(self):
+ d = super(TraceValue, self).AsDict()
+ d['file_id'] = self._file_handle.id
+ return d

Powered by Google App Engine
This is Rietveld 408576698