[Webkit-unassigned] [Bug 55129] New: is this a bug?

[bugzilla-daemon at webkit.org]

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

           Summary: is this a bug?
           Product: WebKit
           Version: 528+ (Nightly build)
          Platform: PC
               URL: http://trac.webkit.org/browser/trunk/Source/WebCore/lo
                    ader/FrameLoader.cpp
        OS/Version: Mac OS X 10.5
            Status: UNCONFIRMED
          Severity: Normal
          Priority: P2
         Component: WebCore Misc.
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: elephoenix at gmail.com


Hi,

Source code url:
http://trac.webkit.org/browser/trunk/Source/WebCore/loader/FrameLoader.cpp

2326    
2327        // FIXME: POST documents are always Reloads, but their subresources should still be Revalidate.
2328        // If we bring the CachePolicy.h and ResourceRequest cache policy enums in sync with each other and
2329        // remember "Revalidate" in ResourceRequests, we can remove this "POST" check and return either "Reload"
2330        // or "Revalidate" if the DocumentLoader was requested with either.
2331        const ResourceRequest& request(documentLoader()->request());
2332        if (request.cachePolicy() == ReloadIgnoringCacheData && !equalIgnoringCase(request.httpMethod(), "post"))

I think the line 2332 should change to 

-2332        if (request.cachePolicy() == ReloadIgnoringCacheData && !equalIgnoringCase(request.httpMethod(), "post"))
+2332        if (request.cachePolicy() == ReloadIgnoringCacheData && equalIgnoringCase(request.httpMethod(), "post"))

I'm not sure this is a bug or not. can someone take a look at this issue?
Thanks.

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