Changeset 8031

Timestamp:

09/17/08 19:39:24 (3 months ago)

Author:

tschaub

Message:

We were breaking the firebug console in Firefox 3. This change makes things less efficient for browsers without Firebug (we loop through every script checking for firebug lite), but it fixes the console with OL and Firefox 3. If you open up the console in Firefox 3 and type 'window.console' you get an error about too much recursion (nothing to do with OL). So, we avoid touching window.console unless firebug lite has defined it. r=euzuro (see #1594)

Files:

• trunk/openlayers/lib/OpenLayers/Console.js (modified) (1 diff)

trunk/openlayers/lib/OpenLayers/Console.js

@@ -234 +234 @@
-     * OpenLayers.Console calls to the console object.
-     */
-if(window.console) {
-    var scripts = document.getElementsByTagName("script");
-for(var i=0, len=scripts.length; i<len; ++i
-scripts[i].src.indexOf("firebug.js") != -1
+var scripts = document.getElementsByTagName("script");
+for(var i=0, len=scripts.length; i<len; ++i) {
+if(scripts[i].src.indexOf("firebug.js") != -1
+console
-                OpenLayers.Util.extend(OpenLayers.Console, console);
-                break;