1948793 - Intermittent comm/mail/base/test/browser/browser_feedBodyScroll.js | Test timed out

Status: RESOLVED FIXED

(Thunderbird :: Message Reader UI, defect, P5)

Description

[Treeherder Bug Filer]

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

---

Comment 1

[Magnus Melin [:mkmelin]]

This was from bug 1830755

Comment 2

[John Bieling]

Attachment: Bug 1948793 - Attempt to fix browser_feedBodyScroll.js. r=#thunderbird-reviewers

From my observations, the test times out because the listener for the
scroll event is registered after the keypress event has been issued.

Since the EventUtils are also available inside a spawned SpecialPowers,
this patch moves the call to EventUtils.synthesizeKey() inside and
after the registration of the scroll event.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/736a4251b5b0
Attempt to fix browser_feedBodyScroll.js. r=vineet

Comment 4

[Intermittent Failures Robot]

Summary

8 failures in 4340 pushes (0.002 failures/push) were associated with this bug in the last 7 days.

Repository breakdown:

• comm-central: 8

Platform and build breakdown:

• linux1804-64-ccov-qr: 2
  • opt: 2
• linux1804-64-qr: 3
  • debug: 3
• windows11-32-2009-qr: 2
  • debug: 2
• windows11-64-2009-qr: 1
  • debug: 1

Table

	msix	no_variant
linux1804-64-ccov-qr--/opt		2
linux1804-64-qr--/debug		3
windows11-32-2009-qr--/debug		2
windows11-64-2009-qr--/debug	1

For more details, see:

https://treeherder.mozilla.org/intermittent-failures/bugdetails?bug=1948793&startday=2025-02-17&endday=2025-02-23&tree=all

Comment 5

[Magnus Melin [:mkmelin]]

Still happening (but not as frequently?)

Comment 6

[John Bieling]

Sadly true

• https://treeherder.mozilla.org/jobs?repo=try-comm-central&revision=05c969cac01f62c8b31d085386d58fbcc99aa2dd&selectedTaskRun=HalrZMxpRAyQh0ymcELuZg.0

Comment 7

[Intermittent Failures Robot]

Summary

6 failures in 4564 pushes (0.001 failures/push) were associated with this bug in the last 7 days.

Repository breakdown:

• comm-central: 6

Platform and build breakdown:

• linux1804-64-qr: 3
  • debug: 3
• windows11-64-2009-qr: 3
  • debug: 3

Table

	msix	no_variant
linux1804-64-qr--/debug		3
windows11-64-2009-qr--/debug	1	2

For more details, see:

https://treeherder.mozilla.org/intermittent-failures/bugdetails?bug=1948793&startday=2025-02-24&endday=2025-03-02&tree=all

Comment 8

[Magnus Melin [:mkmelin]]

Attachment: Bug 1948793 - Fixes to intermittent comm/mail/base/test/browser/browser_feedBodyScroll.js. r=freaktechnik

Make it pass with --headless --verify and other things

Comment 9

[John Bieling]

If this continues to fail, we might have to consider using a different test approach. A single keystroke may cause multiple scroll events.

Since we only wait for one, the next time we wait for the scroll from the keystroke, it could be resolved by one of the other scroll events from the prior keystroke. We probably have to wait until no more scroll events are issued. The scrollend event is also not helpful, since it does not mark the end of scroll, there will be more scroll events after it. See Bug 1869514.

Comment 10

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/622c14b21f6c
Fixes to intermittent comm/mail/base/test/browser/browser_feedBodyScroll.js. r=freaktechnik

Comment 11

[Magnus Melin [:mkmelin]]

