Closed Bug 1522370 Opened 5 years ago Closed 5 years ago

Lazily report anti-tracking warnings to console

Categories

(Firefox :: Protections UI, enhancement)

enhancement
Not set
normal

Tracking

()

RESOLVED FIXED
Firefox 66
Tracking Status
firefox66 --- fixed

People

(Reporter: ehsan.akhgari, Assigned: ehsan.akhgari)

References

Details

Attachments

(1 file)

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!

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
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 66

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

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.