675617 - Fix browser_installssl.js to support keep-alive connections.

Status: RESOLVED FIXED

(Testing :: General, defect)

Description

[Honza Bambas (:mayhemer)]

Attachment: v1

As part of the work to make httpd.js support keep-alive connections this test needs a fix.

When reusing keep-alive connections while adding/removing/adding cert exceptions, the test suffers from bug 471995 and fails.  To work that around we need to force connection close for the .rdf file being queried, what is simply done just by adding ^headers^ file with Connection: close header.

Comment 1

[Jeff Walden [:Waldo]]

Is it possible to make the test not depend on the connection always being immediately and promptly closed?  That ^headers^ will include the Connection header (and various other transport-level-y headers) is arguably a bug, and it's possible at some point I'll do what XMLHttpRequest does and blacklist certain headers from being set there.

Comment 2

[Honza Bambas (:mayhemer)]

According the patches from the httpd.js keep-alive bug (at this moment unreviewed and unsettled) I would like to allow files say it wants to close the connection this way.  Also request handlers should be able to do it by adding the header.

The patch in this bug is mostly a workaround for bug 471995, which I want to start working on soon.  So the ^headers^ files will go away with fixing that bug.

Comment 3

[Honza Bambas (:mayhemer)]

http://hg.mozilla.org/integration/mozilla-inbound/rev/efb4b265e268

Comment 4

[Marco Bonardo [:mak]]

http://hg.mozilla.org/mozilla-central/rev/efb4b265e268