Source/WebCore: [chromium] Some SubstituteData loads broken after r121912
authorjaphet@chromium.org <japhet@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 6 Sep 2012 04:48:20 +0000 (04:48 +0000)
committerjaphet@chromium.org <japhet@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 6 Sep 2012 04:48:20 +0000 (04:48 +0000)
commit118d5ee9e769789873f810bb546074314e30c48d
tree0d21ba071d584d2ea30344e8b969913ef50a833f
parent21fd51acde3668a6210a3dd153bff8df672fea4c
Source/WebCore: [chromium] Some SubstituteData loads broken after r121912
https://bugs.webkit.org/show_bug.cgi?id=91685

Reviewed by Adam Barth.

Test: WebFrameTest.ReplaceNavigationAfterHistoryNavigation in chromium's webkit_unit_tests.

* loader/DocumentLoader.cpp:
(WebCore::DocumentLoader::commitData): receivedFirstData() should be called exactly once per load,
    on the first commit. I had mistakely assumed in r121912 that isReplacing() was true only for multipart
    loads, and only after the first commit (chromium uses it for some SubstituteData loads to ensure the error page
    replaces the failed load). We need to check whether we are loading multipart content before assuming isReplacing()
    will tell us what we need to know.

Source/WebKit/chromium: Test for https://bugs.webkit.org/show_bug.cgi?id=91685

Reviewed by Adam Barth.

* tests/WebFrameTest.cpp:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@127690 268f45cc-cd09-0410-ab3c-d52691b4dbfc
Source/WebCore/ChangeLog
Source/WebCore/loader/DocumentLoader.cpp
Source/WebKit/chromium/ChangeLog
Source/WebKit/chromium/tests/WebFrameTest.cpp