Prepared adblockplus for Safari

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 35 matching lines @@
 
   function onDOMLoaded()
   {
     // Show warning if data corruption was detected
     if (typeof backgroundPage != "undefined" && backgroundPage.seenDataCorruption)
     {
       E("dataCorruptionWarning").removeAttribute("hidden");
       setLinks("dataCorruptionWarning", Utils.getDocLink("knownIssuesChrome_filterstorage"));
     }
 
+    // Show warning if Safari < 6
+    var info = require('info');

[Felix Dahlke, 2013/10/15 10:31:23]

Wladimir requested double quotes here, that's in line with the Mozilla
JavaScript-style which we mostly follow, and also consistent with the
surrounding code.

[Sebastian Noack, 2013/10/18 14:40:41]

I must have been overlooked that string. I'll fix that.

+        if (info.platform == "safari" && Services.vc.compare(info.platformVersion, "6.0") < 0)

[Felix Dahlke, 2013/10/15 10:31:23]

Looks like the indentation is off here, should be 4 spaces.

[Sebastian Noack, 2013/10/18 14:40:41]

I'll fix that.

+      E("legacySafariWarning").removeAttribute("hidden");
+
     // Set up URL
     setLinks("acceptableAdsExplanation", Utils.getDocLink("acceptable_ads_criteria"), openFilters);
 
     shade = E("shade");
     shade.addEventListener("mouseover", scrollPage, false);
     shade.addEventListener("mouseout", stopScroll, false);
 
     // Set up feature buttons linked to subscriptions
     featureSubscriptions.forEach(setToggleSubscriptionButton);
     var filterListener = function(action)
@@ skipping 191 matching lines @@
   {
     var button = E("toggle-" + feature);
     if (isEnabled)
       button.classList.remove("off");
     else
       button.classList.add("off");
   }
 
   document.addEventListener("DOMContentLoaded", onDOMLoaded, false);
 })();