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

Unified Diff: dashboard/dashboard/pinpoint/models/quest/execution.py

Issue 3001163002: Pinpoint - Surface info from executions for display in UI. (Closed)
Patch Set: Rebase again. Created 3 years, 4 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: dashboard/dashboard/pinpoint/models/quest/execution.py
diff --git a/dashboard/dashboard/pinpoint/models/quest/execution.py b/dashboard/dashboard/pinpoint/models/quest/execution.py
index f7a89e4a73db21fab44d9a6383813ac5263a8a0a..3e978283efa01c3e311f870ac1de071982e14f71 100644
--- a/dashboard/dashboard/pinpoint/models/quest/execution.py
+++ b/dashboard/dashboard/pinpoint/models/quest/execution.py
@@ -65,6 +65,17 @@ class Execution(object):
assert self.completed
return self._result_arguments
+ def AsDict(self):
+ d = {
+ 'result_arguments': self.result_arguments if self.completed else {},
+ 'result_values': self.result_values if self.completed else None,
+ }
+ d.update(self._AsDict())
+ return d
+
+ def _AsDict(self):
+ raise NotImplementedError()
+
def Poll(self):
"""Update the Execution status."""
assert not self.completed
« no previous file with comments | « dashboard/dashboard/pinpoint/models/job.py ('k') | dashboard/dashboard/pinpoint/models/quest/find_isolate.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698