Status

defect
P4
normal
RESOLVED FIXED
9 years ago
3 years ago

People

(Reporter: mitcho, Assigned: mitcho)

Tracking

unspecified
Firefox 4.0b11
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [cleanup][ux][qa-])

Attachments

(2 obsolete attachments)

killNewTabGroup was originally part of the "implementing new tab experience" bug 575851, but now it is completely inappropriate to ship this. In particular, it will automagically close any group that someone created called "New Tabs" after a reboot. Scary!
Posted patch Trivial patch (obsolete) — Splinter Review
Assignee: nobody → mitcho
Status: NEW → ASSIGNED
Attachment #506148 - Flags: review?(ian)
Whiteboard: [cleanup][ux] → [cleanup][ux][qa-]
Comment on attachment 506148 [details] [diff] [review]
Trivial patch

Yes!
Attachment #506148 - Flags: review?(ian) → review+
Attachment #506148 - Flags: approval2.0?
Comment on attachment 506148 [details] [diff] [review]
Trivial patch

a=beltzner, virtuous
Attachment #506148 - Flags: approval2.0? → approval2.0+
Posted patch Patrch for checkin (obsolete) — Splinter Review
Attachment #506148 - Attachment is obsolete: true

Comment 5

9 years ago
Comment on attachment 506823 [details] [diff] [review]
Patrch for checkin

Wrong patch :)
Attachment #506823 - Attachment is obsolete: true

Comment 6

9 years ago
http://hg.mozilla.org/mozilla-central/rev/c8ea68010263
Keywords: checkin-needed
Target Milestone: --- → Firefox 4.0b11

Updated

9 years ago
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
(In reply to comment #5)
> Comment on attachment 506823 [details] [diff] [review]
> Patrch for checkin
> 
> Wrong patch :)

Oh no! Thanks for taking care of this!
Product: Firefox → Firefox Graveyard
You need to log in before you can comment on or make changes to this bug.