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

Unified Diff: include.preload.js

Issue 29363445: Issue 4577 - Remove Safari support (Closed)
Patch Set: Created Nov. 17, 2016, 11:36 a.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 | « dependencies ('k') | lib/icon.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: include.preload.js
diff --git a/include.preload.js b/include.preload.js
index 0fdf8327bcd596763b0d3f0cfe5610ef919b699b..30acfab91121d3f8f8cde6ec1d3cc1c4311acc5f 100644
--- a/include.preload.js
+++ b/include.preload.js
@@ -345,16 +345,13 @@ function runInPageContext(fn, arg)
document.documentElement.removeChild(script);
}
-// Neither Chrome[1] nor Safari allow us to intercept WebSockets, and therefore
+// Chrome doesn't allow us to intercept WebSockets[1], and therefore
// some ad networks are misusing them as a way to serve adverts and circumvent
// us. As a workaround we wrap WebSocket, preventing blocked WebSocket
// connections from being opened.
// [1] - https://bugs.chromium.org/p/chromium/issues/detail?id=129353
function wrapWebSocket()
{
- if (typeof WebSocket == "undefined")
- return;
-
var eventName = "abpws-" + Math.random().toString(36).substr(2);
document.addEventListener(eventName, function(event)
« no previous file with comments | « dependencies ('k') | lib/icon.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld