Closed Bug 485994 Opened 16 years ago Closed 16 years ago

Intermittent MochiChrome test_wheeltransaction.xul failure

Categories

(Core :: Widget, defect)

x86
Linux
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla1.9.2a1

People

(Reporter: sgautherie, Assigned: masayuki)

References

()

Details

(Keywords: intermittent-failure)

Attachments

(1 file, 1 obsolete file)

I've notice "this" error a few times recently. Example: { http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1238429969.1238437840.6007.gz Linux mozilla-central unit test on 2009/03/30 09:19:29 *** 1591 ERROR TEST-UNEXPECTED-FAIL | chrome://mochikit/content/chrome/widget/tests/test_wheeltransaction.xul | wrong view was scrolled: Reset transaction by a mouse move event (v-7) - got "subview1", expected "rootview" }
Flags: wanted1.9.2?
Attached patch Patch v1.0 (obsolete) — Splinter Review
Probably, this can fix this bug. The cause should be unexpected timeout of IgnoreMoveDelayTime. Therefore, at that time, we should retry the tests with longer delay settings.
Assignee: nobody → masayuki
Status: NEW → ASSIGNED
Attachment #370148 - Flags: review?(Olli.Pettay)
I really really don't want to see yet another event just for testing. Couldn't you just capture mousemove and DOMMouseScroll events?
Attached patch Patch v2.0Splinter Review
ok, this should be simplest fix.
Attachment #370148 - Attachment is obsolete: true
Attachment #370668 - Flags: review?(Olli.Pettay)
Attachment #370148 - Flags: review?(Olli.Pettay)
Attachment #370668 - Flags: review?(Olli.Pettay) → review+
Attachment #370668 - Flags: superreview?(roc)
Comment on attachment 370668 [details] [diff] [review] Patch v2.0 Let's just try this :)
Attachment #370668 - Flags: superreview?(roc) → superreview+
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9.2a1
Looks like this isn't fixed after all -- reopening: *** 2155 ERROR TEST-UNEXPECTED-FAIL | chrome://mochikit/content/chrome/widget/tests/test_wheeltransaction.xul | wrong view was scrolled: Reset transaction by a mouse move event (h-3) - got "subview1", expected "rootview" http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1241614679.1241623545.11790.gz WINNT 5.2 mozilla-central unit test on 2009/05/06 05:57:59 (I initially filed bug 491712 on this before discovering this bug here.)
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Whiteboard: [orange]
Status: REOPENED → RESOLVED
Closed: 16 years ago16 years ago
Resolution: --- → FIXED
(ah, sorry -- disregard comment 7 -- looks like smaug's considering bug 491712 to be a new issue.)
Flags: wanted1.9.2?
Whiteboard: [orange]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: