Intermittent browser_markupview_dragdrop_autoscroll.js | The view was scrolled down - | The view was scrolled up -

RESOLVED FIXED in Firefox 45

Status

()

Firefox
Developer Tools
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: KWierso, Assigned: Sami Jaktholm)

Tracking

({intermittent-failure})

unspecified
Firefox 45
intermittent-failure
Points:
---

Firefox Tracking Flags

(firefox45 fixed)

Details

MozReview Requests

()

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(1 attachment)

(Reporter)

Description

2 years ago
https://treeherder.mozilla.org/logviewer.html#?job_id=16491281&repo=mozilla-inbound

Comment 1

2 years ago
15 automation job failures were associated with this bug in the last 7 days.

Repository breakdown:
* mozilla-inbound: 10
* b2g-inbound: 3
* mozilla-aurora: 2

Platform breakdown:
* windows7-32: 9
* windows8-64: 4
* windowsxp: 2

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1219953&startday=2015-10-26&endday=2015-11-01&tree=all

Comment 2

2 years ago
15 automation job failures were associated with this bug yesterday.

Repository breakdown:
* mozilla-inbound: 7
* fx-team: 4
* mozilla-central: 2
* b2g-inbound: 2

Platform breakdown:
* windows8-64: 13
* windows7-32: 2

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1219953&startday=2015-11-06&endday=2015-11-06&tree=all

Comment 3

2 years ago
46 automation job failures were associated with this bug in the last 7 days.

Repository breakdown:
* mozilla-inbound: 16
* fx-team: 14
* b2g-inbound: 9
* mozilla-central: 6
* try: 1

Platform breakdown:
* windows8-64: 31
* windows7-32: 13
* windowsxp: 2

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1219953&startday=2015-11-02&endday=2015-11-08&tree=all

Comment 4

2 years ago
26 automation job failures were associated with this bug in the last 7 days.

Repository breakdown:
* mozilla-inbound: 12
* fx-team: 7
* b2g-inbound: 4
* mozilla-central: 3

Platform breakdown:
* windows7-32: 15
* windows8-64: 11

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1219953&startday=2015-11-09&endday=2015-11-15&tree=all
(Assignee)

Comment 5

2 years ago
Created attachment 8690492 [details]
MozReview Request: Bug 1219953 - Consider Markup View autoscroll finished only if the view has not scrolled for 30 consecutive frames. r?pbrosset

Bug 1219953 - Consider Markup View autoscroll finished only if the view has not scrolled for 30 consecutive frames. r?pbrosset

Waiting for 50ms and checking if the scrollTop has changed does not
necessarily mean that the scrolling has stopped (or even started). It
might also mean that a GC has blocked the autoscroll interval from
running and the scrolling will happen in a few moments. If the view
hasn't even started to scroll, the assertion that the view has scrolled
fails. If the view has started to scroll, the next call to
waitForViewScroll() might resolve when before the direction of the
scrolling is reversed causing the assertions that the direction has
changed to fail.

Instead of relying on intervals to detect if scrolling has stopped, the
test could wait for a certain number of frames during which the view
has not scrolled to determine if the scrolling has ended. This method has
successfully been used in layout/base/tests/test_scroll_snapping.html
to detect when scrolling has stopped.

These changes make browser_markupview_dragdrop_autoscroll.js to use the
previously presented method to detect the end of markup-view autoscroll.
The code used in the layout test has been included in the test and
modified to yield and use promises when waiting for a frame.
Attachment #8690492 - Flags: review?(pbrosset)
(Assignee)

Comment 6

2 years ago
Try: https://treeherder.mozilla.org/#/jobs?repo=try&revision=34ae04ee1f81

The changes should fix bug 1219490 too.
Assignee: nobody → sjakthol
Status: NEW → ASSIGNED

Comment 7

2 years ago
11 automation job failures were associated with this bug in the last 7 days.

Repository breakdown:
* mozilla-inbound: 4
* mozilla-central: 4
* fx-team: 2
* b2g-inbound: 1

Platform breakdown:
* windows7-32: 5
* windowsxp: 4
* windows8-64: 1
* osx-10-10: 1

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1219953&startday=2015-11-16&endday=2015-11-22&tree=all
Attachment #8690492 - Flags: review?(pbrosset) → review+
Comment on attachment 8690492 [details]
MozReview Request: Bug 1219953 - Consider Markup View autoscroll finished only if the view has not scrolled for 30 consecutive frames. r?pbrosset

https://reviewboard.mozilla.org/r/25867/#review23299

Really nice. Thanks.
(Assignee)

Updated

2 years ago
Keywords: checkin-needed

Comment 9

2 years ago
https://hg.mozilla.org/integration/fx-team/rev/fc6459ef44a4
Keywords: checkin-needed

Comment 10

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/fc6459ef44a4
Status: ASSIGNED → RESOLVED
Last Resolved: 2 years ago
status-firefox45: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 45
You need to log in before you can comment on or make changes to this bug.