716031 - Intermittent chrome://mochitests/content/chrome/editor/libeditor/base/tests/test_selection_move_commands.xul | cmd_scrollBottom - got 0, expected -300

Status: RESOLVED FIXED

(Core :: DOM: Editor, defect)

Description

[Ed Morley [:emorley]]

https://tbpl.mozilla.org/php/getParsedLog.php?id=8265070&tree=Firefox
Rev3 MacOSX Leopard 10.5.8 mozilla-central opt test mochitest-other on https://hg.mozilla.org/mozilla-central/rev/d702c3cf4e4a

9916 ERROR TEST-UNEXPECTED-FAIL | chrome://mochitests/content/chrome/editor/libeditor/base/tests/test_selection_move_commands.xul | cmd_scrollBottom - got 0, expected -300

Comment 1

[Legacy TBPL/Treeherder Robot]

edmorley
https://tbpl.mozilla.org/php/getParsedLog.php?id=8360874&tree=Mozilla-Inbound
Rev4 MacOSX Snow Leopard 10.6 mozilla-inbound opt test mochitest-other on 2012-01-05 16:06:19

9948 ERROR TEST-UNEXPECTED-FAIL | chrome://mochitests/content/chrome/editor/libeditor/base/tests/test_selection_move_commands.xul | cmd_scrollBottom - got 0, expected -300
31753 ERROR TEST-UNEXPECTED-FAIL | chrome://mochitests/content/chrome/widget/tests/test_bug596600.xul | box shouldn't be hovered

Comment 2

[Legacy TBPL/Treeherder Robot]

edmorley
https://tbpl.mozilla.org/php/getParsedLog.php?id=8485758&tree=Firefox
Rev4 MacOSX Snow Leopard 10.6 mozilla-central opt test mochitest-other on 2012-01-11 11:18:27

9948 ERROR TEST-UNEXPECTED-FAIL | chrome://mochitests/content/chrome/editor/libeditor/base/tests/test_selection_move_commands.xul | cmd_scrollBottom - got 0, expected -300

Comment 3

[Legacy TBPL/Treeherder Robot]

ttaubert
https://tbpl.mozilla.org/php/getParsedLog.php?id=8530865&tree=Fx-Team
Rev4 MacOSX Snow Leopard 10.6 fx-team opt test mochitest-other on 2012-01-13 06:08:45

9948 ERROR TEST-UNEXPECTED-FAIL | chrome://mochitests/content/chrome/editor/libeditor/base/tests/test_selection_move_commands.xul | cmd_scrollBottom - got 0, expected -300

Comment 4

[Legacy TBPL/Treeherder Robot]

bz
https://tbpl.mozilla.org/php/getParsedLog.php?id=8543590&tree=Try
Rev3 Fedora 12x64 try opt test mochitest-other on 2012-01-13 22:52:04

10637 ERROR TEST-UNEXPECTED-FAIL | chrome://mochitests/content/chrome/editor/libeditor/base/tests/test_selection_move_commands.xul | cmd_scrollBottom - got 0, expected -300

Comment 5

[Legacy TBPL/Treeherder Robot]

bz
https://tbpl.mozilla.org/php/getParsedLog.php?id=8543630&tree=Try
Rev3 WINNT 6.1 try opt test mochitest-other on 2012-01-13 22:47:06

10706 ERROR TEST-UNEXPECTED-FAIL | chrome://mochitests/content/chrome/editor/libeditor/base/tests/test_selection_move_commands.xul | cmd_scrollBottom - got 0, expected -300

Comment 6

[Legacy TBPL/Treeherder Robot]

vporof
https://tbpl.mozilla.org/php/getParsedLog.php?id=8540895&tree=Try
Rev4 MacOSX Snow Leopard 10.6 try opt test mochitest-other on 2012-01-13 18:50:52

9948 ERROR TEST-UNEXPECTED-FAIL | chrome://mochitests/content/chrome/editor/libeditor/base/tests/test_selection_move_commands.xul | cmd_scrollBottom - got 0, expected -300

Comment 7

[Legacy TBPL/Treeherder Robot]

bz
https://tbpl.mozilla.org/php/getParsedLog.php?id=8544469&tree=Try
Rev3 WINNT 6.1 try opt test mochitest-other on 2012-01-14 00:16:07

10706 ERROR TEST-UNEXPECTED-FAIL | chrome://mochitests/content/chrome/editor/libeditor/base/tests/test_selection_move_commands.xul | cmd_scrollBottom - got 0, expected -300

Comment 8

[Legacy TBPL/Treeherder Robot]

bz
https://tbpl.mozilla.org/php/getParsedLog.php?id=8544116&tree=Try
Rev3 WINNT 6.1 try opt test mochitest-other on 2012-01-13 23:32:50

