Canceling on Fetch reader fails with DOMException InvalidStateError

Fixed Issue #11862013

Details

Author
Kagami R.
Created
May 2, 2017
Privacy
This issue is public.
Found in
  • Microsoft Edge
Standard affected
Streams Standard

Found in build #
15.16184
Reports
Reported by 1 person

Sign in to watch or report this issue.

Steps to reproduce

const reader = stream.getReader();
const aborter = () => {
    //return reader.cancel(); // does not work on Edge 15
    reader.releaseLock();
    return stream.cancel();
}
let aborterId = setTimeout(aborter, 90000);

We recently tried transition from MSStream to Fetch stream on our UWP JavaScript app, and experienced a bug on reader.cancel()

Per the spec the reader.cancel() should work as same as reader.releaseLock() + stream.cancel() but the former fails, the latter succeeds. No idea why.

Also, the spec never refers InvalidStateError.


Repro: https://msedge-fetch-invalidstateerror.herokuapp.com/

  1. Click the button to cancel stream.
  2. Expected to stop as it does on Chrome but not.

Attachments

0 attachments

    Comments and activity

    • Changed Steps to Reproduce

    • Microsoft Edge Team

      Changed Assigned To to “James M.”

    • Hello,

      Thank you for providing this information about the issue. Unfortunately, we are unable to reproduce this problem in Edge with the information at hand. Please update this case with a sample project or webpage which shows the problem.

      Best Wishes,
      The MS Edge Team

    • Changed Steps to Reproduce

    • Hi, I updated the repro step.

    • Microsoft Edge Team

      Changed Assigned To to “Venkat K.”

      Changed Assigned To from “Venkat K.” to “Mohamed K.”

      Changed Status to “Confirmed”

    • Edge now exhibits same behavior as Opera does, probably fixed now :D

    • Microsoft Edge Team

      Changed Status from “Confirmed” to “Fixed”

    • This issue was fixed in the latest Windows insider preview build. Thank you for your feedback!

    You need to sign in to your Microsoft account to add a comment.

    Sign in