680431 - Test test_bug511075.html is calling finish() twice

Status: RESOLVED FIXED

(Testing :: Mochitest, defect)

Description

[Neil Deakin]

Attachment: sample fix

This issue causes the next test to not run properly.

The attached patch shows the issue: the scroll event is firing twice, but I don't know whether this is a test issue or the test showing an actual bug.

Comment 1

[Olli Pettay [:smaug][[e-mail]]]

Comment on attachment 554410 [details] [diff] [review]
sample fix

Clicking scrollbar sends several scroll events.

Comment 2

[Olli Pettay [:smaug][[e-mail]]]

at least if smooth scrolling is enabled.

Comment 3

[Olli Pettay [:smaug][[e-mail]]]

Hmm, but without smooth scroll, there should be only one scroll event.

Comment 4

[Olli Pettay [:smaug][[e-mail]]]

Attachment: test

Comment 5

[Kyle Huey (Exited; not receiving bugmail, old account, do not use)]

This landed as http://hg.mozilla.org/integration/mozilla-inbound/rev/f19404f45022, but was backed out in http://hg.mozilla.org/integration/mozilla-inbound/rev/dbc9ed28ba95 for causing perma-orange in test_resizer.xul.

Comment 6

[Neil Deakin]

Hmmm. I for some reason forgot about that issue, even though I have a patch for it.

Comment 7

[Ed Morley [:emorley]]

http://hg.mozilla.org/mozilla-central/rev/f19404f45022
and backout:
http://hg.mozilla.org/mozilla-central/rev/dbc9ed28ba95

Comment 11

[Matt Brubeck (:mbrubeck)]

https://hg.mozilla.org/mozilla-central/rev/6cc4966475c2