Bug 1090931 - Update logcat-filtering regex in reftest analyzer. r=dbaron
authorKartikaya Gupta <kgupta@mozilla.com>
Wed, 29 Oct 2014 21:09:10 -0400
changeset 213037 4cb91f2c872c566bd0250dcda5271612f53ac5b8
parent 213036 a67645f84f65308560ceced095b8fa2f8676ec76
child 213038 e1e7603c2455d33cbbf4660ab92af33c8b98cad8
push id27738
push usercbook@mozilla.com
push dateThu, 30 Oct 2014 13:46:07 +0000
treeherdermozilla-central@1aa1b23d799e [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersdbaron
bugs1090931
milestone36.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1090931 - Update logcat-filtering regex in reftest analyzer. r=dbaron The logcat format used by tbpl jobs in some (maybe all) cases now has a timestamp and other decorations at the beginning of the line. The regex that was previously added to filter out reftest failure lines duplicated in logcat no longer matches the lines correctly; this makes the regex more generic so that the filtering works again.
layout/tools/reftest/reftest-analyzer.xhtml
--- a/layout/tools/reftest/reftest-analyzer.xhtml
+++ b/layout/tools/reftest/reftest-analyzer.xhtml
@@ -215,17 +215,17 @@ function log_pasted() {
 var gTestItems;
 
 function process_log(contents) {
   var lines = contents.split(/[\r\n]+/);
   gTestItems = [];
   for (var j in lines) {
     var line = lines[j];
     // Ignore duplicated output in logcat.
-    if (line.match(/^I\/Gecko/))
+    if (line.match(/I\/Gecko.*?REFTEST/))
       continue;
     var match = line.match(/^.*?REFTEST (.*)$/);
     if (!match)
       continue;
     line = match[1];
     match = line.match(/^(TEST-PASS|TEST-UNEXPECTED-PASS|TEST-KNOWN-FAIL|TEST-UNEXPECTED-FAIL|TEST-DEBUG-INFO)(\(EXPECTED RANDOM\)|) \| ([^\|]+) \|(.*)/);
     if (match) {
       var state = match[1];