Mozilla /5.0 (Android;Linux armv7l;rv:9.0a1) Gecko/20110905 Firefox/9.0a1 Fennec/9.0a1 Device: LG Optimus 2X Steps to reproduce: 1. Open a page and scroll down to the bottom 2. Close the page 3. Undo closed tab Expected results: The undone closed page should be reloaded and scrolled to the bottom Actual result: Page is at the top, scroll position is not remembered. Note: bug 604463 is verified fixed but it seems that only for the browsing history not for the scroll offsets also.
So it seems like the patch in bug 604463 should have taken care of this. If it's not working now, perhaps it regressed?
I tried this on few builds after the patch landed in that bug and I could reproduce it on all of them, so I guess the patch didn't fix it.
Depends on: 604463
OS: Android → All
Hardware: ARM → All
Whiteboard: [fennec-sessionstore] → [fennec-sessionstore][firstname.lastname@example.org][good first bug]
Matt, could you add some pointers for where a new contributor should start looking to fix this?
This bug might actually be more complex than I thought. I'll remove the good-first-bug tag, but I can still help mentor this if someone wants to pick it up. I think the right way to fix this is to save the scroll position of the tab in the SessionStore "extData" property using SessionStore.saveTabValue: https://hg.mozilla.org/releases/mozilla-beta/file/cf0a29826586/mobile/components/SessionStore.js#l617 and then pull it out of there and re-apply it after the tab is restored.
Whiteboard: [fennec-sessionstore][email@example.com][good first bug] → [fennec-sessionstore][firstname.lastname@example.org]
Matt, is this still a valid bug for a new contributor to be working on? In other words, does this still apply to the Native UI, so that the work done won't be wasted?
This bug applies only to XUL Fennec, though we might need a separate bug to add the same feature to the new native Android UI.
Whiteboard: [fennec-sessionstore][email@example.com] → [fennec-sessionstore]
The bug is also reproducible on: Mozilla/5.0(Android;Linux armv7l;rv:11.0a2)Gecko/20120201 Firefox/11.0a2 Fennec/11.0a2 Device:Samsung Galaxy Tab OS:Android 3.1
Whiteboard: [fennec-sessionstore] → [fennec-sessionstore][fennec 13.0b1]
You need to log in before you can comment on or make changes to this bug.