Closed
Bug 1141618
Opened 10 years ago
Closed 10 years ago
Put reader worker logging behind a pref
Categories
(Toolkit :: Reader Mode, defect, P5)
Toolkit
Reader Mode
Tracking
()
RESOLVED
FIXED
mozilla39
People
(Reporter: Margaret, Assigned: Margaret)
References
Details
Attachments
(1 file)
|
1.60 KB,
patch
|
mossop
:
review+
|
Details | Diff | Splinter Review |
No description provided.
|
Assignee | |
Comment 1•10 years ago
|
||
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)
|
||
Updated•10 years ago
|
Attachment #8575524 -
Flags: review?(dtownsend) → review+
|
|
Assignee | |
Comment 2•10 years ago
|
||
|
||
Updated•10 years ago
|
Priority: -- → P5
|
||
Comment 3•10 years ago
|
||
Status: NEW → RESOLVED
Closed: 10 years ago
status-firefox39:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla39
|
||
Comment 5•10 years ago
|
||
status-firefox38:
--- → fixed
|
||
Comment 6•10 years ago
|
||
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.
Description
•