1605674 - Test failure: comm/calendar/test/browser/browser_import.js | Test timed out -

Status: RESOLVED FIXED

(Calendar :: Import and Export, defect, P5)

Description

[Treeherder Bug Filer]

Filed by: geoff [at] darktrojan.net
Parsed log: https://treeherder.mozilla.org/logviewer.html#?job_id=282262830&repo=comm-central
Full log: https://firefox-ci-tc.services.mozilla.com/api/queue/v1/task/bCynVxmiRyigoNtusekQlA/runs/0/artifacts/public/logs/live_backing.log

---

I've bisected this failure to https://hg.mozilla.org/mozilla-central/rev/b874f97516f591f72a72676466871f67b43d800b but don't know how that causes the call to picker.open in loadEventsFromFile to fail.

Comment 1

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/239192d9246b
Disable test calendar/test/browser/browser_import.js; rs=me

Comment 2

[Intermittent Failures Robot]

1 failures in 4582 pushes (0.0 failures/push) were associated with this bug in the last 7 days.

Repository breakdown:

• comm-central: 1

Platform breakdown:

• linux32: 1

For more details, see:
https://treeherder.mozilla.org/intermittent-failures.html#/bugdetails?bug=1605674&startday=2019-12-16&endday=2019-12-22&tree=all

Comment 3

[Geoff Lankow (:darktrojan)]

This setTimeout in MockFilePicker.jsm never fires in this test. From reading around the changes in bug 1597433 I think that this.window has been suspended and never resumes. I'm not sure why that would happen as this.window is the main Thunderbird window.

Calling window.setTimeout from outside the JSM using the same window works fine too. (So does importing setTimeout from Timer.jsm and using it instead, but that's not really relevant.)

Tetsuharu, could you please have a look? I think there is an underlying issue that isn't specific to Thunderbird.

Comment 4

[Jorg K (CEST = GMT+2)]

Drive-by comment: https://hg.mozilla.org/mozilla-central/rev/b874f97516f591f72a72676466871f67b43d800b#l1.17 removes use of nsIDocShellTreeItem which is on the hit list, see bug 1603649 comment #35.

Comment 5

[Tetsuharu OHZEKI [:tetsuharu] (UTC+9)]

(In reply to Geoff Lankow (:darktrojan) from comment #3)

Tetsuharu, could you please have a look? I think there is an underlying issue that isn't specific to Thunderbird.

Umm, I was going to replaced nsIDocShellTreeItem simply, and I'm not sure about Thunderbird well...
Sorry, I cannot reply what causes this regression soon.

Comment 6

[Geoff Lankow (:darktrojan)]

Attachment: Bug 1605674 - Stop using window.setTimeout in MockFilePickerInstance. r?kmag

Using window.setTimeout is causing a Thunderbird test to stall for not entirely clear reasons. Instead use Timer.jsm setTimeout.

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/35dade3eb213
Stop using window.setTimeout in MockFilePickerInstance. r=kmag

Comment 8

[Cristian Brindusan [:cbrindusan]]

https://hg.mozilla.org/mozilla-central/rev/35dade3eb213

Comment 9

[Geoff Lankow (:darktrojan)]

https://hg.mozilla.org/comm-central/rev/ed741aa127edc115ccfdd103b864e77891fef371