10706 ERROR TEST-UNEXPECTED-FAIL | chrome://mochitests/content/chrome/editor/libeditor/base/tests/test_selection_move_commands.xul | cmd_scrollBottom - got 0, expected -300

Comment 9

[Legacy TBPL/Treeherder Robot]

bz
https://tbpl.mozilla.org/php/getParsedLog.php?id=8550929&tree=Try
Rev3 WINNT 6.1 try opt test mochitest-other on 2012-01-14 08:51:33

10706 ERROR TEST-UNEXPECTED-FAIL | chrome://mochitests/content/chrome/editor/libeditor/base/tests/test_selection_move_commands.xul | cmd_scrollBottom - got 0, expected -300

Comment 10

[Boris Zbarsky [:bzbarsky]]

Attachment: This should fix it, I think; fixes things for me in the try pushes where I was seeing this a lot

Comment 11

[Boris Zbarsky [:bzbarsky]]

Pushed as https://hg.mozilla.org/integration/mozilla-inbound/rev/1e0405f18207 in the meantime, because I needed it for something else.

Comment 12

[Jonathan Kew [:jfkthame]]

This merged to m-c:
https://hg.mozilla.org/mozilla-central/rev/1e0405f18207

I'm not resolving the bug for now, though, as it's still pending review - not sure how best to handle that.

Comment 13

[Boris Zbarsky [:bzbarsky]]

I think we should just go ahead and resolve, then file followups if Ehsan has a problem with the patch or reopen if this keeps happening.

Comment 14

[Legacy TBPL/Treeherder Robot]

edmorley
https://tbpl.mozilla.org/php/getParsedLog.php?id=11465096&tree=Profiling
Rev4 MacOSX Snow Leopard 10.6 profiling opt test mochitest-other on 2012-05-04 06:33:44
slave: talos-r4-snow-022

10201 ERROR TEST-UNEXPECTED-FAIL | chrome://mochitests/content/chrome/editor/libeditor/base/tests/test_selection_move_commands.xul | cmd_scrollPageUp - got -300, expected -210
10202 ERROR TEST-UNEXPECTED-FAIL | chrome://mochitests/content/chrome/editor/libeditor/base/tests/test_selection_move_commands.xul | Can scroll by lines

Comment 15

[Legacy TBPL/Treeherder Robot]

edmorley
https://tbpl.mozilla.org/php/getParsedLog.php?id=11840749&tree=Fx-Team
Rev3 WINNT 5.1 fx-team debug test mochitest-other on 2012-05-17 11:58:09
slave: talos-r3-xp-009

10456 ERROR TEST-UNEXPECTED-FAIL | chrome://mochitests/content/chrome/editor/libeditor/base/tests/test_selection_move_commands.xul | Can scroll by lines
10457 ERROR TEST-UNEXPECTED-FAIL | chrome://mochitests/content/chrome/editor/libeditor/base/tests/test_selection_move_commands.xul | cmd_scrollLineUp - got -243, expected -300

Comment 16

[Legacy TBPL/Treeherder Robot]

jorendorff
https://tbpl.mozilla.org/php/getParsedLog.php?id=11970986&tree=Try
Rev3 WINNT 5.1 try debug test mochitest-other on 2012-05-22 19:00:07
slave: talos-r3-xp-014

10454 ERROR TEST-UNEXPECTED-FAIL | chrome://mochitests/content/chrome/editor/libeditor/base/tests/test_selection_move_commands.xul | Can scroll by lines
10455 ERROR TEST-UNEXPECTED-FAIL | chrome://mochitests/content/chrome/editor/libeditor/base/tests/test_selection_move_commands.xul | cmd_scrollLineUp - got -243, expected -300

Comment 17

[Legacy TBPL/Treeherder Robot]

paul
https://tbpl.mozilla.org/php/getParsedLog.php?id=14705653&tree=Fx-Team
Rev3 WINNT 6.1 fx-team debug test mochitest-other on 2012-08-25 08:19:46
slave: talos-r3-w7-035

9569 ERROR TEST-UNEXPECTED-FAIL | chrome://mochitests/content/chrome/editor/libeditor/base/tests/test_selection_move_commands.xul | Can scroll by lines
9570 ERROR TEST-UNEXPECTED-FAIL | chrome://mochitests/content/chrome/editor/libeditor/base/tests/test_selection_move_commands.xul | cmd_scrollLineUp - got -243, expected -300

Comment 18

[Vladimir Vukicevic [:vlad] [:vladv] (needinfo me, slow to respond)]

Attachment: alternate patch, using explicit refresh driver timing

Alternate/different fix -- drive the refreshdriver directly, which is what is doing the scrolling.  This seems to fix this for me in a pretty bulletproof way; ehsan r+'d it over my shoulder.  May also fix bug 752808 and bug 759254.

Comment 19

[Ed Morley [:emorley]]

https://hg.mozilla.org/mozilla-central/rev/e28ab3e5e961