Rewrite PurgeTrackerService.jsm logging to use console.createInstance
Categories
(Core :: Privacy: Anti-Tracking, task, P3)
Tracking
()
| Tracking | Status | |
|---|---|---|
| firefox76 | --- | fixed |
People
(Reporter: johannh, Assigned: aarushivij)
References
Details
Attachments
(1 file)
Like https://searchfox.org/mozilla-central/rev/c80fa7258c935223fe319c5345b58eae85d4c6ae/browser/modules/Sanitizer.jsm#37 which is a bit more idiomatic.
|
Assignee | |
Comment 1•4 years ago
|
||
Hello Joahnn
Can I work on this issue?
Do we need to change this function https://searchfox.org/mozilla-central/source/toolkit/components/antitracking/PurgeTrackerService.jsm#281
as the one mentioned in the above comment?
Thanks :)
Aarushi
|
Reporter | |
Comment 2•4 years ago
|
||
Hey, I noticed that Erica filed bug 1624906 so this bug may be invalid if we use our own custom logging mechanism. Erica, what do you think?
If this bug is still valid it would be a good bug to work on, yes.
|
||
Comment 3•4 years ago
•
|
||
We've been discussing adding permanent logging, but it looks like if it will happen it's further in the future, due to some complications. In that case, it is worth it to make this change.
Aarushi, thank you for volunteering to take this!
|
|
Assignee | |
Comment 4•4 years ago
|
||
Hey Erica, just want to make sure that Do we need to change this function https://searchfox.org/mozilla-central/source/toolkit/components/antitracking/PurgeTrackerService.jsm#281
as the one mentioned in the above comment (Like https://searchfox.org/mozilla-central/rev/c80fa7258c935223fe319c5345b58eae85d4c6ae/browser/modules/Sanitizer.jsm#37 which is a bit more idiomatic.)?
Thanks
Aarushi
|
|
Reporter | |
Comment 5•4 years ago
|
||
Yeah, that sounds about right :)
|
|
Assignee | |
Comment 6•4 years ago
|
||
Pushed by shindli@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/3e03712c2056 Rewrite PurgeTrackerService.jsm logging to use console.createInstance r=johannh
|
||
Comment 8•4 years ago
|
||
| bugherder | ||
Description
•