Bug 1311116 - Use info level logging in Marionette debug builds; r?automatedtester draft
authorAndreas Tolfsen <ato@mozilla.com>
Thu, 10 Nov 2016 16:09:38 +0000
changeset 437160 1b863d4f2fb7fc2912300ed09dc60804f914059b
parent 437137 d2e33428e45718bbeff192e80fd3a75ad264210d
child 536591 05269be6d46223b2cb91ca75667f0a944e4e2631
push id35360
push userbmo:ato@mozilla.com
push dateThu, 10 Nov 2016 16:12:01 +0000
reviewersautomatedtester
bugs1311116
milestone52.0a1
Bug 1311116 - Use info level logging in Marionette debug builds; r?automatedtester Increase the default logging level for debug builds to `Log.Level.Info`. If we desire a finer log level for debug builds on try jobs, we can set the `marionette.logging` preference from the Marionette client in the future. MozReview-Commit-ID: 2vspGheBodq
testing/marionette/components/marionettecomponent.js
--- a/testing/marionette/components/marionettecomponent.js
+++ b/testing/marionette/components/marionettecomponent.js
@@ -62,19 +62,16 @@ MarionetteComponent.prototype.setupLogge
   let log = Log.repository.getLogger("Marionette");
   log.level = level;
   log.addAppender(new Log.DumpAppender());
   return log;
 };
 
 MarionetteComponent.prototype.determineLoggingLevel_ = function() {
   let level = Log.Level.Info;
-#ifdef DEBUG
-  level = Log.Level.Trace;
-#endif
 
   // marionette.logging pref can override default
   // with an entry from the Log.Level enum
   if (Preferences.has(LOG_PREF)) {
     let p = Preferences.get(LOG_PREF);
 
     switch (typeof p) {
       // Gecko >= 46