Closed
Bug 700675
Opened 13 years ago
Closed 13 years ago
Last title not restored along with screenshot
Categories
(Firefox for Android Graveyard :: General, defect)
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: kats, Assigned: kats)
References
Details
Attachments
(2 files)
1.98 KB,
patch
|
mfinkle
:
review+
|
Details | Diff | Splinter Review |
2.42 KB,
patch
|
mfinkle
:
review+
|
Details | Diff | Splinter Review |
Open some page, then quit fennec. Wait a few seconds for fennec to shut down properly, then start it up again. Note how the url display doesn't show the title of the page that matches the screenshot. From a code inspection, the last-title saved pref is never used.
Assignee | ||
Comment 1•13 years ago
|
||
Attachment #572828 -
Flags: review?(mark.finkle)
Assignee | ||
Comment 2•13 years ago
|
||
Attachment #572829 -
Flags: review?
Assignee | ||
Updated•13 years ago
|
Attachment #572829 -
Flags: review? → review?(mark.finkle)
Updated•13 years ago
|
Attachment #572828 -
Flags: review?(mark.finkle) → review+
Comment 3•13 years ago
|
||
Comment on attachment 572829 [details] [diff] [review]
(2/2) Restore last-title
Looks good: If we are launching with a URL in the intent, we are not attempting the show the previous screenshot uri/title.
Attachment #572829 -
Flags: review?(mark.finkle) → review+
Assignee | ||
Comment 4•13 years ago
|
||
http://hg.mozilla.org/projects/birch/rev/c8d7aa623f9b
http://hg.mozilla.org/projects/birch/rev/3e0e4edc8f00
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Comment 5•13 years ago
|
||
These patches were backed while investigating Talos failures. Now that tests are green again, we will need to reland.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Comment 6•13 years ago
|
||
backout was backed out https://hg.mozilla.org/projects/birch/rev/6f925b45a547
Status: REOPENED → RESOLVED
Closed: 13 years ago → 13 years ago
Resolution: --- → FIXED
Comment 7•13 years ago
|
||
No longer reproducible on:
Fennec Native 13.0a1 (2012-02-09)
Device: Samsung Galaxy SII
Status: RESOLVED → VERIFIED
Updated•4 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
•