1587416 - toolkit/components/passwordmgr/test/mochitest/test_autocomplete_tab_between_fields.html | Test timed out. when Gecko 71 merges to Beta on 2019-10-14

Status: VERIFIED FIXED

(Toolkit :: Password Manager, defect, P1)

Description

[Narcis Beleuzu [:NarcisB]]

Central as beta:
https://treeherder.mozilla.org/#/jobs?repo=try&resultStatus=testfailed%2Cbusted%2Cexception%2Crunnable&revision=cb7f32501a17e2dd1003d1ae2b1944658ac613c7&selectedJob=270467210

Log link: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=270466797&repo=try&lineNumber=4590

Log snippet:
[task 2019-10-09T12:42:55.934Z] 12:42:55 INFO - TEST-PASS | toolkit/components/passwordmgr/test/mochitest/test_autocomplete_tab_between_fields.html | Test value of field pword in form 1
[task 2019-10-09T12:42:55.935Z] 12:42:55 INFO - waiting for AC results
[task 2019-10-09T12:42:55.935Z] 12:42:55 INFO - Buffered messages finished
[task 2019-10-09T12:42:55.935Z] 12:42:55 INFO - TEST-UNEXPECTED-FAIL | toolkit/components/passwordmgr/test/mochitest/test_autocomplete_tab_between_fields.html | Test timed out.
[task 2019-10-09T12:42:55.935Z] 12:42:55 INFO - SimpleTest.ok@https://example.com/tests/SimpleTest/SimpleTest.js:277:18
[task 2019-10-09T12:42:55.935Z] 12:42:55 INFO - reportError@https://example.com/tests/SimpleTest/TestRunner.js:121:22
[task 2019-10-09T12:42:55.936Z] 12:42:55 INFO - TestRunner._checkForHangs@https://example.com/tests/SimpleTest/TestRunner.js:142:18
[task 2019-10-09T12:42:56.648Z] 12:42:56 INFO - GECKO(3356) | MEMORY STAT | vsize 20974526MB | residentFast 719MB
[task 2019-10-09T12:42:56.663Z] 12:42:56 INFO - TEST-OK | toolkit/components/passwordmgr/test/mochitest/test_autocomplete_tab_between_fields.html | took 312941ms

Comment 1

[Sam Foster [:sfoster] (he/him)]

The timeout is happening when we run the "single_login_subdomain_no_inputs" test, which relies on the signon.includeOtherSubdomainsInLookup pref being set - which is only true for Nightly currently. Probably the best fix here is to use pushPrefEnv to toggle that pref on when we run this test, just in case we don't end up letting that pref ride the trains before 71.

Comment 2

[Sam Foster [:sfoster] (he/him)]

Attachment: Bug 1587416 - Ensure signon.includeOtherSubdomainsInLookup is true when testing subdomain form fill/autocomplete behavior. r?MattN

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/ba8a929a8b77
Ensure signon.includeOtherSubdomainsInLookup is true when testing subdomain form fill/autocomplete behavior. r=MattN

Comment 4

[Cristina Coroiu [:ccoroiu]]

https://hg.mozilla.org/mozilla-central/rev/ba8a929a8b77

Comment 5

[Andreea Pavel [:apavel]]

Does not occur in beta simulations: https://treeherder.mozilla.org/#/jobs?repo=try&group_state=expanded&selectedJob=270667076&resultStatus=success%2Ctestfailed%2Cbusted%2Cexception%2Cretry%2Cusercancel%2Crunnable&revision=bb2fcf3fbaadf65542cbd01b7cbb82a93f1a6bba&searchStr=debug%2Cmochitest%2C5