Do not treat substitute-data resources as having successful HTTP status codes.

Do not treat substitute-data resources as having successful HTTP status codes.

Some embedders of Blink (Chrome in particular) rely on error pages having an HTTP
status code of `0`. [1] changed this behavior to synthesize a 200 status code. This
patch changes that behavior back to the original expectation, matching the
expectation that non-network requests don't have network status information.

[1]: https://chromium.googlesource.com/chromium/blink/+/63b8349e922a3691bb77f74d7b2f143a5f023626

BUG=656845
Review-Url: https://codereview.chromium.org/2676443004
Cr-Commit-Position: refs/heads/master@{#455810}
(cherry picked from commit 1709b0599a7329585a3d781b21b1cbbb15379d7c)

Review-Url: https://codereview.chromium.org/2750433002 .
Cr-Commit-Position: refs/branch-heads/3029@{#141}
Cr-Branched-From: 939b32ee5ba05c396eef3fd992822fcca9a2e262-refs/heads/master@{#454471}
Committed: https://chromium.googlesource.com/chromium/src/+/786e9d4707ec38cdb51eef38f6c635028e94b046

[Mike West, 2017-03-13 07:51:50]

Description was changed from

==========
Do not treat substitute-data resources as having successful HTTP status codes.

Some embedders of Blink (Chrome in particular) rely on error pages having an
HTTP
status code of `0`. [1] changed this behavior to synthesize a 200 status code.
This
patch changes that behavior back to the original expectation, matching the
expectation that non-network requests don't have network status information.

[1]:
https://chromium.googlesource.com/chromium/blink/+/63b8349e922a3691bb77f74d7b...

BUG=656845

Review-Url: https://codereview.chromium.org/2676443004
Cr-Commit-Position: refs/heads/master@{#455810}
(cherry picked from commit 1709b0599a7329585a3d781b21b1cbbb15379d7c)
==========

to

==========
Do not treat substitute-data resources as having successful HTTP status codes.

Some embedders of Blink (Chrome in particular) rely on error pages having an
HTTP
status code of `0`. [1] changed this behavior to synthesize a 200 status code.
This
patch changes that behavior back to the original expectation, matching the
expectation that non-network requests don't have network status information.

[1]:
https://chromium.googlesource.com/chromium/blink/+/63b8349e922a3691bb77f74d7b...

BUG=656845

Review-Url: https://codereview.chromium.org/2676443004
Cr-Commit-Position: refs/heads/master@{#455810}
(cherry picked from commit 1709b0599a7329585a3d781b21b1cbbb15379d7c)

Review-Url: https://codereview.chromium.org/2750433002 .
Cr-Commit-Position: refs/branch-heads/3029@{#141}
Cr-Branched-From:
939b32ee5ba05c396eef3fd992822fcca9a2e262-refs/heads/master@{#454471}
Committed:
https://chromium.googlesource.com/chromium/src/+/786e9d4707ec38cdb51eef38f6c6...
==========

[Mike West, 2017-03-13 07:51:52]

Committed patchset #1 (id:1) manually as
786e9d4707ec38cdb51eef38f6c635028e94b046 (tree was closed).