Issue #580 - "Disable on this page only" should discard the final # from URLs

lib/ui.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 1486 matching lines @@
       let host = null;
       try
       {
         host = location.host.replace(/^www\./, "");
       } catch (e) {}
 
       if (host)
       {
         let ending = "|";
         location = location.clone();
-        if (location instanceof Ci.nsIURL && location.ref)
-          location.ref = "";
+        location.ref = "";

[Wladimir Palant, 2014/06/12 08:09:36]

Please don't remove |location instanceof Ci.nsIURL| check - it might be an
instance of nsIURI (e.g. data: URI), these don't have a hash part.

[saroyanm, 2014/06/12 14:20:40]

Done.

         if (location instanceof Ci.nsIURL && location.query)
         {
           location.query = "";
           ending = "?";
         }
 
         siteWhitelist = Filter.fromText("@@||" + host + "^$document");
         setChecked(prefix + "whitelistsite", siteWhitelist.subscriptions.length && !siteWhitelist.disabled);
         generateLabel(prefix + "whitelistsite", host);
         hideElement(prefix + "whitelistsite", false);
@@ skipping 431 matching lines @@
   ["abp-command-contribute", "command", UI.openContributePage.bind(UI)],
   ["abp-command-contribute-hide", "command", UI.hideContributeButton.bind(UI)]
 ];
 
 onShutdown.add(function()
 {
   for (let window in UI.applicationWindows)
     if (UI.isBottombarOpen(window))
       UI.toggleBottombar(window);
 });