Closed Bug 613147 Opened 9 years ago Closed 9 years ago

[SeaMonkey] mochitest-plain-1: new test fails = "test_bug574663.html | Ctrl-scrolling should zoom while the user is touching the touchpad - didn't expect 1, but got it"

Categories

(Core :: Widget, defect, major)

defect
Not set
major

Tracking

()

VERIFIED FIXED
mozilla2.0b8

People

(Reporter: sgautherie, Assigned: mstange)

References

(Blocks 1 open bug, )

Details

(Whiteboard: [perma-orange])

Attachments

(1 file)

3-platform t-b:
{
36806 ERROR TEST-UNEXPECTED-FAIL | /tests/content/events/test/test_bug574663.html | Ctrl-scrolling should zoom while the user is touching the touchpad - didn't expect 1, but got it
36807 ERROR TEST-UNEXPECTED-FAIL | /tests/content/events/test/test_bug574663.html | Ctrl-scrolling shouldn't scroll while the user is touching the touchpad - got 15, expected 0
}


(Fwiw, this is the last 3-platform SeaMonkey perma-orange for this suite...)
This might fix it. Is there any way I can test it?
Comment on attachment 492365 [details] [diff] [review]
set the zoom pref

(In reply to comment #1)
> This might fix it. Is there any way I can test it?

(Download build and tests then apply patch manually.)

>+  prefSvc.setBoolPref("mousewheel.withcontrolkey.action", 3);

with s/setBoolPref/setIntPref/, this patch fixes the test on my local Windows 2000 :-)

Additional nits:
*Using http://mxr.mozilla.org/mozilla-central/source/testing/mochitest/mozprefs.js#51 pref() would be even better.
*s/SimpleTest.finish()/SimpleTest.executeSoon(SimpleTest.finish)/ with a comment, as there is |setTimeout(nextTest, 0);|,
 see http://mxr.mozilla.org/mozilla-central/source/testing/mochitest/tests/SimpleTest/SimpleTest.js#780

Thanks.
Attachment #492365 - Flags: feedback+
Assignee: nobody → mstange
Status: NEW → ASSIGNED
Hardware: x86 → All
(In reply to comment #2)
> *s/SimpleTest.finish()/SimpleTest.executeSoon(SimpleTest.finish)/ with a
> comment

To be explicit: "// finish(), yet let the test actually end first, to be safe."
Pushed with setIntPref, thanks for the catch:
http://hg.mozilla.org/mozilla-central/rev/118d61cdfb06
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla2.0b8
http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1291046976.1291049978.4891.gz
Linux comm-central-trunk debug test mochitests-1/5 on 2010/11/29 08:09:36
http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1291049972.1291053200.18616.gz
OS X 10.5 comm-central-trunk debug test mochitests-1/5 on 2010/11/29 08:59:32
http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1291047565.1291050294.6341.gz
OS X 10.6 comm-central-trunk debug test mochitests-1/5 on 2010/11/29 08:19:25
http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1291052806.1291055371.28156.gz
WINNT 5.2 comm-central-trunk debug test mochitests-1/5 on 2010/11/29 09:46:46

V.Fixed
Status: RESOLVED → VERIFIED
Flags: in-testsuite+
You need to log in before you can comment on or make changes to this bug.