Use the logger element ID for updating the tooltip.

mozilla · Feb 29, 2012 · b5304cc · b5304cc
1 parent 608e48a
commit b5304cc
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 6 deletions.
diff --git a/extension/data/widget/widget.js b/extension/data/widget/widget.js
@@ -57,7 +57,7 @@ logger.onclick = function toggle() {
 };
 
 logger.onmouseover = function update_tooltip() {
-  self.port.emit("update_tooltip", "logger_" + logger.className);
+  self.port.emit("update_tooltip", logger.id);
 };
 
 logger.onmouseout = function update_tooltip() {

diff --git a/extension/lib/main.js b/extension/lib/main.js
@@ -89,11 +89,12 @@ exports.main = function (options, callbacks) {
 
   widget.port.on("update_tooltip", function (data) {
     switch(data) {
-      case "logger_enabled":
-        widget.tooltip = "MemChaser logging is currently enabled. Click to disable.";
-        break;
-      case "logger_disabled":
-        widget.tooltip = "MemChaser logging is currently disabled. Click to enable.";
+      case "logger":
+        if (logger.active) {
+          widget.tooltip = "MemChaser logging is currently enabled. Click to disable.";
+        } else {
+          widget.tooltip = "MemChaser logging is currently disabled. Click to enable.";
+        }
         break;
       default:
         widget.tooltip = "MemChaser";