1646318 - test_post and test_patch in netwerk/test/unit/test_http3.js time out

Status: RESOLVED FIXED

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

Description

[Michal Novotny [:michal]]

Bug 1644374 fixed an assertion when running these tests, but they still don't pass. The connection is blocked when we try to write POST data and we never try to write the data again. In most cases we never receive event Http3Event::Tag::DataWritable although this code in neqo is executed. Even if the event is received and mReadyForWrite.Push(stream) is called we don't try to write the POST data again.

Comment 1

[Dragana Damjanovic [:dragana]]

Attachment: Split the sending side states and the receiving side states of a Http3Stream and allow to write and read at the same time. Also stop writing data as soon as the Http3Stream has received the response or has been reset. A NS_BASE_STREAM_WOULD_BLOCK...

...should be ignored after calling ReadSegments. r=michal

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/e84b5a5ea42b
Split the sending side states and the receiving side states of a Http3Stream and allow to write and read at the same time. Also stop writing data as soon as the Http3Stream has received the response or has been reset. A NS_BASE_STREAM_WOULD_BLOCK... r=michal,necko-reviewers

Comment 3

[Noemi Erli[:noemi_erli]]

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