Issue jbolster#7 - Fixed it so multiple context menus aren't added (a…

…lways removeAll() before setting).
jsfiddle · Feb 25, 2011 · 6303542 · 6303542
1 parent 3e4848d
commit 6303542
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/Extension/Html/background.html b/Extension/Html/background.html
@@ -19,6 +19,8 @@
 	});
 
 	function enableContextMenu(enable){
+		chrome.contextMenus.removeAll();
+
 		if(enable){
 			chrome.contextMenus.create({
 				"title": "Stack Fiddle", 
@@ -38,8 +40,6 @@
 				"onclick" : function(info,tab){
 					chrome.tabs.create({ url:"/Html/options.html", index: (tab.index + 1) });
 			}});
-		}else{
-			chrome.contextMenus.removeAll();
 		}
 	}