1372614 - Remove HTTP Pipelining preferences UI after bug 1340655 removed support for it

Status: RESOLVED FIXED

(SeaMonkey :: Preferences, enhancement)

Description

[rsx11m]

As frg pointed out, bug 1340655 has removed the pipelining feature for HTTP/1.1 which has been experimental for ages anyway and superseded by other methods.

Consequently, network.http.pipelining.* and network.http.proxy.pipelining do no longer have any function, yet we are exposing the top-level enabling prefs in the Advanced > HTTP Networking prefpane.

Charge of this bug is to remove those from pref-http.{xul,js,dtd} along with respective Help content in cs_nav_prefs_advanced.xhtml.

Comment 1

[rsx11m]

Attachment: Proposed patch

This removes the two checkboxes and related logic.

I have retained the thin separator between the protocol and UA-string options.

Asking IanN for review as it involves Help-content changes, updating HTTP/1.1 to be the "current" protocol rather than the "new" one. Also, let me know if you want any further changes while I'm there.

Comment 2

[Frank-Rainer Grahl (:frg)]

Comment on attachment 8877339 [details] [diff] [review]
Proposed patch

Stealing review. Tested and works as expected. Help content is and looks fine in my eyes.

Comment 3

[Frank-Rainer Grahl (:frg)]

https://hg.mozilla.org/comm-central/rev/a2a5296a88089da072a381f163ea78cc3ad89bc8

Comment 4

[Frank-Rainer Grahl (:frg)]

Target milestone 2.53. 

I think it can ride the trains. If not please ask for beta approval.

Comment 5

[rsx11m]

Thanks for pushing. Yes, given that 2.52 won't be released as a final, no need for beta.