1790401 - Outgoing WebTransportStream management

Status: RESOLVED FIXED

(Core :: Networking, task, P2)

Description

[Kershaw Chang [:kershaw]]

We will implement the function CreateOutgoingBidirectionalStream and CreateOutgoingUnidirectionalStream in nsIWebTransport interface below.

interface nsIWebTransport {
  // When called, perform steps in "Initialization WebTransport over HTTP".
  void AsyncConnect(in nsIURI aURI,
                    in OriginAttributes aOriginAttributes,
                    in WebTransportSessionEventListener aListener);
  // Asynchronously get states.
  void GetStats();

  // Close the session.
  void CloseSession(in uint32_t aErrorCode,
                    in ACString aReason);

  // Create and open a new WebTransport stream. The function requires
  // the nsIWebTransportStreamCallback parameter. The result will be
  // delivered via the callback.
  void CreateOutgoingBidirectionalStream(in nsIWebTransportStreamCallback aCallback);
  void CreateOutgoingUnidirectionalStream(in nsIWebTransportStreamCallback aCallback);

  bool SendDatagram(in AUTF8String aDatagram);
};

// This interface is used as a callback when creating an outgoing
// unidirectional or bidirectional stream.
interface nsIWebTransportStreamCallback {
  void OnStreamReady(in nsIWebTransportSendStream aStream);
  
  void OnError(in uint8_t aError);
};

Comment 1

[Kershaw Chang [:kershaw]]

Attachment: Bug 1790401 - Implement the creation of outgoing WebTransport stream, r=#necko

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/ce2291439b81
Implement the creation of outgoing WebTransport stream, r=necko-reviewers,valentin

Comment 3

[Marian-Vasile Laza]

Backed out 2 changesets (Bug 1790401, Bug 1790402) for causing bustages on WebTransportStreamProxy.h.
Backout link
Push with failures <--> Bp-hybrid
Failure Log
Also Bb Failure Log

Comment 4

[Kershaw Chang [:kershaw]]

https://treeherder.mozilla.org/jobs?repo=try&revision=c1aea5abfc134adf1838ba90719f81ca5054e918&selectedTaskRun=WUPRNA5lSI6-f2_VrttnRQ.1

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/1c84926b18f8
Implement the creation of outgoing WebTransport stream, r=necko-reviewers,valentin

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/022dc09c27b1
static_cast<uint32_t> an nsresult formated to hex with PRIx32 to fix build bustage with gcc -Werror=format=

Comment 7

[Iulian Moraru]

https://hg.mozilla.org/mozilla-central/rev/1c84926b18f8
https://hg.mozilla.org/mozilla-central/rev/022dc09c27b1