1498892 - Scroll position test with frames

Status: RESOLVED FIXED

(Firefox for Android Graveyard :: Session Restore, enhancement, P1)

Comment 1

[Jan Henning [:JanH]]

Attachment: Bug 1498892 - Part 1: Clean up existing scroll position test. r?gbrown

Extract some repetitive logic into common helper functions/variables and pre-
pare support for setting/checking the scroll position on frames, too.

Additionally, I'm not sure why I ever thought it was necessary to manually fake
"scroll" events after calling scrollTo(x,y), but in any case it is no longer
necessary.

Comment 2

[Jan Henning [:JanH]]

Attachment: Bug 1498892 - Part 2: Add session store test for scroll positions of frames. r?gbrown

Comment 3

[Jan Henning [:JanH]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=2fa76f56c27b1324c4fa4b2fb597e8aed12ae43a

Comment 4

[Jan Henning [:JanH]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=46a43e263bd9aaaf687492e9578cd20d28283c20

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/fb3685a86f81
Part 1: Clean up existing scroll position test. r=gbrown
https://hg.mozilla.org/integration/autoland/rev/8e482a5a51ad
Part 2: Add session store test for scroll positions of frames. r=gbrown

Comment 6

[Andrei Ciure[:aciure]]

https://hg.mozilla.org/mozilla-central/rev/fb3685a86f81
https://hg.mozilla.org/mozilla-central/rev/8e482a5a51ad