Prepared adblockplus for Safari

chrome/content/ui/i18n.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
  * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
  * GNU General Public License for more details.
  *
  * You should have received a copy of the GNU General Public License
  * along with Adblock Plus.  If not, see <http://www.gnu.org/licenses/>.
  */
 
-var i18n;
-if (typeof chrome != "undefined")
-{
-  i18n = chrome.i18n;
-}
-else
+var i18n = (window.ext && ext.i18n) || (window.chrome && chrome.i18n);

[Felix Dahlke, 2013/09/06 08:08:24]

Why not make ext.i18n point to chrome.i18n in ext.js for Chrome? That way we'd
have all the port specific logic in ext.js, and only differentiate between
Firefox and the ports here.

[Sebastian Noack, 2013/09/06 09:05:37]

That's what I do. The only reason I check for both here is compatibility. So we
can merge this changes before merging my changes on the adblockpluschrome
repository.

+if (!i18n)
 {
   // Using Firefox' approach on i18n instead
 
   // Randomize URI to work around bug 719376
   var pageName = location.pathname.replace(/.*\//, '').replace(/\..*?$/, '');
   var stringBundle = Services.strings.createBundle("chrome://adblockplus/locale/" + pageName +
     ".properties?" + Math.random());
 
   function getI18nMessage(key)
   {
@@ skipping 97 matching lines @@
 
   var now = new Date();
   if (d.toDateString() == now.toDateString())
     return [timeString];
   else
     return [timeString, d.toLocaleDateString()];
 }
 
 // Fill in the strings as soon as possible
 window.addEventListener("DOMContentLoaded", loadI18nStrings, true);