Closed Bug 680431 Opened 9 years ago Closed 9 years ago

Test test_bug511075.html is calling finish() twice

Categories

(Testing :: Mochitest, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
mozilla10

People

(Reporter: enndeakin, Assigned: enndeakin)

References

Details

Attachments

(2 files)

Attached patch sample fixSplinter Review
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.
Attachment #554410 - Flags: feedback?(Olli.Pettay)
Comment on attachment 554410 [details] [diff] [review]
sample fix

Clicking scrollbar sends several scroll events.
Attachment #554410 - Flags: feedback?(Olli.Pettay) → feedback+
at least if smooth scrolling is enabled.
Hmm, but without smooth scroll, there should be only one scroll event.
Attached file test
Hmmm. I for some reason forgot about that issue, even though I have a patch for it.
http://hg.mozilla.org/mozilla-central/rev/f19404f45022
and backout:
http://hg.mozilla.org/mozilla-central/rev/dbc9ed28ba95
Assignee: nobody → enndeakin
Version: unspecified → Trunk
Status: NEW → ASSIGNED
Duplicate of this bug: 686022
Depends on: 659338
Blocks: 668728
https://hg.mozilla.org/mozilla-central/rev/6cc4966475c2
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla10
Flags: in-testsuite-
You need to log in before you can comment on or make changes to this bug.