Re-enable toolkit/components/reader/test/browser_readerMode_with_anchor.js for Fission
Categories
(Toolkit :: Reader Mode, task, P3)
Tracking
()
Tracking | Status | |
---|---|---|
firefox79 | --- | fixed |
People
(Reporter: hsivonen, Assigned: hsivonen)
References
Details
Attachments
(1 file)
(deleted),
text/x-phabricator-request
|
Details |
I'm going to mark toolkit/components/reader/test/browser_readerMode_with_anchor.js skipped in Fission. This bug is about re-enabling it.
Assignee | ||
Updated•5 years ago
|
Updated•5 years ago
|
Comment 2•5 years ago
|
||
Gijs, can you please help find an engineer to fix and re-enable this Reader Mode test for Fission?
The test is skipped for Fission here:
Comment 3•5 years ago
|
||
(In reply to Chris Peterson [:cpeterson] from comment #2)
Gijs, can you please help find an engineer to fix and re-enable this Reader Mode test for Fission?
The test is skipped for Fission here:
I'm not a manager so I can't really find engineers... and I don't really have time to dive into this myself right now. Henri disabled this, maybe he has at least some inkling of what is needed to re-enable this?
Assignee | ||
Comment 4•4 years ago
|
||
Assignee | ||
Comment 5•4 years ago
|
||
Assignee | ||
Comment 6•4 years ago
|
||
Try run looks good. I'll try again with more platforms just in case.
Comment 8•4 years ago
|
||
Hi Chris, I think we should move the Fission milestone to M6c per the dependency.
Comment 9•4 years ago
|
||
(In reply to Hsin-Yi Tsai [:hsinyi] from comment #8)
Hi Chris, I think we should move the Fission milestone to M6c per the dependency.
Sounds good since the dependency (bug 1634363) is an M6 bug.
Comment 10•4 years ago
|
||
In bug 1638027 Andrew hypothesises that the repeated creation/destruction of workers is to blame for the leaks in the trypush...
Comment 11•4 years ago
|
||
Comment 12•4 years ago
|
||
bugherder |
Description
•