Bug 1354232 - Support skipping output lines in handlers, r=ato draft
authorJames Graham <james@hoppipolla.co.uk>
Thu, 24 May 2018 17:31:58 +0100
changeset 805838 9895cd155aa15b8fcbd3083fb6d477471fcfd565
parent 805837 aa11253963ca407aec3c1de18d28f05be2bba017
child 805839 9a8af95f81a63e6252b6fc7c8fb78faf8c565dab
push id112776
push userbmo:james@hoppipolla.co.uk
push dateFri, 08 Jun 2018 15:53:57 +0000
reviewersato
bugs1354232
milestone62.0a1
Bug 1354232 - Support skipping output lines in handlers, r=ato MozReview-Commit-ID: JxcPzSDt20V
testing/web-platform/tests/tools/wptrunner/wptrunner/browsers/firefox.py
--- a/testing/web-platform/tests/tools/wptrunner/wptrunner/browsers/firefox.py
+++ b/testing/web-platform/tests/tools/wptrunner/wptrunner/browsers/firefox.py
@@ -329,20 +329,21 @@ class FirefoxBrowser(Browser):
         """Write a line of output from the firefox process to the log"""
         if "GLib-GObject-CRITICAL" in line:
             return
         if line:
             data = line.decode("utf8", "replace")
             if self.stack_fixer:
                 data = self.stack_fixer(data)
             if self.lsan_handler:
-                self.lsan_handler.log(data)
-            self.logger.process_output(self.pid(),
-                                      data,
-                                      command=" ".join(self.runner.command))
+                data = self.lsan_handler.log(data)
+            if data is not None:
+                self.logger.process_output(self.pid(),
+                                           data,
+                                           command=" ".join(self.runner.command))
 
     def is_alive(self):
         if self.runner:
             return self.runner.is_running()
         return False
 
     def cleanup(self):
         self.stop()