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

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

Issue 545523002: [Telemetry] Add capability for values to reference external files. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Address staticmethod nit Created 6 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: 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..44e828c6c55279f33108055b14eeef91ab7771f9
--- /dev/null
+++ b/tools/telemetry/telemetry/value/trace.py
@@ -0,0 +1,63 @@
+# 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
+
+
+class TraceValue(value_module.Value):
+ def __init__(self, page, name, timeline_data, important=False,
+ description=None):
+ super(TraceValue, self).__init__(page, name, None, important, description)
+
+ tf = tempfile.NamedTemporaryFile()
+ self._file_handle = file_handle.FileHandle.FromTempFile(tf)
nduca 2014/09/16 20:56:24 swap line 17 and 18
+ timeline_data.Serialize(tf)
nduca 2014/09/16 20:56:24 tf = ... timelineD_ata.Serialize(tf) tf2 = Named
+
+ 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 GetChartAndTraceNameForPerPageResult(self):
+ return None
+
+ def GetRepresentativeNumber(self):
+ return None
+
+ def GetRepresentativeString(self):
+ return None
+
+ @staticmethod
+ def GetJSONTypeName():
+ return 'trace'
+
+ @classmethod
+ def MergeLikeValuesFromSamePage(cls, values):
nednguyen 2014/09/16 16:15:04 This should explode if these values are not the sa
nduca 2014/09/16 20:56:24 this is fine, for now. There is an even better sol
+ 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