[Fetch API] Fix leaky layout tests.

[Fetch API] Fix leaky layout tests.

Chrome implementation of Fetch API keep response alive while response.body
is readable and locked by someone. This CL fix some layout tests
to avoid leaks. See https://codereview.chromium.org/1899873006/ for
more information.

This CL also fixes 'acquiring a reader should not set bodyUsed'
in stream-reader.js, because it was wrong but the assertion failure
was caught and not reported.

BUG=596832
Committed: https://crrev.com/40a9fad2888f2e0395aac0fa1eae7a7c1ac0e262
Cr-Commit-Position: refs/heads/master@{#389751}

[yhirano, 2016-04-25 02:02:26]

Description was changed from

==========
[Fetch API] Fix leaky layout tests.

Chrome implementation of Fetch API keep response alive while response.body
is readable and locked by someone. Fix such tests to avoid leaks. See
https://codereview.chromium.org/1899873006/ for more information.

BUG=596832
==========

to

==========
[Fetch API] Fix leaky layout tests.

Chrome implementation of Fetch API keep response alive while response.body
is readable and locked by someone. Fix such tests to avoid leaks. See
https://codereview.chromium.org/1899873006/ for more information.

This CL also fixes 'acquiring a reader should not set bodyUsed'
in stream-reader.js/, because it's wrong but the assertion failure
by caught and not reported.

BUG=596832
==========

[yhirano, 2016-04-25 02:27:43]

yhirano@chromium.org changed reviewers:
+ tyoshino@chromium.org

[Yuki, 2016-04-25 06:57:09]

yukishiino@chromium.org changed reviewers:
+ yukishiino@chromium.org

[Yuki, 2016-04-25 06:57:10]

https://codereview.chromium.org/1920703002/diff/20001/third_party/WebKit/Layo...
File third_party/WebKit/LayoutTests/http/tests/fetch/script-tests/body-mixin.js
(right):

https://codereview.chromium.org/1920703002/diff/20001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/http/tests/fetch/script-tests/body-mixin.js:265:
// TODO: yhirano Use finally when it's available. Ditto below.
typo: TODO(yhirano): ...

[yhirano, 2016-04-25 07:01:51]

https://codereview.chromium.org/1920703002/diff/20001/third_party/WebKit/Layo...
File third_party/WebKit/LayoutTests/http/tests/fetch/script-tests/body-mixin.js
(right):

https://codereview.chromium.org/1920703002/diff/20001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/http/tests/fetch/script-tests/body-mixin.js:265:
// TODO: yhirano Use finally when it's available. Ditto below.
On 2016/04/25 06:57:10, Yuki wrote:
> typo: TODO(yhirano): ...

Done.

[yhirano, 2016-04-25 07:02:40]

Description was changed from

==========
[Fetch API] Fix leaky layout tests.

Chrome implementation of Fetch API keep response alive while response.body
is readable and locked by someone. Fix such tests to avoid leaks. See
https://codereview.chromium.org/1899873006/ for more information.

This CL also fixes 'acquiring a reader should not set bodyUsed'
in stream-reader.js/, because it's wrong but the assertion failure
by caught and not reported.

BUG=596832
==========

to

==========
[Fetch API] Fix leaky layout tests.

Chrome implementation of Fetch API keep response alive while response.body
is readable and locked by someone. Fix such tests to avoid leaks. See
https://codereview.chromium.org/1899873006/ for more information.

This CL also fixes 'acquiring a reader should not set bodyUsed'
in stream-reader.js, because it was wrong but the assertion failure
was caught and not reported.

BUG=596832
==========

[yhirano, 2016-04-25 07:35:47]

Description was changed from

==========
[Fetch API] Fix leaky layout tests.

Chrome implementation of Fetch API keep response alive while response.body
is readable and locked by someone. Fix such tests to avoid leaks. See
https://codereview.chromium.org/1899873006/ for more information.

This CL also fixes 'acquiring a reader should not set bodyUsed'
in stream-reader.js, because it was wrong but the assertion failure
was caught and not reported.

BUG=596832
==========

to

==========
[Fetch API] Fix leaky layout tests.

Chrome implementation of Fetch API keep response alive while response.body
is readable and locked by someone. This CL fix some layout tests
to avoid leaks. See https://codereview.chromium.org/1899873006/
for more information.

This CL also fixes 'acquiring a reader should not set bodyUsed'
in stream-reader.js, because it was wrong but the assertion failure
was caught and not reported.

BUG=596832
==========

[yhirano, 2016-04-25 07:36:09]

Description was changed from

==========
[Fetch API] Fix leaky layout tests.

Chrome implementation of Fetch API keep response alive while response.body
is readable and locked by someone. This CL fix some layout tests
to avoid leaks. See https://codereview.chromium.org/1899873006/
for more information.

This CL also fixes 'acquiring a reader should not set bodyUsed'
in stream-reader.js, because it was wrong but the assertion failure
was caught and not reported.

BUG=596832
==========

to

==========
[Fetch API] Fix leaky layout tests.

Chrome implementation of Fetch API keep response alive while
response.body is readable and locked by someone. This CL fix some
layout tests to avoid leaks. See https://codereview.chromium.org/1899873006/
for more information.

This CL also fixes 'acquiring a reader should not set bodyUsed'
in stream-reader.js, because it was wrong but the assertion failure
was caught and not reported.

BUG=596832
==========

[yhirano, 2016-04-25 07:36:33]

Description was changed from

==========
[Fetch API] Fix leaky layout tests.

Chrome implementation of Fetch API keep response alive while
response.body is readable and locked by someone. This CL fix some
layout tests to avoid leaks. See https://codereview.chromium.org/1899873006/
for more information.

This CL also fixes 'acquiring a reader should not set bodyUsed'
in stream-reader.js, because it was wrong but the assertion failure
was caught and not reported.

BUG=596832
==========

to

==========
[Fetch API] Fix leaky layout tests.

Chrome implementation of Fetch API keep response alive while response.body
is readable and locked by someone. This CL fix some layout tests
to avoid leaks. See https://codereview.chromium.org/1899873006/ for
more information.

This CL also fixes 'acquiring a reader should not set bodyUsed'
in stream-reader.js, because it was wrong but the assertion failure
was caught and not reported.

BUG=596832
==========

[tyoshino (SeeGerritForStatus), 2016-04-26 08:15:59]

lgtm

https://codereview.chromium.org/1920703002/diff/40001/third_party/WebKit/Layo...
File
third_party/WebKit/LayoutTests/http/tests/fetch/script-tests/stream-reader.js
(right):

https://codereview.chromium.org/1920703002/diff/40001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/http/tests/fetch/script-tests/stream-reader.js:45:
return res.text().then(unreached_rejection(t), function() {
unreached_fulfillment

[yhirano, 2016-04-26 11:15:40]

https://codereview.chromium.org/1920703002/diff/40001/third_party/WebKit/Layo...
File
third_party/WebKit/LayoutTests/http/tests/fetch/script-tests/stream-reader.js
(right):

https://codereview.chromium.org/1920703002/diff/40001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/http/tests/fetch/script-tests/stream-reader.js:45:
return res.text().then(unreached_rejection(t), function() {
On 2016/04/26 08:15:58, tyoshino wrote:
> unreached_fulfillment

Done.

[yhirano, 2016-04-26 11:15:47]

The CQ bit was checked by yhirano@chromium.org

[yhirano, 2016-04-26 11:15:48]

The patchset sent to the CQ was uploaded after l-g-t-m from
tyoshino@chromium.org
Link to the patchset: https://codereview.chromium.org/1920703002/#ps60001
(title: " ")

[commit-bot: I haz the power, 2016-04-26 11:15:59]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1920703002/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1920703002/60001

[commit-bot: I haz the power, 2016-04-26 12:15:50]

Description was changed from

==========
[Fetch API] Fix leaky layout tests.

Chrome implementation of Fetch API keep response alive while response.body
is readable and locked by someone. This CL fix some layout tests
to avoid leaks. See https://codereview.chromium.org/1899873006/ for
more information.

This CL also fixes 'acquiring a reader should not set bodyUsed'
in stream-reader.js, because it was wrong but the assertion failure
was caught and not reported.

BUG=596832
==========

to

==========
[Fetch API] Fix leaky layout tests.

Chrome implementation of Fetch API keep response alive while response.body
is readable and locked by someone. This CL fix some layout tests
to avoid leaks. See https://codereview.chromium.org/1899873006/ for
more information.

This CL also fixes 'acquiring a reader should not set bodyUsed'
in stream-reader.js, because it was wrong but the assertion failure
was caught and not reported.

BUG=596832
==========

[commit-bot: I haz the power, 2016-04-26 12:15:51]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-04-26 12:16:56]

Description was changed from

==========
[Fetch API] Fix leaky layout tests.

Chrome implementation of Fetch API keep response alive while response.body
is readable and locked by someone. This CL fix some layout tests
to avoid leaks. See https://codereview.chromium.org/1899873006/ for
more information.

This CL also fixes 'acquiring a reader should not set bodyUsed'
in stream-reader.js, because it was wrong but the assertion failure
was caught and not reported.

BUG=596832
==========

to

==========
[Fetch API] Fix leaky layout tests.

Chrome implementation of Fetch API keep response alive while response.body
is readable and locked by someone. This CL fix some layout tests
to avoid leaks. See https://codereview.chromium.org/1899873006/ for
more information.

This CL also fixes 'acquiring a reader should not set bodyUsed'
in stream-reader.js, because it was wrong but the assertion failure
was caught and not reported.

BUG=596832

Committed: https://crrev.com/40a9fad2888f2e0395aac0fa1eae7a7c1ac0e262
Cr-Commit-Position: refs/heads/master@{#389751}
==========

[commit-bot: I haz the power, 2016-04-26 12:16:57]

Patchset 4 (id:??) landed as
https://crrev.com/40a9fad2888f2e0395aac0fa1eae7a7c1ac0e262
Cr-Commit-Position: refs/heads/master@{#389751}