Changeset 2528

Timestamp:

11/14/08 15:07:26 (8 weeks ago)

Author:

ahodgkinson

Message:

digitalme: Firefox add-on logging changes suggested by Axel.

Location:

trunk/digitalme/launchers/firefox-addon/xul/chrome/content

Files:

• IdentitySelector.js (modified) (10 diffs)
• IdentitySelectorDiag.js (modified) (2 diffs)

trunk/digitalme/launchers/firefox-addon/xul/chrome/content/IdentitySelector.js

@@ -47 +47 @@
 var gLastFailedGetTokenDate = null;
 var gDisableStartTime = null;
-var gDebugMode = false;
 var gSelectorRegistry = [];
 
@@ -301 +300 @@
                 catch( e)
                 {
-                        if( gDebugMode)
-                        {
-                                IdentitySelectorDiag.reportError( "onDOMChanged", e);
-                        }
+                        IdentitySelectorDiag.debugReportError( "onDOMChanged", e);
                 }
         },
@@ -375 +371 @@
                 catch( e)
                 {
-                        if( gDebugMode)
-                        {
-                                IdentitySelectorDiag.reportError( "onProcessItems", e);
-                        }
+                        IdentitySelectorDiag.debugReportError( "onProcessItems", e);
                 }
         },
@@ -448 +441 @@
                 catch( e)
                 {
-                        if( gDebugMode)
-                        {
-                                IdentitySelectorDiag.reportError( "processICardItems", e);
-                        }
+                        IdentitySelectorDiag.debugReportError( "processICardItems", e);
                 }
         },
@@ -560 +550 @@
                 catch( e)
                 {
-                        if( gDebugMode)
-                        {
-                                IdentitySelectorDiag.reportError( "setParamsFromElem", e);
-                        }
+                        IdentitySelectorDiag.debugReportError( "setParamsFromElem", e);
                 }
         },
@@ -686 +673 @@
                 catch( e)
                 {
-                        if( gDebugMode)
-                        {
-                                IdentitySelectorDiag.reportError( "onFormSubmit", e);
-                        }
+                        IdentitySelectorDiag.debugReportError( "onFormSubmit", e);
                 }
         },
@@ -832 +816 @@
                 catch( e)
                 {
-                        if( gDebugMode)
-                        {
-                                IdentitySelectorDiag.reportError( "onDOMChanged", e);
-                        }
+                        IdentitySelectorDiag.debugReportError( "onDOMChanged", e);
                 }
         },
@@ -930 +911 @@
                 catch( e)
                 {
-                        if( gDebugMode)
-                        {
-                                IdentitySelectorDiag.reportError( "onICardObjectLoaded", e);
-                        }
+                        IdentitySelectorDiag.debugReportError( "onICardObjectLoaded", e);
                 }
                 
@@ -1012 +990 @@
                 catch( e)
                 {
-                        if( gDebugMode)
-                        {
-                                IdentitySelectorDiag.reportError( "onICardElementLoaded", e);
-                        }
+                        IdentitySelectorDiag.debugReportError( "onICardElementLoaded", e);
                 }
         },
@@ -1536 +1511 @@
                 catch( e)
                 {
-                        if( gDebugMode)
-                        {
-                                IdentitySelectorDiag.reportError( "onLocationChange", e);
-                        }
+                        IdentitySelectorDiag.debugReportError( "onLocationChange", e);
                 }
                 

trunk/digitalme/launchers/firefox-addon/xul/chrome/content/IdentitySelectorDiag.js

@@ -25 +25 @@
                         getService( Components.interfaces.nsIConsoleService);
 var gbLoggingEnabled = false;
+var gDebugMode = false;
 
 // **************************************************************************
@@ -64 +65 @@
                 }
         },
+        
+        // ***********************************************************************
+        // Method: debugReportError
+        // ***********************************************************************
+        debugReportError : function( location, description)
+        {
+                if( gDebugMode)
+                {
+                        alert( "IdentitySelector Error:" + location + ": " + description);
+                } 
+                else
+                {
+                        IdentitySelectorDiag.logMessage( location, "Error:" + description);
+                }
+        },      
         
         // ***********************************************************************