Skip to main content

Browse the archive

Revision f2ae7641fad03150b4238a8b34dc9bdfdf58c0dd authored by Adam Rice on 20 March 2018, 12:30:43 UTC, committed by Blink WPT Bot on 20 March 2018, 12:38:49 UTC 
Correctly reject in-progress body methods with AbortError
 
Prior to this change, response body methods such as
response.body.arrayBuffer() would reject with a TypeError if the fetch
was aborted. This change makes them correctly reject with an AbortError
instead.

This implements stage #3 of the "Body Cancellation" section of the
design doc:

https://docs.google.com/document/d/1OuoCG2uiijbAwbCw9jaS7tHEO0LBO_4gMNio1ox0qlY/edit#heading=h.fvc7d7q07oio

Bug: 817687
Change-Id: Ifde322d9c22485a8ba9d14fd4ffca65c4fb4745a
Reviewed-on: https://chromium-review.googlesource.com/954765
Commit-Queue: Adam Rice <ricea@chromium.org>
Reviewed-by: Matt Falkenhagen <falken@chromium.org>
Reviewed-by: Yutaka Hirano <yhirano@chromium.org>
Cr-Commit-Position: refs/heads/master@{#544335}
1 parent 7cb5db5
Raw File
.pyup.yml 
# search for requirement files
# default: True
# allowed: True, False
search: False

# Specify requirement files by hand, default is empty
# default: empty
# allowed: list
requirements:
 - tools/wptrunner/requirements_chrome_android.txt:
      update: all
      pin: True
 - tools/wptrunner/requirements_edge.txt:
      update: all
      pin: True
 - tools/wptrunner/requirements_opera.txt:
      update: all
      pin: True
 - tools/wptrunner/requirements_servo.txt:
      update: all
      pin: True
 - tools/wptrunner/requirements_safari.txt:
      update: all
      pin: True
 - tools/wptrunner/requirements_sauce.txt:
      update: all
      pin: True
 - tools/wptrunner/requirements_ie.txt:
      update: all
      pin: True
 - tools/wptrunner/requirements.txt:
      update: all
      pin: True
 - tools/wptrunner/requirements_firefox.txt:
      update: all
      pin: True
 - tools/wptrunner/requirements_chrome.txt:
      update: all
      pin: True
 - tools/wpt/requirements.txt:
      update: all
      pin: True
back to top