[webkit-reviews] review granted: [Bug 132523] http/tests/security/xss-DENIED-xsl-document-redirect.xml fails with NetworkProcess : [Attachment 231599] Patch

[bugzilla-daemon at webkit.org]

Alexey Proskuryakov <ap at webkit.org> has granted Myles C. Maxfield
<mmaxfield at apple.com>'s request for review:
Bug 132523: http/tests/security/xss-DENIED-xsl-document-redirect.xml fails with
NetworkProcess
https://bugs.webkit.org/show_bug.cgi?id=132523

Attachment 231599: Patch
https://bugs.webkit.org/attachment.cgi?id=231599&action=review

------- Additional Comments from Alexey Proskuryakov <ap at webkit.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=231599&action=review


r=me assuming there is a good response to the question below. Otherwise, please
remove the httpStatusCode check.

> Source/WebCore/loader/DocumentThreadableLoader.cpp:413
> +    // If we have an HTTP response, then it wasn't a network error (even if
!error.isNull()).

Why? I don't see how this can happen.