Intermittent test_navigation.py TestNavigate.test_type_to_remote_tab | TimeoutException: Timed out after 5.0 seconds with message: 'http://127.0.0.1:49326/test.html' hasn't been loaded

RESOLVED FIXED in Firefox 53

Status

Testing
Marionette
RESOLVED FIXED
8 months ago
3 months ago

People

(Reporter: Treeherder Bug Filer, Assigned: whimboo)

Tracking

({intermittent-failure})

Version 3
mozilla55
intermittent-failure
Points:
---

Firefox Tracking Flags

(firefox-esr52 wontfix, firefox53 fixed, firefox54 fixed, firefox55 fixed)

Details

MozReview Requests

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(1 attachment)

(Reporter)

Description

8 months ago
treeherder
Filed by: wkocher [at] mozilla.com

https://treeherder.mozilla.org/logviewer.html#?job_id=89240177&repo=autoland

https://archive.mozilla.org/pub/firefox/tinderbox-builds/autoland-win64-debug/1491495212/autoland_win8_64-debug_test-marionette-e10s-bm119-tests1-windows-build401.txt.gz
This is a debug build and having a timeout of 5s for a page to be loaded seems wrong. Instead it should use `self.marionette.timeout.page_load`.

>        Wait(self.marionette).until(
>            lambda mn: mn.get_url() == self.test_page_remote,
>            message="'{}' hasn't been loaded".format(self.test_page_remote))

The same applies to the test `test_type_to_non_remote_tab`, and maybe others.
Assignee: nobody → hskupin
Status: NEW → ASSIGNED
Blocks: 1303942
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)

Comment 4

8 months ago
mozreview-review
Comment on attachment 8855673 [details]
Bug 1354310 - Ensure to use page load timeout for Wait().until() in unit tests.

https://reviewboard.mozilla.org/r/127538/#review130352
Attachment #8855673 - Flags: review?(ato) → review+

Comment 5

8 months ago
Pushed by hskupin@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/ef76a3b0eb7d
Ensure to use page load timeout for Wait().until() in unit tests. r=ato
https://hg.mozilla.org/mozilla-central/rev/ef76a3b0eb7d
Status: ASSIGNED → RESOLVED
Last Resolved: 8 months ago
status-firefox55: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
Please uplift this test-only patch to aurora and beta. Thank you.
Whiteboard: [checkin-needed-aurora][checkin-needed-beta]

Comment 8

8 months ago
bugherderuplift
https://hg.mozilla.org/releases/mozilla-aurora/rev/0d8df2fe95cc
status-firefox54: --- → fixed
Whiteboard: [checkin-needed-aurora][checkin-needed-beta] → [checkin-needed-beta]

Comment 9

8 months ago
bugherderuplift
https://hg.mozilla.org/releases/mozilla-beta/rev/0fe6857edaff
status-firefox53: --- → fixed
Whiteboard: [checkin-needed-beta]
This'll need rebasing if you want to uplift to esr52.
Flags: needinfo?(hskupin)
If it would be a (near) perma-fail a backport patch would make sense. But as of now it doesn't warrant the work. So I will mark esr52 as wontfix. We can re-evaluate if it becomes a problem in the future.
status-firefox-esr52: --- → wontfix
Flags: needinfo?(hskupin)
Comment hidden (obsolete)

Comment 13

3 months ago
1 failures in 939 pushes (0.001 failures/push) were associated with this bug in the last 7 days.   

Repository breakdown:
* autoland: 1

Platform breakdown:
* linux32: 1

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1354310&startday=2017-08-28&endday=2017-09-03&tree=all
You need to log in before you can comment on or make changes to this bug.