New issue
Advanced search Search tips

Issue 656601 link

Starred by 1 user
Status: Fixed
Owner:
yhirano@chromium.org
Closed: Oct 2016
Cc:
domenic@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Response.body from Network should be rejected by TypeError, not DOMException

Project Member Reported by yhirano@chromium.org, Oct 17 2016 
Run the web-platform-test: fetch/api/basic/error-after-response.html

See the following result:

assert_throws: function "function () { throw e }" threw object "NetworkError: network error" ("NetworkError") expected object "TypeError" ("TypeError")

Note that Chrome should include https://codereview.chromium.org/2323623002/. Otherwise another error is reported.

See also: https://github.com/w3c/web-platform-tests/pull/3951
Project Member

Comment 1 by bugdroid1@chromium.org, Oct 24 2016 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/0856679f482230c36c3e1261247f30a0dcfa916e

commit 0856679f482230c36c3e1261247f30a0dcfa916e
Author: yhirano <yhirano@chromium.org>
Date: Mon Oct 24 12:27:48 2016

[Fetch API] Streaming error should be reported as a TypeError

When the browser encounters an error while reading a response body, it should
error the body stream with a TypeError.

BUG= 656601 

Review-Url: https://codereview.chromium.org/2439673002
Cr-Commit-Position: refs/heads/master@{#427052}

[modify] https://crrev.com/0856679f482230c36c3e1261247f30a0dcfa916e/third_party/WebKit/LayoutTests/http/tests/fetch/script-tests/body-mixin.js
[modify] https://crrev.com/0856679f482230c36c3e1261247f30a0dcfa916e/third_party/WebKit/LayoutTests/http/tests/fetch/script-tests/stream-reader.js
[modify] https://crrev.com/0856679f482230c36c3e1261247f30a0dcfa916e/third_party/WebKit/Source/modules/fetch/BodyStreamBuffer.cpp

Comment 2 by yhirano@chromium.org, Oct 24 2016

Status: Fixed (was: Assigned)
►

Sign in to add a comment