806697 - Move browser_privatebrowsing_sslsite_transition.js to the obsolete directory

Status: RESOLVED FIXED

(Firefox :: Private Browsing, defect)

Description

[(no longer active)]

This test does not really apply to the new world, and we should move it to the obsolete directory.

Comment 1

[(no longer active)]

Attachment: Patch (v1)

Comment 2

[Josh Matthews [:jdm]]

Comment on attachment 676456 [details] [diff] [review]
Patch (v1)

It may be be desirable to ensure that SSL page loads are unaffected by our PB session cleanup code that will run after bug 769288 lands.

Comment 3

[(no longer active)]

The backstory of this code is bug 463256, where we added a terrible hack and wrote a test for the bug as described in bug 463256 comment 0.  This was a workaround to what was discovered to be bug 480619.  Then, over in bug 496335 we took out that hack but left the test in (mostly out of fear of the unknown!).  The bug in question came from the code which you took out in bug 769283.  So, now all traces of this problem are gone, especially after bug 769283 where the basic thing causing this was reworked.

We can definitely port some simulation of this test to the new APIs (by opening a PB window and closing it immediately), I guess.  But I don't see the point as it will be testing something else, entirely random.  Thus I question the value of existence of this test post per-window PB.  If you disagree, please let me know.

(/me shivers from the rush of memories of bug 463256)

Comment 4

[Josh Matthews [:jdm]]

Comment on attachment 676456 [details] [diff] [review]
Patch (v1)

This sounds ok to me, in that case.

Comment 5

[(no longer active)]

https://hg.mozilla.org/integration/mozilla-inbound/rev/10eca8d80a84

Comment 6

[Ed Morley [:emorley]]

https://hg.mozilla.org/mozilla-central/rev/10eca8d80a84