1608967 - Adapt necko to new neqo

Status: RESOLVED FIXED

(Core :: Networking: HTTP, task, P1)

Description

[Dragana Damjanovic [:dragana]]

• process_timer used to be called by process_output, that is not the case any more, therefore we need to call it for necko,
• in ProcessEvents we should not close the session, we should return an error and let nsHttpConnection or ProcessOutputAndEvents closes it. (this used to work with current code, something has change in neqo, it think it was about closing and closed state)
• we need to call ProcessOutput in WriteSegmentsAgain because neqo processing has change (I will look into if this split process_output, process_input, process_http3 and process_timer is the best, it is a bit confusing how we call it from necko -> this will be a separate bug)

Comment 1

[Dragana Damjanovic [:dragana]]

Attachment: Adapt necko to new neqo

• process_timer used to be called by process_output, that is not the case any more, therefore we nee$
• how closing/closed event has change: on idle-timeout only closed even is called which means that neqo connection is completely closed and should not be kept alive.

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/f2dfd755ee66
Adapt necko to new neqo r=michal

Comment 3

[Oana Pop-Rus]

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