Ticket #122: firebugConsole_rev2.patch

File firebugConsole_rev2.patch, 1.0 kB (added by sevenfurna..@gmail.com, 3 months ago)

another approach that doesn't depend on eval's second argument

  • components/greasemonkey.js

    old new  
    400400      if (!chromeWin.Firebug.Console.isEnabled(firebugContext)) { 
    401401        return null; 
    402402      } 
    403       var handler = eval("FirebugConsoleHandler", 
    404                          chromeWin.Firebug.Console.injector.attachConsole); 
    405       firebugConsole = new handler(firebugContext, unsafeContentWin); 
     403      var contentWin = new XPCNativeWrapper(unsafeContentWin); 
     404      var consoleElm = contentWin.document.getElementById("_firebugConsole"); 
     405      if (!consoleElm) { 
     406        chromeWin.Firebug.Console.injector.attachConsole(firebugContext, contentWin); 
     407      } 
     408      for (var i = 0; i < firebugContext.consoleHandler.length; i++) { 
     409        if (firebugContext.consoleHandler[i].window == contentWin) { 
     410          firebugConsole = firebugContext.consoleHandler[i].handler; 
     411          break; 
     412        } 
     413      } 
    406414    } 
    407415    return firebugConsole; 
    408416  }