Refactor prior to adding task-page

appengine/swarming/elements/res/imp/tasklist/task-list.html

 <!--
   Copyright 2016 The LUCI Authors. All rights reserved.
   Use of this source code is governed under the Apache License, Version 2.0
   that can be found in the LICENSE file.
 
   This in an HTML Import-able file that contains the definition
   of the following elements:
 
     <task-list>
 
@@ skipping 250 matching lines @@
       "user": "Requesting User",
     };
 
     // Given a time attribute like "abandoned_ts", sortableTime returns a function
     // that compares the tasks based on the attribute.  This is used for sorting.
     function sortableTime(attr) {
       // sort times based on the string they come with, formatted like
       // "2016-08-16T13:12:40.606300" which sorts correctly.  Locale time
       // (used in the columns), does not.
       return function(dir, a, b) {
-        var aCol = this._attribute(a, attr)[0];
-        var bCol = this._attribute(b, attr)[0];
+        var aCol = this._attribute(a, attr, "0")[0];
+        var bCol = this._attribute(b, attr, "0")[0];
 
         return dir * (aCol - bCol);
       }
     }
     var specialSort = {
       abandoned_ts: sortableTime("abandoned_ts"),
       completed_ts: sortableTime("completed_ts"),
       created_ts: sortableTime("created_ts"),
       duration: sortableTime("duration"),
       modified_ts: sortableTime("modified_ts"),
@@ skipping 80 matching lines @@
         this.$.prompt.open();
       },
 
       _tag: function(task, col) {
         if (!task || !task.tagMap) {
           return undefined;
         }
         return task.tagMap[col];
       },
 
-      _taskLink: function(taskId) {
-        if (!taskId) {
-          return undefined;
-        }
-        // TODO(kjlubick) Make this point to /newui/ when appropriate.
-        return "/user/task/"+taskId;
-      },
-
       _taskClass: function(task) {
         var state = this._column("state", task);
         if (state === "CANCELED" ||state === "TIMED_OUT" || state === "EXPIRED") {
           return "exception";
         }
         if (state === "BOT_DIED") {
           return "died";
         }
         if (state === "COMPLETED (FAILURE)") {
           return "failed";
         }
         if (state === "RUNNING" || state === "PENDING") {
           return "pending";
         }
         return "";
       }
 
     });
   })();
   </script>
 </dom-module>