Closed Bug 1724149 Opened 4 months ago Closed 3 months ago

Fix test_interrupted_reflow.html on Fis+xorig runs

Categories

(Core :: Panning and Zooming, defect)

defect

Tracking

()

RESOLVED FIXED
93 Branch
Fission Milestone Future
Tracking Status
firefox92 --- wontfix
firefox93 --- fixed

People

(Reporter: hiro, Assigned: hiro)

References

Details

Attachments

(1 file)

Currently it's marked as fails-if=xorigin, not skip-if since it fails on finding the root content document scroller so that we can't collect scroll offsets properly.

Just opening a new tab and running the test in the new tab would solve the issue, but I am not 100% sure it's okay. IIUC the test purpose, it was for an issue on restoring scroll position (bug 1292781)? And restoring scroll position should happen only on the top level content root scroller I think, so it seems to be okay?

Hmm, the test seems to be written assuming it is the root content document? So I would think running it in a new tab would be a good idea.

So for clarity....does the test need to be tweaked, rewritten, or some other action?

I will rewrite the test to open a new tab instead of running the test in an iframe.

Assignee: nobody → hikezoe.birchill
Status: NEW → ASSIGNED

Setting Fission Milestone = Future because this test fix doesn't need to block shipping Fission MVP, IIUC.

Fission Milestone: --- → Future
Pushed by hikezoe.birchill@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/2623e1197fd1
Run the test in test_interrupted_reflow.html in a new tab. r=tnikkel
Status: ASSIGNED → RESOLVED
Closed: 3 months ago
Resolution: --- → FIXED
Target Milestone: --- → 93 Branch

Setting status-firefox92=wontfix because I assume we don't need to uplift this test fix to Beta 92.

You need to log in before you can comment on or make changes to this bug.