Re-enable devtools/client/debugger/test/mochitest/browser_dbg-bfcache.js test for Fission
Categories
(DevTools :: Debugger, defect, P3)
Tracking
(Fission Milestone:MVP)
Fission Milestone | MVP |
People
(Reporter: cpeterson, Unassigned)
References
(Blocks 2 open bugs)
Details
(Whiteboard: dt-fission-future, fission-bfcache)
Attachments
(1 obsolete file)
This devtools/client/debugger/test/mochitest/browser_dbg-bfcache.js test is currently skipped for Fission (by bug 1665165):
[browser_dbg-bfcache.js]
skip-if = fission # bug 1665165
Is this test still failing with Fission?
Does fixing and re-enabling this test need to block shipping Fission MVP?
Reporter | ||
Updated•4 years ago
|
Updated•4 years ago
|
Reporter | ||
Comment 1•4 years ago
|
||
Tracking DevTools Fission bugs for Fission M7 (blocking Beta experiment).
Comment 2•4 years ago
|
||
Seems to work locally when enabling fission? Let's check on try: https://treeherder.mozilla.org/jobs?repo=try&revision=48248c5936c2a0c5b5a1e9c11f711c8d0fff0b65
Comment 3•4 years ago
|
||
Updated•4 years ago
|
Comment 4•4 years ago
|
||
smaug: quick sanity check, this test was disabled as part of bfcache work, in bug 1665165.
You're still working on bfcache, so I'm not sure if we can re-enable the tests (it seems to pass fine with fission?) or if we should wait?
Comment 5•4 years ago
|
||
bfcache work has started quite recently, so we can't enable the test yet.
Also, most of the bfcache tests will need some tweaking to work with the new implementation.
Comment 6•4 years ago
|
||
Will un-assign and remove the patch for now.
Updated•4 years ago
|
Comment 7•4 years ago
|
||
smaug: Thanks for the info. Is there a bug we can block this on?
Comment 9•4 years ago
|
||
This is already blocking fission-bfcache
I saw that but my question was the other way around :)
We want to know when we can start working on this bug.
Right now it's in our backlog for fission, but not actionable.
Otherwise we just remove this from the devtools backlog, feels like it can't be addressed & triaged by our team.
Comment 10•4 years ago
|
||
Removing from devtools backlog does sound reasonable at this point, given that the test itself is rather simple.
Updated•4 years ago
|
Updated•4 years ago
|
Comment 12•4 years ago
|
||
Honza said this is not part of dt-fission-m3-mvp and is not considered Fission M7 blocker.
Reporter | ||
Comment 13•4 years ago
|
||
dt-fission-future don't need to block Fission MVP.
Comment 14•4 years ago
|
||
Fission Future should be used for stuff that aren't issues for Fission. So bringing this back into MVP.
Reporter | ||
Updated•4 years ago
|
Reporter | ||
Comment 15•4 years ago
|
||
dt-fission-future
bugs don't need to block Fission MVP, so I'm moving them from Fission Milestone MVP to Future.
Comment 16•4 years ago
|
||
I'm most likely going to fix and renable this test in bug 1694651, with https://phabricator.services.mozilla.com/D110557
Comment 17•3 years ago
|
||
Got re-enabled in bug 1694651.
Description
•