Closed
Bug 700667
Opened 13 years ago
Closed 13 years ago
[layers] Saved screenshot size/visible area not saved
Categories
(Firefox for Android Graveyard :: General, defect, P1)
Tracking
(fennec11+)
RESOLVED
INVALID
Tracking | Status | |
---|---|---|
fennec | 11+ | --- |
People
(Reporter: kats, Assigned: cwiiis)
References
Details
Attachments
(1 file, 1 obsolete file)
7.64 KB,
patch
|
kats
:
review+
|
Details | Diff | Splinter Review |
The saved screenshot doesn't always appear in the right part of the page. Page size is not saved. This is because we only save a screenshot, while we actually need to save some more info (page size, etc.) in order to pan and zoom during load.
Reporter | ||
Updated•13 years ago
|
Summary: [panzoom] Saved screenshot size/visible area not saved → [layers] Saved screenshot size/visible area not saved
Reporter | ||
Comment 1•13 years ago
|
||
Applies on top of the patch from bug 700654
Attachment #573003 -
Flags: review?(pwalton)
Comment 2•13 years ago
|
||
Comment on attachment 573003 [details] [diff] [review] Save/restore tab state Review of attachment 573003 [details] [diff] [review]: ----------------------------------------------------------------- Looks good to me.
Attachment #573003 -
Flags: review?(pwalton) → review+
Updated•13 years ago
|
Priority: -- → P1
Reporter | ||
Comment 3•13 years ago
|
||
Un-bitrotted such that it applies on top of a bunch of patches in the birch-pan-zoom queue (http://hg.mozilla.org/users/pwalton_mozilla.com/birch-pan-zoom/file/1deab2cd4eaf)
Attachment #573003 -
Attachment is obsolete: true
Attachment #573915 -
Flags: review+
Reporter | ||
Comment 4•13 years ago
|
||
This patch has been rolled into Chris' correct-displayport patch in the patch queue.
Assignee: kgupta → chrislord.net
Reporter | ||
Comment 5•13 years ago
|
||
The screenshot was taken out, so this is a moot issue now.
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → INVALID
Updated•13 years ago
|
tracking-fennec: --- → 11+
Updated•3 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•