Add per-stream getStats().

[jan-ivar]

Fixes #372.

---

Preview | Diff

[jan-ivar]

@ylafon @dontcallmedom any idea why these checks are failing?

Error: w3c/spec-prod/v2/action.yml (Line: 113, Col: 24):
Error: The template is not valid. w3c/spec-prod/v2/action.yml (Line: 113, Col: 24): Error reading JToken from JsonReader. Path '', line 0, position 0.

[martinthomson]

bytesAcknowledged ?

[jan-ivar]

Meeting:

• No objections. @yutakahirano to ping @ricea to have a look on the subclassing.

[dontcallmedom]

@ylafon @dontcallmedom any idea why these checks are failing?

Error: w3c/spec-prod/v2/action.yml (Line: 113, Col: 24):
Error: The template is not valid. w3c/spec-prod/v2/action.yml (Line: 113, Col: 24): Error reading JToken from JsonReader. Path '', line 0, position 0.

The actual error is hidden above that bogus error message:

  LINK ERROR: No 'idl' refs found for '[[InternalStream]]'.

{{SendStream/[[InternalStream]]}}
✘ Did not generate, due to fatal errors

[ricea]

Subclassing looks okay to me.

The IDL for createUnidirectionalStream() needs to be updated to reflect its new return type.

[jan-ivar]

@martinthomson does this LGTY to merge, or should we keep bikeshedding?

[jan-ivar]

With the benfit of some hindsight, I'm questioning whether our concerns over understandability justify the awkward naming:

dictionary WebTransportSendStreamStats {
  DOMHighResTimeStamp timestamp;
  unsigned long long amountWrittenTo;
  unsigned long long sentProgress;
  unsigned long long acknowledgedProgress;
};

dictionary WebTransportReceiveStreamStats {
  DOMHighResTimeStamp timestamp;
  unsigned long long receivedProgress;
  unsigned long long amountReadFrom;
};

The following does seem simpler:

dictionary WebTransportSendStreamStats {
  DOMHighResTimeStamp timestamp;
  unsigned long long bytesWritten;
  unsigned long long bytesSent;
  unsigned long long bytesAcknowledged;
};

dictionary WebTransportReceiveStreamStats {
  DOMHighResTimeStamp timestamp;
  unsigned long long bytesReceived;
  unsigned long long bytesRead;
};

[jan-ivar]

Meeting:

• Add note to sentProgress to mention this is progress of app data sent on a single stream only and does not include any network overhead.
• Add a note that acknowledgedProgress will match sentProgress over HTTP/2
• Reorder received ahead of amountReadFrom
• Go with the latter naming.