Skip to content

Commit 31e64c1

Browse files
committed
Remove ContentPolicy implementation
Registering content policy in chrome.manifest is incompatible with e10s.
1 parent b4f8257 commit 31e64c1

2 files changed

Lines changed: 0 additions & 41 deletions

File tree

src/chrome.manifest

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,6 @@ component {32c165b4-fe5e-4964-9250-603c410631b4} components/https-everywhere.js
4141
contract @eff.org/https-everywhere;1 {32c165b4-fe5e-4964-9250-603c410631b4}
4242

4343
category profile-after-change HTTPSEverywhere @eff.org/https-everywhere;1
44-
category content-policy HTTPSEverywhere @eff.org/https-everywhere;1
4544

4645
overlay chrome://browser/content/browser.xul chrome://https-everywhere/content/toolbar_button.xul
4746
overlay chrome://navigator/content/navigator.xul chrome://https-everywhere/content/toolbar_button.xul

src/components/https-everywhere.js

Lines changed: 0 additions & 40 deletions
Original file line numberDiff line numberDiff line change
@@ -34,8 +34,6 @@ const Cr = Components.results;
3434
Cu.import("resource://gre/modules/Services.jsm");
3535
Cu.import("resource://gre/modules/FileUtils.jsm");
3636

37-
const CP_SHOULDPROCESS = 4;
38-
3937
const SERVICE_CTRID = "@eff.org/https-everywhere;1";
4038
const SERVICE_ID=Components.ID("{32c165b4-fe5e-4964-9250-603c410631b4}");
4139
const SERVICE_NAME = "Encrypts your communications with a number of major websites";
@@ -223,44 +221,6 @@ function HTTPSEverywhere() {
223221
}
224222

225223

226-
// nsIContentPolicy interface
227-
// we use numeric constants for performance sake:
228-
const TYPE_OTHER = 1;
229-
const TYPE_SCRIPT = 2;
230-
const TYPE_IMAGE = 3;
231-
const TYPE_STYLESHEET = 4;
232-
const TYPE_OBJECT = 5;
233-
const TYPE_DOCUMENT = 6;
234-
const TYPE_SUBDOCUMENT = 7;
235-
const TYPE_REFRESH = 8;
236-
const TYPE_XBL = 9;
237-
const TYPE_PING = 10;
238-
const TYPE_XMLHTTPREQUEST = 11;
239-
const TYPE_OBJECT_SUBREQUEST = 12;
240-
const TYPE_DTD = 13;
241-
const TYPE_FONT = 14;
242-
const TYPE_MEDIA = 15;
243-
// --------------
244-
// REJECT_SERVER = -3
245-
// ACCEPT = 1
246-
247-
248-
// Some of these types are known by arbitrary assertion at
249-
// https://bugzilla.mozilla.org/show_bug.cgi?id=677643#c47
250-
// TYPE_FONT was required to fix https://trac.torproject.org/projects/tor/ticket/4194
251-
// TYPE_SUBDOCUMENT was required to fix https://trac.torproject.org/projects/tor/ticket/4149
252-
// I have NO IDEA why JS won't let me use the constants above in defining this
253-
const shouldLoadTargets = {
254-
1 : true,
255-
3 : true,
256-
5 : true,
257-
12 : true,
258-
14 : true,
259-
7 : true
260-
};
261-
262-
263-
264224
/*
265225
In recent versions of Firefox and HTTPS Everywhere, the call stack for performing an HTTP -> HTTPS rewrite looks like this:
266226

0 commit comments

Comments
 (0)