810933 - Regression: Unable to initially scroll pages

Status: VERIFIED FIXED

(Firefox for Android Graveyard :: Toolbar, defect)

Description

[Aaron Train [:aaronmt]]

http://hg.mozilla.org/integration/mozilla-inbound/pushgloghtml?fromchange=eb636c95b1c0&tochange=d6a48ba051bc

--
Nightly (11/12)
Samsung Galaxy Nexus (Android 4.1.2)
Samsung Galaxy Note II (Android 4.1.1)

Comment 1

[Aaron Train [:aaronmt]]

I am using http://www.arstechnica.com as a test-case here; when the document is done - attempt to scroll down.

Comment 2

[Aaron Train [:aaronmt]]

Work-around that works for me; triggering a rotation that triggers a redraw 'restores' functionality

Comment 4

[Aaron Train [:aaronmt]]

Bug 809199

Comment 5

[Aaron Train [:aaronmt]]

Comment 0 should be http://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=eb636c95b1c0&tochange=d6a48ba051bc

Comment 6

[Kartikaya Gupta (email:[e-mail])]

Attachment: Patch

The arguments passed to the ImmutableViewportMetrics constructor are different than those passed to the old ViewportMetrics constructor, and my patch from bug 809199 that updated the JNI code failed to update it.

Comment 7

[Chris Lord [:cwiiis]]

Comment on attachment 680805 [details] [diff] [review]
Patch

Review of attachment 680805 [details] [diff] [review]:
-----------------------------------------------------------------

LGTM.

Comment 8

[Kartikaya Gupta (email:[e-mail])]

https://hg.mozilla.org/integration/mozilla-inbound/rev/cd854e9fe7c1

Comment 10

[Ed Morley [:emorley]]

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