Bug 942154 - Use normal console logging instead of debug(). r=Standard8

Differential Revision: https://phabricator.services.mozilla.com/D176322

UltraBlame original commit: 07112703f3aae05cb75a4628eeefb4a3948fb0d8

Author: marco-c

browser/components/sessionstore/ContentSessionStore.sys.mjs

@@ -8,10 +8,6 @@ import {
   setTimeoutWithTarget,
 } from "resource://gre/modules/Timer.sys.mjs";
 
-function debug(msg) {
-  Services.console.logStringMessage("SessionStoreContent: " + msg);
-}
-
 const lazy = {};
 
 ChromeUtils.defineESModuleGetters(lazy, {
@@ -372,7 +368,7 @@ class MessageQueue extends Handler {
           );
           break;
         default:
-          debug("received unknown message '" + data + "'");
+          console.error("received unknown message '" + data + "'");
           break;
       }
     }
@@ -570,7 +566,7 @@ export class ContentSessionStore {
         this.mm.docShell.persistLayoutHistoryState();
         break;
       default:
-        debug("received unknown message '" + name + "'");
+        console.error("received unknown message '" + name + "'");
         break;
     }
   }

toolkit/modules/sessionstore/SessionHistory.sys.mjs

@@ -8,10 +8,6 @@ ChromeUtils.defineESModuleGetters(lazy, {
   E10SUtils: "resource://gre/modules/E10SUtils.sys.mjs",
 });
 
-function debug(msg) {
-  Services.console.logStringMessage("SessionHistory: " + msg);
-}
-
 /**
  * The external API exported by this module.
  */
@@ -564,7 +560,7 @@ var SessionHistoryInternal = {
         // and this ensures we always have a principal returned from this function.
         // We must always have a triggering principal for a load to work.
         // A null principal won't always work however is safe to use.
-        debug(
+        console.warn(
           "Couldn't deserialize the triggeringPrincipal, falling back to NullPrincipal"
         );
         return Services.scriptSecurityManager.createNullPrincipal({});