Closed Bug 1386391 Opened 2 years ago Closed 2 years ago
_sessionhistory .html | Should have loaded a new document
59 bytes, text/x-review-board-request
Looks like this frequent intermittent failure started with the changes for bug 1385981.
I took at look at that test and the error message: - TEST-UNEXPECTED-FAIL | docshell/test/navigation/test_sessionhistory.html | Should have loaded a new document - testDone@docshell/test/navigation/file_fragment_handling_during_load.html:8:9 - setTimeout handler*test@docshell/test/navigation/file_fragment_handling_during_load.html:19:19 - setTimeout handler*onload@docshell/test/navigation/file_fragment_handling_during_load.html:1:1 This tests relies on a flaky timeout (predating our changes within Bug 1385981). I think because we are loading content from an external file instead of using a data: URI (see ) some of our old testing machines are not fast enough to load content before the timeout fires. Extending the timeout to 3000 should hopefully fix the issue.  https://hg.mozilla.org/mozilla-central/rev/46fcd660ce40#l1.22
Attachment #8895277 - Flags: review?(bugs)
I will be on PTO starting today. Henry is going to take over and drag this one over the finishing line. In case the patch doesn't fix the intermittent, Henry offered to take another look at the issue.
Assignee: nobody → hchang
Status: NEW → ASSIGNED
Priority: -- → P1
Whiteboard: [stockwell needswork:DOM] → [stockwell needswork:DOM][domsecurity-active]
Comment on attachment 8895277 [details] [diff] [review] bug_1386391_intermittent_sessionhistory.patch We can try this.
Attachment #8895277 - Flags: review?(bugs) → review+
The patch doesn't seem to work according to https://treeherder.mozilla.org/logviewer.html#?job_id=122512012&repo=try&lineNumber=63905 I will need to dig deeper into this issue.
hg error in cmd: hg push -r . -f try: pushing to ssh://hg.mozilla.org/try searching for changes remote: abort: abandoned transaction found! remote: (run 'hg recover' to clean up transaction) abort: stream ended unexpectedly (got 0 bytes, expected 4)
Comment on attachment 8895277 [details] [diff] [review] bug_1386391_intermittent_sessionhistory.patch This patch doesn't work.
Attachment #8895277 - Attachment is obsolete: true
According to https://treeherder.mozilla.org/#/jobs?repo=try&revision=5142782a73c73d124d91741d86e0be833c199e99 It seems my patch works well and the solution is fairly straightforward so I am confident to ask :smaug for a review.  https://reviewboard.mozilla.org/r/168112/
Comment on attachment 8896822 [details] Bug 1386391 - Poll to check if a document has loaded rather than a definite timeout. https://reviewboard.mozilla.org/r/168114/#review173388
Attachment #8896822 - Flags: review?(bugs) → review+
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/autoland/rev/2cca20fca12c Poll to check if a document has loaded rather than a definite timeout. r=smaug
You need to log in before you can comment on or make changes to this bug.