{Request, Response}.blob() should release its lock when resolved.

{Request, Response}.blob() should release its lock when resolved.

Request.blob() and Response.blob() should read all data from their body and
release the reader when reading is finished. Previously the implementation
acquired the lock permanently by mistake, so this CL fixes that.

BUG=477995
R=hiroshige@chromium.org

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=194190

[yhirano, 2015-04-17 09:35:12]

On 2015/04/17 09:30:34, yhirano wrote:

PS2 addressed the point pointed at
https://codereview.chromium.org/1088823002/#msg8 .

[hiroshige, 2015-04-17 09:45:17]

Should we have a test for Response?

[yhirano, 2015-04-21 06:47:37]

On 2015/04/17 09:45:17, hiroshige wrote:
> Should we have a test for Response?

Done.

[hiroshige, 2015-04-21 07:17:13]

lgtm.

https://codereview.chromium.org/1068783005/diff/60001/LayoutTests/http/tests/...
File LayoutTests/http/tests/fetch/script-tests/response.js (right):

https://codereview.chromium.org/1068783005/diff/60001/LayoutTests/http/tests/...
LayoutTests/http/tests/fetch/script-tests/response.js:318:
}).then(function(blob) {
Please add a comment that states what we are testing here.

[yhirano, 2015-04-22 02:35:30]

https://codereview.chromium.org/1068783005/diff/60001/LayoutTests/http/tests/...
File LayoutTests/http/tests/fetch/script-tests/response.js (right):

https://codereview.chromium.org/1068783005/diff/60001/LayoutTests/http/tests/...
LayoutTests/http/tests/fetch/script-tests/response.js:318:
}).then(function(blob) {
On 2015/04/21 07:17:13, hiroshige wrote:
> Please add a comment that states what we are testing here.

Done.

[yhirano, 2015-04-22 02:35:33]

The CQ bit was checked by yhirano@chromium.org

[yhirano, 2015-04-22 02:35:33]

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

[commit-bot: I haz the power, 2015-04-22 02:35:39]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1068783005/80001

[commit-bot: I haz the power, 2015-04-22 03:53:18]

Committed patchset #5 (id:80001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=194190