[Fetch] Omit console error messages for cancellation, timeout, and non-BlinkInternal

[Fetch] Omit console error messages for cancellation, timeout, and non-BlinkInternal

This CL aims to reduce duplicated console error messages.

This CL also suppresses error messages when loading via WebDataConsumerHandle is cancelled.
In such cases, didFail() is called by https://codereview.chromium.org/1186053004/ but we don't want to print error messages.

The condition for printing error messages is based on the condition used by XHR (XMLHttpRequest::didFail()), except for we also output error messages for didFailRedirectCheck().

BUG=500258, 480746
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=197376

[hiroshige, 2015-06-17 12:48:21]

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

[hiroshige, 2015-06-17 12:48:22]

PTAL.

[hiroshige, 2015-06-17 18:29:54]

If this code is fine I'll add layout tests under http/tests/fetch/chromium in
this CL.

[yhirano, 2015-06-18 05:22:44]

Looks good.

[hiroshige, 2015-06-18 08:15:24]

Added a layout test.

[yhirano, 2015-06-18 08:35:53]

https://codereview.chromium.org/1195453005/diff/20001/LayoutTests/http/tests/...
File LayoutTests/http/tests/fetch/chromium/error-messages.html (right):

https://codereview.chromium.org/1195453005/diff/20001/LayoutTests/http/tests/...
LayoutTests/http/tests/fetch/chromium/error-messages.html:29: // outside
modules/fetch, and thus we don't need messages from modules/fetch.
Is this comment correct?

[hiroshige, 2015-06-18 08:40:47]

https://codereview.chromium.org/1195453005/diff/20001/LayoutTests/http/tests/...
File LayoutTests/http/tests/fetch/chromium/error-messages.html (right):

https://codereview.chromium.org/1195453005/diff/20001/LayoutTests/http/tests/...
LayoutTests/http/tests/fetch/chromium/error-messages.html:29: // outside
modules/fetch, and thus we don't need messages from modules/fetch.
On 2015/06/18 08:35:53, yhirano wrote:
> Is this comment correct?
Oh, corrected.

[yhirano, 2015-06-18 10:57:53]

lgtm

[hiroshige, 2015-06-18 11:07:45]

The CQ bit was checked by hiroshige@chromium.org

[commit-bot: I haz the power, 2015-06-18 11:08:38]

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

[commit-bot: I haz the power, 2015-06-18 13:31:08]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-06-18 13:31:09]

Try jobs failed on following builders:
  win_blink_rel on tryserver.blink (JOB_FAILED,
http://build.chromium.org/p/tryserver.blink/builders/win_blink_rel/builds/67211)

[hiroshige, 2015-06-18 16:19:36]

The CQ bit was checked by hiroshige@chromium.org

[commit-bot: I haz the power, 2015-06-18 16:19:53]

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

[commit-bot: I haz the power, 2015-06-18 17:17:36]

Committed patchset #3 (id:40001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=197376