Closed Bug 1619764 Opened 5 years ago Closed 5 years ago

Make Http3Session easier to understand

Categories

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

Product:
Core
Component:
Networking: HTTP
Type:
enhancement

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla75
Tracking Flags:
Tracking Status
firefox75 --- fixed

People

(Reporter: dragana, Assigned: dragana)

References

(Blocks 1 open bug)

Details

(Whiteboard: [necko-triaged])

Attachments

(1 file)

Bug 1619764 - Move some parts of HttpConnectionUDP to Http3Session. r=valentin
47 bytes, text/x-phabricator-request
Details | Review

Now that we have HttpConnectionUDP we can make Http3Session and HttpConnectionUDP code easier to understand.

We do not need:

  • mSocketIn/OutCondition - this can be properly handled by Http3Session
  • We do not need to loop in HttpConnectionUDP::OnSocketWritable/Readable -> this can be done in Http3Session
  • We do not need to handle tls hsandshake separately (EnsureNPNComplete), this was needed for http/2 vs http1.1
Pushed by ddamjanovic@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/583ce79ba232 Move some parts of HttpConnectionUDP to Http3Session. r=valentin

https://hg.mozilla.org/mozilla-central/rev/583ce79ba232

Status: NEW → RESOLVED
Closed: 5 years ago
status-firefox75: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla75
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: