Sort suites in suite table based on number of failed test cases.

build/android/pylib/results/presentation/javascript/main_html.js

 // Copyright 2017 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 function getArguments() {
   // Returns the URL arguments as a dictionary.
   args = {}
   var s = location.search;
   if (s) {
     var vals = s.substring(1).split('&');
@@ skipping 70 matching lines @@
 }
 
 function sortByColumn(head) {
   var table = head.parentNode.parentNode.parentNode;
   var rowBlocks = Array.prototype.slice.call(
       table.getElementsByTagName('tbody'));
 
   // Determine whether to asc or desc and set arrows.
   var headers = head.parentNode.getElementsByTagName('th');
   var headIndex = Array.prototype.slice.call(headers).indexOf(head);
-  var asc = 1;
+  var asc = -1;
   for (var i = 0; i < headers.length; i++) {
     if (headers[i].dataset.ascSorted != 0) {
       if (headers[i].dataset.ascSorted == 1) {
           headers[i].getElementsByClassName('up')[0]
               .style.display = 'none';
       } else {
         headers[i].getElementsByClassName('down')[0]
             .style.display = 'none';
       }
       if (headers[i] == head) {
@@ skipping 78 matching lines @@
       var bvalue = ((btotal_minus_skipped == 0) ? -1 :
           (b_rows.length - b_success_or_skipped) / btotal_minus_skipped);
     }
     return asc * (avalue - bvalue);
   });
 
   for (var i = 0; i < rowBlocks.length; i++) {
     table.appendChild(rowBlocks[i]);
   }
 }
+
+function sortSuiteTableByFailedTestCases() {
+  sortByColumn(document.getElementById('number_fail_tests'));

[the real yoland, 2017/05/12 19:09:43]

potential future improvements: there are a lot of string values in these js/html
files. Since the result detail page are all generated through jinja on the
python side, I think it might be good to refactor and have python control over
what these string values are and send them in through jinja

+}