Issue 1708 - Integrate first-run page changes in Chrome/Opera/Safari

options.js

 /*
  * This file is part of Adblock Plus <http://adblockplus.org/>,
  * Copyright (C) 2006-2014 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 76 matching lines @@
 }
 $(loadOptions);
 
 function onMessage(msg)
 {
   switch (msg.type)
   {
     case "add-subscription":
       startSubscriptionSelection(msg.title, msg.url);
       break;
-    default:

[Sebastian Noack, 2014/12/19 17:57:53]

While changing this code, mind changing the verbose switch-case to a simpler
if-statement? IMO there is no point to use switch-case to handle a single case.

[Wladimir Palant, 2014/12/19 19:38:42]

Done.

-      console.log("got unexpected message: " + msg.type);
   }
 };
 
 // Reloads the displayed subscriptions and filters
 function reloadFilters()
 {
   // Load user filter URLs
   var container = document.getElementById("filterLists");
   while (container.lastChild)
     container.removeChild(container.lastChild);
@@ skipping 538 matching lines @@
       links[i].href = arguments[i + 1];
       links[i].setAttribute("target", "_blank");
     }
     else if (typeof arguments[i + 1] == "function")
     {
       links[i].href = "javascript:void(0);";
       links[i].addEventListener("click", arguments[i + 1], false);
     }
   }
 }