Closed
Bug 451677
Opened 17 years ago
Closed 17 years ago
deckbrowser is storing the window scroll, but needs browser scroll
Categories
(Firefox for Android Graveyard :: General, defect)
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: mfinkle, Assigned: mfinkle)
Details
Attachments
(1 file)
|
723 bytes,
patch
|
enndeakin
:
review+
|
Details | Diff | Splinter Review |
Just from visual inspection (I didn't test) I noticed that we are saving the window.scrollX/Y when saving a browser state. When restoring state, we use the values to scroll the browser contentWindow.
Therefore, I assume we want to save the browser.contentWindow scroll position.
Attachment #335007 -
Flags: review?(enndeakin)
|
||
Updated•17 years ago
|
Attachment #335007 -
Flags: review?(enndeakin) → review+
|
Assignee | |
Comment 1•17 years ago
|
||
changeset: 112:94a05f8e1d5e
tag: tip
user: Mark Finkle <mfinkle@mozilla.com>
date: Fri Aug 22 09:33:00 2008 -0400
summary: b=451677, r=enn. deckbrowser saves window scroll but should save browser scroll
Status: NEW → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
|
||
Comment 2•16 years ago
|
||
this code has been rewritten since this change. Verified via mxr.
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•