Skip to content
  • vsevik@chromium.org's avatar
    Source/WebCore: Web Inspector: Incorrect XHR responses when two async xhrs are sent synchronously · dcc56abd
    vsevik@chromium.org authored
    https://bugs.webkit.org/show_bug.cgi?id=91630
    
    Patch by Pavel Chadnov <chadnov@google.com> on 2012-08-15
    Reviewed by Vsevolod Vlasov.
    
    CachedResource object for XHR response is now taken from ResourceLoader (if it's possible).
    
    Test: http/tests/inspector/network/network-xhr-async-double.html
    
    * inspector/InspectorInstrumentation.cpp:
    (WebCore):
    (WebCore::InspectorInstrumentation::didReceiveResourceResponseImpl):
    (WebCore::InspectorInstrumentation::didReceiveResourceResponseButCanceledImpl):
    * inspector/InspectorInstrumentation.h:
    (InspectorInstrumentation):
    (WebCore::InspectorInstrumentation::didReceiveResourceResponse):
    * inspector/InspectorResourceAgent.cpp:
    (WebCore::InspectorResourceAgent::willSendRequest):
    (WebCore::InspectorResourceAgent::didReceiveResponse):
    * inspector/InspectorResourceAgent.h:
    (WebCore):
    (InspectorResourceAgent):
    * loader/DocumentThreadableLoader.cpp:
    (WebCore::DocumentThreadableLoader::didReceiveResponse):
    * loader/ResourceLoadNotifier.cpp:
    (WebCore::ResourceLoadNotifier::didReceiveResponse):
    (WebCore::ResourceLoadNotifier::dispatchDidReceiveResponse):
    * loader/ResourceLoadNotifier.h:
    (ResourceLoadNotifier):
    * loader/ResourceLoader.cpp:
    (WebCore::ResourceLoader::isSubresourceLoader):
    (WebCore):
    * loader/ResourceLoader.h:
    (ResourceLoader):
    * loader/SubresourceLoader.cpp:
    (WebCore::SubresourceLoader::cachedResource):
    (WebCore):
    (WebCore::SubresourceLoader::isSubresourceLoader):
    * loader/SubresourceLoader.h:
    (SubresourceLoader):
    * loader/appcache/ApplicationCacheGroup.cpp:
    (WebCore::ApplicationCacheGroup::didReceiveResponse):
    
    LayoutTests: Web Inspector: Incorrect XHR responses when two async xhrs are sent synchronously
    https://bugs.webkit.org/show_bug.cgi?id=91630
    
    Patch by Pavel Chadnov <chadnov@google.com> on 2012-08-15
    Reviewed by Vsevolod Vlasov.
    
    New test for two async XHRs sent synchronously.
    
    * http/tests/inspector/network-test.js:
    (doXHR):
    (doXHRWithPayload.xhr.onreadystatechange):
    (doXHRWithPayload):
    * http/tests/inspector/network/network-xhr-async-double-expected.txt: Added.
    * http/tests/inspector/network/network-xhr-async-double.html: Added.
    * http/tests/inspector/network/resources/echo-payload.php: Added.
    * platform/chromium/http/tests/inspector/console-xhr-logging-async-expected.txt:
    * platform/chromium/http/tests/inspector/console-xhr-logging-expected.txt:
    
    git-svn-id: http://svn.webkit.org/repository/webkit/trunk@125681 268f45cc-cd09-0410-ab3c-d52691b4dbfc
    dcc56abd