Added log levels

Author: gsingh93

src/Log.ts

@@ -0,0 +1,28 @@
+module Log {
+    var LOG_LEVEL: LogLevel = LogLevel.Error;
+    export enum LogLevel {
+        Error, Warning, Info, Debug
+    }
+
+    export function error(message: any) {
+        log(LogLevel.Error, message);
+    }
+
+    export function warning(message: any) {
+        log(LogLevel.Warning, message);
+    }
+
+    export function info(message: any) {
+        log(LogLevel.Info, message);
+    }
+
+    export function debug(message: any) {
+        log(LogLevel.Debug, message);
+    }
+
+    function log(level: LogLevel, message: any) {
+        if (level <= LOG_LEVEL) {
+            console.log(message);
+        }
+    }
+}

src/Utils.ts

@@ -1,13 +1,6 @@
 /// <reference path="d.ts/DefinitelyTyped/chrome/chrome.d.ts"/>
 
 module Utils {
-    var logging = false;
-    export function log(message: any) {
-        if (logging) {
-            console.log(message);
-        }
-    }
-
     export interface ActiveTabCallback {
         (tab: chrome.tabs.Tab) : void;
     }

src/background/KeyboardHandler.ts

@@ -1,14 +1,15 @@
 /// <reference path="../d.ts/DefinitelyTyped/chrome/chrome.d.ts"/>
 /// <reference path="../Utils.ts"/>
+/// <reference path="../Log.ts"/>
 /// <reference path="TabStateManager.ts"/>
 
 module KeyboardHandler {
     var lastCalled: number = 0;
 
     export function init(tabStates: TabStateManager) {
         chrome.commands.onCommand.addListener(function (command: string) {
-            Utils.log("Received command " + command);
-            Utils.withActiveTab(function (tab: chrome.tabs.Tab) {
+            Log.info("Received command " + command);
+            Utils.withActiveTab(function(tab: chrome.tabs.Tab) {
                 var id = tab.id;
 
                 // The time hack is to get around this function being called twice when

src/background/TabStateManager.ts

@@ -23,11 +23,11 @@ class TabStateManager {
     public set(tabId: number, propName: string, propVal: any);
     public set(tabId: number, stateOrPropName: any, propVal?: any) {
         if (typeof propVal === "undefined") {
-            Utils.log("Set " + tabId + " to:");
-            Utils.log(stateOrPropName);
+            Log.debug("Set " + tabId + " to:");
+            Log.debug(stateOrPropName);
             this.tabStates[tabId] = stateOrPropName;
         } else {
-            Utils.log("Set " + stateOrPropName + " for " + tabId + " to " + propVal);
+            Log.debug("Set " + stateOrPropName + " for " + tabId + " to " + propVal);
             this.tabStates[tabId][stateOrPropName] = propVal;
         }
     }

src/popup/popup.ts

@@ -1,6 +1,7 @@
 /// <reference path="../d.ts/DefinitelyTyped/chrome/chrome.d.ts"/>
 /// <reference path="../bg-interface.ts"/>
 /// <reference path="../Utils.ts"/>
+/// <reference path="../Log.ts"/>
 
 module Popup {
     var prevButton  = document.getElementById("prev");
@@ -16,10 +17,9 @@ module Popup {
         // may be a few edge cases where we need to initialize it ourselves,
         // like if we enable/reload the extension
         if (!tabStates.exists(id)) {
-            Utils.log("ID doesn't exist. Initializing entry.")
+            Log.warning("ID doesn't exist. Initializing entry.")
             tabStates.resetState(id);
             var tabState = tabStates.get(id);
-            Utils.log(tabState);
         }
 
         addListeners(id, tabStates);
@@ -60,7 +60,7 @@ module Popup {
 
         var queryInputKeyDown = function(event) {
             if (event.keyCode == 13) {
-                Utils.log("Enter pressed");
+                Log.info("Enter pressed");
                 search(id, tabStates);
             }
         }
@@ -80,7 +80,7 @@ module Popup {
         }
 
         var checkboxClick = function() {
-            Utils.log("Set checkbox state to " + caseInsensitiveCheckbox.checked);
+            Log.info("Set checkbox state to " + caseInsensitiveCheckbox.checked);
             tabStates.set(id, "caseInsensitive", caseInsensitiveCheckbox.checked);
         }
 
@@ -109,7 +109,7 @@ module Popup {
                                     });
             setSearching(tabId, true, tabStates);
         } else {
-            Utils.log("Invalid regex");
+            Log.info("Invalid regex");
             queryInput.className = 'invalid';
         }
     }