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

RESOLVED FIXED in mozilla18

Status

()

defect
RESOLVED FIXED
8 years ago
7 years ago

People

(Reporter: emorley, Assigned: vlad)

Tracking

({intermittent-failure})

12 Branch
mozilla18
x86
macOS
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

Reporter

Description

8 years ago
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
Reporter

Updated

8 years ago
Summary: chrome://mochitests/content/chrome/editor/libIntermittent editor/base/tests/test_selection_move_commands.xul | cmd_scrollBottom - got 0, expected -300 → Intermittent chrome://mochitests/content/chrome/editor/libeditor/base/tests/test_selection_move_commands.xul | cmd_scrollBottom - got 0, expected -300
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Assignee: nobody → bzbarsky
Whiteboard: [orange] → [need review][orange]
Pushed as https://hg.mozilla.org/integration/mozilla-inbound/rev/1e0405f18207 in the meantime, because I needed it for something else.
Target Milestone: --- → mozilla12
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.
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.
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED

Updated

8 years ago
Attachment #588687 - Flags: review?(ehsan) → review+

Updated

8 years ago
Whiteboard: [need review][orange] → [orange]
Comment hidden (Legacy TBPL/Treeherder Robot)
Reporter

Updated

7 years ago
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Assignee: bzbarsky → nobody
Comment hidden (Legacy TBPL/Treeherder Robot)
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.
Assignee: nobody → vladimir
Attachment #668161 - Flags: review+
Reporter

Comment 19

7 years ago
https://hg.mozilla.org/mozilla-central/rev/e28ab3e5e961
Status: REOPENED → RESOLVED
Closed: 8 years ago7 years ago
Resolution: --- → FIXED
Target Milestone: mozilla12 → mozilla18
Whiteboard: [orange]
You need to log in before you can comment on or make changes to this bug.