Get rid of testlistjson query parameter.

Tools/TestResultServer/handlers/testfilehandler.py

 # Copyright (C) 2010 Google Inc. All rights reserved.
 #
 # Redistribution and use in source and binary forms, with or without
 # modification, are permitted provided that the following conditions are
 # met:
 #
 #     * Redistributions of source code must retain the above copyright
 # notice, this list of conditions and the following disclaimer.
 #     * Redistributions in binary form must reproduce the above
 # copyright notice, this list of conditions and the following disclaimer
@@ skipping 33 matching lines @@
 PARAM_MASTER = "master"
 PARAM_BUILDER = "builder"
 PARAM_BUILD_NUMBER = "buildnumber"
 PARAM_DIR = "dir"
 PARAM_FILE = "file"
 PARAM_NAME = "name"
 PARAM_BEFORE = "before"
 PARAM_NUM_FILES = "numfiles"
 PARAM_KEY = "key"
 PARAM_TEST_TYPE = "testtype"
-PARAM_TEST_LIST_JSON = "testlistjson"
 PARAM_CALLBACK = "callback"
 
 
 def _replace_jsonp_callback(json, callback_name):
     if callback_name and re.search(r"^[A-Za-z0-9_]+$", callback_name):
         if re.search(r"^[A-Za-z0-9_]+[(]", json):
             return re.sub(r"^[A-Za-z0-9_]+[(]", callback_name + "(", json)
         return callback_name + "(" + json + ")"
 
     return json
@@ skipping 83 matching lines @@
     def _get_file_content_from_key(self, key):
         file = db.get(key)
 
         if not file:
             logging.info("File not found, key %s.", key)
             return None
 
         file.load_data()
         return file.data, file.date
 
-    def _get_test_list_json(self, master, builder, test_type, build_number):
-        """Return json file with test name list only, do not include test
-           results and other non-test-data .
-
-        Args:
-            builder: builder name.
-            test_type: type of test results.
-        """
-
-        json, date = self._get_file_content(master, builder, test_type, build_number, "results.json")
-        if not json:
-            return None
-
-        return JsonResults.get_test_list(builder, json), date
-
     def _serve_json(self, json, modified_date):
         if json:
             if "If-Modified-Since" in self.request.headers:
                 old_date = self.request.headers["If-Modified-Since"]
                 if time.strptime(old_date, '%a, %d %b %Y %H:%M:%S %Z') == modified_date.utctimetuple():
                     self.response.set_status(304)
                     return
 
             # The appengine datetime objects are naive, so they lack a timezone.
             # In practice, appengine seems to use GMT.
             self.response.headers["Last-Modified"] = modified_date.strftime('%a, %d %b %Y %H:%M:%S') + ' GMT'
             self.response.headers["Content-Type"] = "application/json"
             self.response.headers["Access-Control-Allow-Origin"] = "*"
             self.response.out.write(json)
         else:
             self.error(404)
 
     def get(self):
         key = self.request.get(PARAM_KEY)
         master = self.request.get(PARAM_MASTER)
         builder = self.request.get(PARAM_BUILDER)
         test_type = self.request.get(PARAM_TEST_TYPE)
         build_number = self.request.get(PARAM_BUILD_NUMBER, default_value=None)
         name = self.request.get(PARAM_NAME)
         before = self.request.get(PARAM_BEFORE)
         num_files = self.request.get(PARAM_NUM_FILES)
-        test_list_json = self.request.get(PARAM_TEST_LIST_JSON)
         callback_name = self.request.get(PARAM_CALLBACK)
 
         logging.debug(
             "Getting files, master %s, builder: %s, test_type: %s, build_number: %s, name: %s, before: %s.",
             master, builder, test_type, build_number, name, before)
 
         if key:
             json, date = self._get_file_content_from_key(key)
-        elif test_list_json:
-            json, date = self._get_test_list_json(master, builder, test_type, build_number)
         elif num_files or not master or not builder or not test_type or (not build_number and not JsonResults.is_aggregate_file(name)) or not name:
             limit = int(num_files) if num_files else 100
             self._get_file_list(master, builder, test_type, build_number, name, before, limit, callback_name)
             return
         else:
             # FIXME: Stop using the old master name style after all files have been updated.
             master_data = master_config.getMaster(master)
             if not master_data:
                 master_data = master_config.getMasterByMasterName(master)
             if not master_data:
@@ skipping 96 matching lines @@
 
 class UploadForm(webapp2.RequestHandler):
     """Show a form so user can upload a file."""
 
     def get(self):
         template_values = {
             "upload_url": "/testfile/upload",
         }
         self.response.out.write(template.render("templates/uploadform.html",
                                                 template_values))