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

Status: RESOLVED FIXED

(DevTools :: General, defect)

Description

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

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

Comment 1

[Intermittent Failures Robot]

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

[Intermittent Failures Robot]

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

[Intermittent Failures Robot]

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

[Intermittent Failures Robot]

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

Comment 5

[Sami Jaktholm]

Attachment: 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.

Comment 6

[Sami Jaktholm]

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

The changes should fix bug 1219490 too.

Comment 7

[Intermittent Failures Robot]

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

Comment 8

[Patrick Brosset <:pbro>]

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.

Comment 9

[Pulsebot]

https://hg.mozilla.org/integration/fx-team/rev/fc6459ef44a4

Comment 10

[Carsten Book [:Tomcat]]

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