Closed
Bug 706383
Opened 14 years ago
Closed 14 years ago
Save screenshot in bundle
Categories
(Firefox for Android Graveyard :: General, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: blassey, Assigned: blassey)
Details
Attachments
(2 files, 1 obsolete file)
5.10 KB,
patch
|
dougt
:
review+
|
Details | Diff | Splinter Review |
6.78 KB,
patch
|
dougt
:
review+
|
Details | Diff | Splinter Review |
No description provided.
Attachment #577854 -
Flags: review?(doug.turner)
Assignee | ||
Comment 1•14 years ago
|
||
this patch removes the code that this patch replaces
Assignee: nobody → blassey.bugs
Assignee | ||
Updated•14 years ago
|
Attachment #577867 -
Flags: review?(doug.turner)
Comment 2•14 years ago
|
||
Wondering: Are we still considering using the screenshot on startup? In which startup cases does it actually make sense (see bug 703334)?
Assignee | ||
Comment 3•14 years ago
|
||
(In reply to Lucas Rocha (:lucasr) from comment #2)
> Wondering: Are we still considering using the screenshot on startup? In
> which startup cases does it actually make sense (see bug 703334)?
The working theory now is we only want to show it if we get "paged out" by the system, which is essentially what this patch does.
Updated•14 years ago
|
Attachment #577867 -
Flags: review?(doug.turner) → review+
Assignee | ||
Comment 4•14 years ago
|
||
the previous patch had a bunch of white space changes, which made it not apply for Doug. This doesn't have those white space changes, but I'll add them back in before landing.
Attachment #577854 -
Attachment is obsolete: true
Attachment #577854 -
Flags: review?(doug.turner)
Attachment #578022 -
Flags: review?(doug.turner)
Updated•14 years ago
|
Attachment #578022 -
Flags: review?(doug.turner) → review+
Assignee | ||
Comment 5•14 years ago
|
||
pushed https://hg.mozilla.org/projects/birch/rev/77a62cb00cc0, https://hg.mozilla.org/projects/birch/rev/f3a5150cf1ee and https://hg.mozilla.org/projects/birch/rev/56fe51d0023c
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Assignee | ||
Updated•14 years ago
|
Flags: in-testsuite?
Flags: in-litmus?(kbrosnan)
Assignee | ||
Comment 6•13 years ago
|
||
we killed this feature, no longer care about having tests for it
Flags: in-testsuite?
Flags: in-litmus?(kbrosnan)
Updated•5 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
•