Last Comment Bug 645799 - Hide the crash reporter from the recently used apps window
: Hide the crash reporter from the recently used apps window
Status: VERIFIED FIXED
:
Product: Fennec Graveyard
Classification: Graveyard
Component: General (show other bugs)
: Trunk
: ARM Android
: P4 normal (vote)
: Firefox 9
Assigned To: Mark Finkle (:mfinkle) (use needinfo?)
:
Mentors:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2011-03-28 12:31 PDT by Kevin Brosnan [:kbrosnan]
Modified: 2011-10-05 20:32 PDT (History)
2 users (show)
See Also:
QA Whiteboard:
Iteration: ---
Points: ---


Attachments
patch (1.17 KB, patch)
2011-09-30 11:06 PDT, Mark Finkle (:mfinkle) (use needinfo?)
blassey.bugs: review+
Details | Diff | Splinter Review
patch (1.53 KB, patch)
2011-09-30 12:56 PDT, Matt Brubeck (:mbrubeck)
mbrubeck: review+
asa: approval‑mozilla‑aurora+
Details | Diff | Splinter Review

Description Kevin Brosnan [:kbrosnan] 2011-03-28 12:31:23 PDT
In Android long clicking on the home button will bring up a list of recently used applications. It currently lists the Crash Reporter, if there is a way to avodi being on the list it would be good. Clicking on the crash reporter from this screen shows the crash dialog again.
Comment 1 Aaron Train [:aaronmt] 2011-09-30 08:50:20 PDT
Looks like the same style fix from bug 690705 is needed:

android:excludeFromRecents="true" defined in the crash reporter activity
Comment 2 Mark Finkle (:mfinkle) (use needinfo?) 2011-09-30 11:06:29 PDT
Created attachment 563780 [details] [diff] [review]
patch

Simply adds the android:excludeFromRecents="true" to the CrashReporter manifest. Also removes some TABs.
Comment 3 Mark Finkle (:mfinkle) (use needinfo?) 2011-09-30 11:41:20 PDT
https://hg.mozilla.org/integration/mozilla-inbound/rev/04377b9142c2
Comment 4 Matt Brubeck (:mbrubeck) 2011-09-30 12:56:20 PDT
Created attachment 563812 [details] [diff] [review]
patch
Comment 5 Matt Brubeck (:mbrubeck) 2011-09-30 13:06:31 PDT
Comment on attachment 563812 [details] [diff] [review]
patch

mfinkle already landed the same change with r=blassey:
http://hg.mozilla.org/integration/mozilla-inbound/rev/04377b9142c2
Comment 6 Marco Bonardo [::mak] 2011-10-01 02:24:33 PDT
https://hg.mozilla.org/mozilla-central/rev/04377b9142c2
Comment 7 Aaron Train [:aaronmt] 2011-10-03 07:04:08 PDT
Mozilla/5.0 (Android; Linux armv7l; rv:10.0a1) Gecko/20111003 Firefox/10.0a1 Fennec/10.0a1
Comment 8 Mark Finkle (:mfinkle) (use needinfo?) 2011-10-04 14:30:04 PDT
Comment on attachment 563812 [details] [diff] [review]
patch

Simple
Comment 9 Mark Finkle (:mfinkle) (use needinfo?) 2011-10-05 20:32:08 PDT
https://hg.mozilla.org/releases/mozilla-aurora/rev/c1343471b0db

Note You need to log in before you can comment on or make changes to this bug.