Bug 1054977 - Disable bug suggestions
authorEd Morley <emorley@mozilla.com>
Wed, 25 Feb 2015 13:09:39 +0000
changeset 1510 056344599c25
parent 1509 2cac016caba2
child 1511 ec1850f9e430
push id873
push useremorley@mozilla.com
push dateWed, 25 Feb 2015 14:28:56 +0000
bugs1054977
Bug 1054977 - Disable bug suggestions
php/inc/AnnotatedSummaryGenerator.php
--- a/php/inc/AnnotatedSummaryGenerator.php
+++ b/php/inc/AnnotatedSummaryGenerator.php
@@ -66,19 +66,21 @@ class AnnotatedSummaryGenerator implemen
                  "data-summary=\"$bug->summary\" " .
                  "data-signature=\"$this->logDescription\" " .
                  "data-logline=\"$line\" " .
                  "data-status=\"$bug->status $bug->resolution\"" .
            "></span>\n";
   }
 
   protected function processLine(&$annotatedLines, $line) {
-    $bugs = array();
     $annotatedLines[] = htmlspecialchars($line);
 
+    // Disable bug suggestions, now EOL is imminent.
+    return;
+
     // On windows, $line can have spurious whitespace/line-endings appended
     $trimmedLine = trim($line);
 
     // Our standard logging format uses the pipe symbol as a delimeter.
     $lineParts = preg_split("#\s\|\s#", $trimmedLine);
     if (count($lineParts) >= 3) {
       // Is in the "FAILURE-TYPE | testNameOrFilePath | message" type format.
       // testNameOrFilePath can either be a test filepath, or an arbitrary
@@ -101,16 +103,17 @@ class AnnotatedSummaryGenerator implemen
     // If the failure line was not in the pipe symbol delimited format or the search term 
     // will likely return too many (or irrelevant) results (eg: too short or matches terms
     // on the blacklist), then we fall back to searching for the entire failure line.
     if ((!isset($searchTerm) || !$this->isHelpfulSearchTerm($searchTerm)) &&
         $this->isHelpfulSearchTerm($trimmedLine)) {
       $searchTerm = $trimmedLine;
     }
 
+    $bugs = array();
     if (isset($searchTerm)) {
       $bugs = $this->getBugsForFailure($searchTerm);
     }
 
     // For crashes with no suggestions, search for the top frame as a fallback.
     // Catches shutdown crashes/anything else that is blacklisted, as well as GC
     // crashes that occur in too many tests to put all of them in bug summaries.
     if (empty($bugs) &&