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

Unified Diff: lib/contentPolicy.js

Issue 9046035: Removed Do-Not-Track handling (Closed)
Patch Set: Created Dec. 18, 2012, 2:06 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 | lib/matcher.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: lib/contentPolicy.js
===================================================================
--- a/lib/contentPolicy.js
+++ b/lib/contentPolicy.js
@@ -473,35 +473,16 @@ let PolicyImplementation =
}
break;
}
case "http-on-modify-request":
{
if (!(subject instanceof Ci.nsIHttpChannel))
return;
- if (Prefs.enabled)
- {
- let match = defaultMatcher.matchesAny(subject.URI.spec, "DONOTTRACK", null, false);
- if (match && match instanceof BlockingFilter)
- {
- FilterStorage.increaseHitCount(match);
- subject.setRequestHeader("DNT", "1", false);
-
- // Bug 23845 - Some routers are broken and cannot handle DNT header
- // following Connection header. Make sure Connection header is last.
- try
- {
- let connection = subject.getRequestHeader("Connection");
- subject.setRequestHeader("Connection", null, false);
- subject.setRequestHeader("Connection", connection, false);
- } catch(e) {}
- }
- }
-
if (this.previousRequest && subject.URI == this.previousRequest[0] &&
subject instanceof Ci.nsIWritablePropertyBag)
{
// We just handled a content policy call for this request - associate
// the data with the channel so that we can find it in case of a redirect.
subject.setProperty("abpRequestType", this.previousRequest[1]);
this.previousRequest = null;
}
« no previous file with comments | « no previous file | lib/matcher.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld