[Webkit-unassigned] [Bug 61205] Web Inspector: Cache XHR content in backend, do not use initialContentSet for XHRs.

[bugzilla-daemon at webkit.org]

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





--- Comment #14 from Yury Semikhatsky <yurys at chromium.org>  2011-05-27 07:03:09 PST ---
(From update of attachment 95157)
View in context: https://bugs.webkit.org/attachment.cgi?id=95157&action=review

> Source/WebCore/inspector/InspectorResourceAgent.cpp:299
> +    m_resourcesData->willLoadXHRSynchronously();

No need to delegate this calls to the storage, lets inline this and the next method and use m_loadingXHRSynchronously only in InspectorResourceAgent. It should make XHR check in InspectorResourceAgent::didReceiveResponse more clear:

else if (m_loadingXHRSynchronously || m_resourcesData->isXHR(identifier))
    type = InspectorPageAgent::XHRResource;

> Source/WebCore/inspector/NetworkResourcesData.cpp:107
> +    if (m_loadingXHRSynchronously)

This flag should move to InspectorResourceAgent which tracks network activity while this class should play role of storage.

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