New toggle button on First-run page

chrome/content/ui/firstRun.js

 /*
  * This file is part of Adblock Plus <http://adblockplus.org/>,
  * Copyright (C) 2006-2013 Eyeo GmbH
  *
  * Adblock Plus is free software: you can redistribute it and/or modify
  * it under the terms of the GNU General Public License version 3 as
  * published by the Free Software Foundation.
  *
  * Adblock Plus is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
@@ skipping 253 matching lines @@
       UI.openFiltersDialog();
     else
     {
       backgroundPage.openOptions();
     }
   }
 
   function updateToggleButton(feature, isEnabled)
   {
     var button = E("toggle-" + feature);
-    if (isEnabled && button.classList.contains("off"))
+    if (isEnabled)
       button.classList.remove("off");
-    else if (!isEnabled && !button.classList.contains("off"))
+    else
       button.classList.add("off");

[Wladimir Palant, 2013/07/08 09:37:29]

classList.remove() and classList.add() already do classList.contains()
internally - the explicit checks really aren't necessary.

[Thomas Greiner, 2013/07/08 10:30:19]

Done.

   }
 
   document.addEventListener("DOMContentLoaded", onDOMLoaded, false);
 })();