743747 - fix bug 603514 landing

Status: RESOLVED FIXED

(Core :: Networking: HTTP, defect)

Description

[Patrick McManus [:mcmanus]]

603514 had merge conflicts with spdy code. The primary effect of this was I didn't merge a couple prefs that were instead left hardcoded with different default values.

Comment 1

[Patrick McManus [:mcmanus]]

Attachment: patch 0

Comment 2

[Honza Bambas (:mayhemer)]

Comment on attachment 613348 [details] [diff] [review]
patch 0

Review of attachment 613348 [details] [diff] [review]:
-----------------------------------------------------------------

r=honzab

Telemetry is the next step.

::: netwerk/protocol/http/nsHttpHandler.h
@@ +242,5 @@
> +    {
> +        return mCapabilities & NS_HTTP_ALLOW_PIPELINING;
> +    }
> +
> +    bool GetPipelineRescheduleOnTimeout()

Maybe call this GetAllowPipelineRescheduleOnTimeout()  But this way it reflects the pref..  Up to you.

Comment 3

[Patrick McManus [:mcmanus]]

(In reply to Honza Bambas (:mayhemer) from comment #2)

> 
> Telemetry is the next step.
> 

yes.. probably something that tracks all the different red event types.

Comment 4

[Patrick McManus [:mcmanus]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/e819f766c586

Comment 5

[(no longer active)]

https://hg.mozilla.org/mozilla-central/rev/e819f766c586