Lazily report anti-tracking warnings to console

RESOLVED FIXED in Firefox 66

Status

()

enhancement
RESOLVED FIXED
5 months ago
5 months ago

People

(Reporter: Ehsan, Assigned: Ehsan)

Tracking

unspecified
Firefox 66
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox66 fixed)

Details

Attachments

(1 attachment)

Assignee

Description

5 months ago

See this profile: http://bit.ly/2FULtlL

We can see that ReportToConsole can be expensive, for example it can incur a call to NS_NewURI, which would cause us to parse a URI from APIs like document.cookie (which can be called quite frequently e.g. from loops inside web pages).

We need to defer these calls to a time when we don't have something better to do, rather than when we have active web page JS code to run!

Comment 2

5 months ago
Pushed by eakhgari@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/efd6f48633c2
Report anti-tracking warnings to console from the idle event queue; r=baku

Comment 3

5 months ago
bugherder
Status: NEW → RESOLVED
Closed: 5 months ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 66
Assignee

Comment 4

5 months ago

Oops, when rebasing this I accidentally pushed the changes for bug 1493361 which I had in my local tree :-(

Comment 5

5 months ago
Pushed by eakhgari@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/c418bc9e3fce
Part 2: Remove the changes for bug 1493361 which accidentally crept in here while rebasing
You need to log in before you can comment on or make changes to this bug.