624102 - Other tabs in tab group hidden after exiting private browsing

Status: VERIFIED FIXED

(Firefox Graveyard :: Panorama, defect)

Description

[Teoh Han Hui]

User-Agent:       Mozilla/5.0 (Windows NT 6.1; WOW64; rv:2.0b9pre) Gecko/20110107 Firefox/4.0b9pre
Build Identifier: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:2.0b9pre) Gecko/20110107 Firefox/4.0b9pre

Only the last active tab is shown after exiting Private Browsing mode. The other tabs in the same tab group are hidden until entering TabCandy, closing the tab or opening a new tab.

Reproducible: Always

Steps to Reproduce:
1. Group tabs in a tab group.
2. Start Private Browsing.
3. Stop Private Browsing.
Actual Results:  
Only the last active tab is visible.

Expected Results:  
Other tabs in the tab group are visible as well.

Tested on a clean profile.

Comment 1

[Alice0775 White]

In local build:
build from f453924d5fe1 : fails
build from 8bb78383bdb6 : works

Comment 2

[Luke Iliffe (Harlequin99)]

Although the STR is different (go into and out of private browsing), this seems the identical changeset to bug 586198 and bug 623792.

Comment 3

[Tim Taubert [:ttaubert] (inactive)]

Attachment: patch v1

Comment 4

[Tim Taubert [:ttaubert] (inactive)]

This is fixed when the patch for bug 624265 lands.

Comment 5

[Kevin Hanes]

bugspam (moving b9 to b10)

Comment 6

[Kevin Hanes]

bugspam (removing b9)

Comment 7

[Dietrich Ayala (:dietrich)]

This should block. I think that the trust in PB as a feature is seriously eroded if browser state is not properly restored when you come out of it. This bug gives the appearance of tab loss. Blocking+.

Comment 8

[George Carstoiu]

Build identifier: Mozilla/5.0 (X11; Linux i686; rv:2.0b9) Gecko/20100101 Firefox/4.0b9

Issue present in the above build.

Comment 9

[Tim Taubert [:ttaubert] (inactive)]

(In reply to comment #8)
> Issue present in the above build.

You're right. This unfortunately didn't make it into b9. It will be in b10 and presumably in the next nightly as we're going to land it soon.

Comment 10

[Tim Taubert [:ttaubert] (inactive)]

I'm sorry, I accidently removed the blocking+ :/ Can somebody please reset it to blocking+? Thanks!

Comment 11

[Tracy Walker [:tracy]]

verified with recent minefield nightly build