Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: options.js

Issue 29829615: Issue 6786 - Switch Safari 12+ users to content blocking API (Closed)
Patch Set: Simplify Safari version parsing logic Created July 13, 2018, 6:57 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | safari/contentBlocking.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: options.js
diff --git a/options.js b/options.js
index f1a3c1d817061ba58b3fd78ce6c65f10c1e280e2..ee45388669745f20dfa408d08451a4148ed1dcf1 100644
--- a/options.js
+++ b/options.js
@@ -17,6 +17,8 @@
"use strict";
+var majorApplicationVersion = parseInt(navigator.userAgent.match(/Version\/([\d]+)/)[1]);
+
/**
* Creates a wrapping function used to conveniently send a type of message.
*
@@ -158,11 +160,14 @@ function loadOptions()
// running Safari and both the legacy and content blocking APIs are
// available.
document.getElementById("safariContentBlockerContainer").hidden = !(
- features.safariContentBlocker &&
- typeof safari != "undefined" &&
- "canLoad" in safari.self.tab &&
- "onbeforeload" in Element.prototype
+ majorApplicationVersion >= 12 || (
+ features.safariContentBlocker &&
+ typeof safari != "undefined" &&
+ "canLoad" in safari.self.tab &&
+ "onbeforeload" in Element.prototype
+ )
);
+ document.getElementById("safariContentBlocker").disabled = majorApplicationVersion >= 12;
});
getPref("notifications_showui", function(notifications_showui)
{
« no previous file with comments | « no previous file | safari/contentBlocking.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld