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

Side by Side 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 unified diff | Download patch
OLDNEW
(Empty)
1 # Copyright 2014 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file.
4
5 import tempfile
6
7 from telemetry import value as value_module
8 from telemetry.util import file_handle
9
10
11 class TraceValue(value_module.Value):
12 def __init__(self, page, name, timeline_data, important=False,
13 description=None):
14 super(TraceValue, self).__init__(page, name, None, important, description)
15
16 tf = tempfile.NamedTemporaryFile()
17 self._file_handle = file_handle.FileHandle.FromTempFile(tf)
nduca 2014/09/16 20:56:24 swap line 17 and 18
18 timeline_data.Serialize(tf)
nduca 2014/09/16 20:56:24 tf = ... timelineD_ata.Serialize(tf) tf2 = Named
19
20 def GetAssociatedFileHandle(self):
21 return self._file_handle
22
23 def __repr__(self):
24 if self.page:
25 page_name = self.page.url
26 else:
27 page_name = None
28 return 'TraceValue(%s, %s)' % (page_name, self.name)
29
30 def GetBuildbotDataType(self, output_context):
31 return None
32
33 def GetBuildbotValue(self):
34 return None
35
36 def GetChartAndTraceNameForPerPageResult(self):
37 return None
38
39 def GetRepresentativeNumber(self):
40 return None
41
42 def GetRepresentativeString(self):
43 return None
44
45 @staticmethod
46 def GetJSONTypeName():
47 return 'trace'
48
49 @classmethod
50 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
51 assert len(values) > 0
52 return values[0]
53
54 @classmethod
55 def MergeLikeValuesFromDifferentPages(cls, values,
56 group_by_name_suffix=False):
57 return None
58
59 def AsDict(self):
60 d = super(TraceValue, self).AsDict()
61 d['file_id'] = self._file_handle.id
62
63 return d
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698