Closed Bug 1141618 Opened 5 years ago Closed 5 years ago

Put reader worker logging behind a pref

Categories

(Toolkit :: Reader Mode, defect, P5)

defect

Tracking

()

RESOLVED FIXED
mozilla39
Tracking Status
firefox38 --- fixed
firefox39 --- fixed

People

(Reporter: Margaret, Assigned: Margaret)

References

Details

Attachments

(1 file)

No description provided.
I found that I can't read prefs in this worker thread, since importing Services.jsm appears to fail because Components in undefined.

We don't have an immediate need for this logging, and this was out of the scope of what was necessary for bug 1140172, so let's just only make this a thing for local builds for now.
Attachment #8575524 - Flags: review?(dtownsend)
Attachment #8575524 - Flags: review?(dtownsend) → review+
Priority: -- → P5
https://hg.mozilla.org/mozilla-central/rev/2e4c0d785052
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla39
Duplicate of this bug: 1141803
Setting qe-verify- for this bug. Margaret, if you think manual QA verification is in fact required here, please flip the flag.
Flags: qe-verify-
You need to log in before you can comment on or make changes to this bug.