[Webkit-unassigned] [Bug 60206] Proper handling of HTTP 204 (No Content) status code

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=60206





--- Comment #4 from Luiz Agostini <luiz at webkit.org>  2011-05-05 11:03:00 PST ---
(In reply to comment #3)
> Here is how I tested:
> 1. Opened http://www.apple.com in Safari 5.0.5 (also tested with ToT).
> 2. Once that finished loading, changed address in address bar to <http://127.0.0.1:8000/navigation/resources/response204.pl>, and hit Enter.
> 
> Results: The address reverted to http://www.apple.com, page content was not affected.

Good. I was expecting this test to fail in some platforms but it seems that Mac is ok with this 204 handling.

> 
> This looks like expected behavior. I haven't investigated what's going on in the test, but there are no known issues with 204 responses (we track bug 4858 here, but it's not really about 204, it's about stopping timers too early on any load).

Yeah, it may be then that I did not provide the proper test. 

The response204.is very simple, I will copy it here:

    <script>
        if (window.layoutTestController) {
            layoutTestController.dumpAsText();
            layoutTestController.dumpBackForwardList();
            layoutTestController.queueLoad("resources/response204.pl");
        }
    </script>

And this is the result that I was expecting:

    ============== Back Forward List ==============
    curr->  http://127.0.0.1:8000/navigation/response204.html  **nav target**
    ===============================================

Meaning that the response from resources/response204.pl was ignored.

But it was not the result for SnowLeopard as you can see here http://build.webkit.org/results/SnowLeopard%20Intel%20Release%20%28Tests%29/r85783%20%2828972%29/http/tests/navigation/response204-diffs.txt :

    --- /Volumes/Data/WebKit-BuildSlave/snowleopard-intel-release-tests/build/layout-test-results/http/tests/navigation/response204-expected.txt    2011-05-04 13:57:57.000000000 -0700
    +++ /Volumes/Data/WebKit-BuildSlave/snowleopard-intel-release-tests/build/layout-test-results/http/tests/navigation/response204-actual.txt    2011-05-04 13:57:57.000000000 -0700
    @@ -1,5 +1,6 @@


     ============== Back Forward List ==============
    -curr->  http://127.0.0.1:8000/navigation/response204.html  **nav target**
    +        http://127.0.0.1:8000/navigation/response204.html  **nav target**
    +curr->  http://127.0.0.1:8000/navigation/resources/response204.pl  **nav target**
     ===============================================

Nor for Leopard as you can see here http://build.webkit.org/results/Leopard%20Intel%20Release%20%28Tests%29/r85783%20%2831877%29/http/tests/navigation/response204-diffs.txt :

    --- /Volumes/Big/WebKit-BuildSlave/leopard-intel-release-tests/build/layout-test-results/http/tests/navigation/response204-expected.txt    2011-05-04 13:47:14.000000000 -0700
    +++ /Volumes/Big/WebKit-BuildSlave/leopard-intel-release-tests/build/layout-test-results/http/tests/navigation/response204-actual.txt    2011-05-04 13:47:14.000000000 -0700
    @@ -1,5 +1,6 @@


     ============== Back Forward List ==============
    -curr->  http://127.0.0.1:8000/navigation/response204.html  **nav target**
    +        http://127.0.0.1:8000/navigation/response204.html  **nav target**
    +curr->  http://127.0.0.1:8000/navigation/resources/response204.pl  **nav target**
     ===============================================

Is that test wrong?

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.