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

Unified Diff: Tools/TestResultServer/handlers/testfilehandler.py

Issue 459243004: Fix 500 in test results server when querying testlistjson (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 6 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
« no previous file with comments | « no previous file | Tools/TestResultServer/handlers/testfilehandler_test.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Tools/TestResultServer/handlers/testfilehandler.py
diff --git a/Tools/TestResultServer/handlers/testfilehandler.py b/Tools/TestResultServer/handlers/testfilehandler.py
index 6a0822de3c7afd2a33488425edd01d409a0a1eee..301730f5da974db12208eefba1b6c6a66095ed87 100644
--- a/Tools/TestResultServer/handlers/testfilehandler.py
+++ b/Tools/TestResultServer/handlers/testfilehandler.py
@@ -207,7 +207,7 @@ class GetFile(webapp2.RequestHandler):
if json is None:
json, date = self._get_file_content(master_data['name'], builder, test_type, build_number, name)
- if test_list_json:
+ if json and test_list_json:
json = JsonResults.get_test_list(builder, json)
if json:
« no previous file with comments | « no previous file | Tools/TestResultServer/handlers/testfilehandler_test.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698