Bug 1311116 - Use info level logging in Marionette debug builds. r=automatedtester, a=test-only
authorAndreas Tolfsen <ato@mozilla.com>
Thu, 10 Nov 2016 16:09:38 +0000
changeset 353586 c3f395000af75c977fd08b3de129bba1290af0f4
parent 353585 3828f20e9bfab38345ff4da40bdb8222f5dcc842
child 353587 320c0b7ee698dda1b0f7f75fbb00e02d82055270
push id6795
push userjlund@mozilla.com
push dateMon, 23 Jan 2017 14:19:46 +0000
treeherdermozilla-esr52@76101b503191 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersautomatedtester, test-only
bugs1311116
milestone52.0a2
Bug 1311116 - Use info level logging in Marionette debug builds. r=automatedtester, a=test-only 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/marionette.js
--- a/testing/marionette/components/marionette.js
+++ b/testing/marionette/components/marionette.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