Closed Bug 958712 Opened 6 years ago Closed 5 years ago

HTTP/2 push continuations not completely implemented

Categories

(Core :: Networking: HTTP, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla38

People

(Reporter: u408661, Assigned: u408661)

References

Details

(Whiteboard: [spdy])

Attachments

(3 files)

See bug 950768 comment 25 for the details. Basically, we're missing some queuing logic for push continuations like we have for pull header continuations. We should fix that, and remove the check about kFlag_END_PUSH_PROMISE from around line 1340 of Http2Session.cpp so that we fully support push header continuations.
Assignee: nobody → hurley
Attachment #8557264 - Flags: review?(mcmanus)
Attachment #8557265 - Flags: review?(mcmanus)
Attachment #8557266 - Flags: review?(mcmanus)
mozreview doesn't like my multi-bug series of patches, so it's back to splinter for this one. These patches (specifically the one that adds tests) depend on the patches in bug 1127568
Depends on: 1127568
Attachment #8557264 - Flags: review?(mcmanus) → review+
Attachment #8557266 - Flags: review?(mcmanus) → review+
Attachment #8557265 - Flags: review?(mcmanus) → review+
You need to log in before you can comment on or make changes to this bug.