Closed Bug 870142 Opened 9 years ago Closed 4 years ago

updatePagesAndWait() has no timeout

Categories

(Firefox for Metro Graveyard :: Tests, defect)

x86_64
Windows 8.1
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED INCOMPLETE

People

(Reporter: jimm, Unassigned)

References

Details

(Whiteboard: p=0)

Attachments

(1 file)

I'm running our tests on an automation test slave and occasionally seeing a timeout in this wait statement. Since it doesn't have a timeout it can hang test runs for a while.

http://hg.mozilla.org/mozilla-central/annotate/b842d26dd5f0/browser/metro/base/tests/mochitest/browser_topsites.js#l130
Blocks: 860899
Thanks for spotting this. I have a patch I just need to test out - I should get it in for review today.
Assignee: nobody → sfoster
I don't think this was a problem with your tests. For some reason the start page wasn't loading occasionally.
Attachment #747396 - Attachment is patch: true
Attachment #747396 - Attachment mime type: message/rfc822 → text/plain
Attachment #747396 - Flags: review?(jmathies) → review+
Blocks: metrobacklog
Whiteboard: p=0
Assignee: sfoster → nobody
OS: Windows 8 Metro → Windows 8.1
Mass close of bugs in obsolete product https://bugzilla.mozilla.org/show_bug.cgi?id=1350354
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → INCOMPLETE
You need to log in before you can comment on or make changes to this bug.