New instance still :( https://treeherder.mozilla.org/logviewer?job_id=497650366&repo=comm-central&lineNumber=2636

Comment 12

[Magnus Melin [:mkmelin]]

(In reply to John Bieling (:TbSync) from comment #9)

Since we only wait for one, the next time we wait for the scroll from the keystroke, it could be resolved by one of the other scroll events from the prior keystroke. We probably have to wait until no more scroll events are issued. The scrollend event is also not helpful, since it does not mark the end of scroll, there will be more scroll events after it. See Bug 1869514.

It seems to me it should use "scrollend" actually, instead of "scroll". What did you mean by this? We do one scroll at a time and then wait for the event

Comment 13

[John Bieling]

If you log "scroll" and "scrollend" events continuously, you will see that a scrollend event is not the last event resulting from a scroll. A single scroll action from A to B will cause multiple "scroll" events, one "scrollend" event and then some more "scroll" events.

Comment 14

[Intermittent Failures Robot]

Summary

7 failures in 4009 pushes (0.002 failures/push) were associated with this bug in the last 7 days.

Repository breakdown:

• comm-central: 7

Platform and build breakdown:

• linux1804-64-ccov-qr: 2
  • opt: 2
• linux1804-64-qr: 1
  • opt: 1
• windows11-32-2009-qr: 1
  • debug: 1
• windows11-64-2009-ccov-qr: 1
  • opt: 1
• windows11-64-2009-qr: 2
  • debug: 2

Table

	msix	no_variant
linux1804-64-ccov-qr--/opt		2
linux1804-64-qr--/opt		1
windows11-32-2009-qr--/debug		1
windows11-64-2009-ccov-qr--/opt		1
windows11-64-2009-qr--/debug	2

For more details, see:

https://treeherder.mozilla.org/intermittent-failures/bugdetails?bug=1948793&startday=2025-03-03&endday=2025-03-09&tree=all

Comment 15

[Intermittent Failures Robot]

Summary

4 failures in 4826 pushes (0.001 failures/push) were associated with this bug in the last 7 days.

Repository breakdown:

• comm-central: 4

Platform and build breakdown:

• linux1804-64-qr: 2
  • opt: 2
• windows11-32-2009-qr: 1
  • debug: 1
• windows11-64-2009-qr: 1
  • debug: 1

Table

	no_variant
linux1804-64-qr--/opt	2
windows11-32-2009-qr--/debug	1
windows11-64-2009-qr--/debug	1

For more details, see:

https://treeherder.mozilla.org/intermittent-failures/bugdetails?bug=1948793&startday=2025-03-10&endday=2025-03-16&tree=all

Comment 16

[Magnus Melin [:mkmelin]]

I wonder if that's not about something else.
https://developer.mozilla.org/en-US/docs/Web/API/Element/scrollend_event says "The scrollend event fires when element scrolling has completed". I think that's all we care about.

A try run with "scrollend" seems alright: https://treeherder.mozilla.org/jobs?repo=try-comm-central&revision=f7cb14130cfe659a44af648f8cf161ff0f710ff6&selectedTaskRun=auW1YRWFQZmuU7aZINWUUg.0

Comment 17

[Magnus Melin [:mkmelin]]

Attachment: Bug 1948793 - Fix intermittent comm/mail/base/test/browser/browser_feedBodyScroll.js by listening for scrollend instead of scroll. r=john.bieling

Comment 18

[Intermittent Failures Robot]

Summary

9 failures in 5035 pushes (0.002 failures/push) were associated with this bug in the last 7 days.

Repository breakdown:

• comm-central: 9

Platform and build breakdown:

• linux1804-64-asan-qr: 1
  • opt: 1
• linux1804-64-ccov-qr: 1
  • opt: 1
• linux1804-64-qr: 2
  • debug: 2
• macosx1015-64-qr: 1
  • debug: 1
• windows11-32-2009-qr: 3
  • opt: 1
  • debug: 2
• windows11-64-2009-ccov-qr: 1
  • opt: 1

Table

	no_variant
linux1804-64-asan-qr--/opt	1
linux1804-64-ccov-qr--/opt	1
linux1804-64-qr--/debug	2
macosx1015-64-qr--/debug	1
windows11-32-2009-qr--/opt	1
windows11-32-2009-qr--/debug	2
windows11-64-2009-ccov-qr--/opt	1

For more details, see:

https://treeherder.mozilla.org/intermittent-failures/bugdetails?bug=1948793&startday=2025-03-17&endday=2025-03-23&tree=all

Comment 19

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/6720bb57b08a
Fix intermittent comm/mail/base/test/browser/browser_feedBodyScroll.js by listening for scrollend instead of scroll. r=freaktechnik

Comment 20

[Intermittent Failures Robot]

Repository breakdown:

• comm-central: 1

Table

	msix
windows11-64-2009-qr--/debug	1

For more details, see:

https://treeherder.mozilla.org/intermittent-failures/bugdetails?bug=1948793&startday=2025-03-24&endday=2025-03-30&tree=all