Changeset 74256 in webkit

Timestamp:

Dec 17, 2010 5:52:53 AM (13 years ago)

Author:

caseq@chromium.org

Message:

2010-12-17 Andrey Kosyakov <​caseq@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: clear extra/override headers when front-end is closed
​https://bugs.webkit.org/show_bug.cgi?id=51244

• inspector/InspectorController.cpp: (WebCore::InspectorController::disconnectFrontend): (WebCore::InspectorController::willSendRequest):

Location:

trunk/WebCore

Files:

• ChangeLog (modified) (1 diff)
• inspector/InspectorController.cpp (modified) (2 diffs)

trunk/WebCore/ChangeLog

@@ +1 @@
+2010-12-17  Andrey Kosyakov  <caseq@chromium.org>
+
+        Reviewed by Pavel Feldman.
+
+        Web Inspector: clear extra/override headers when front-end is closed
+        https://bugs.webkit.org/show_bug.cgi?id=51244
+
+        * inspector/InspectorController.cpp:
+        (WebCore::InspectorController::disconnectFrontend):
+        (WebCore::InspectorController::willSendRequest):
+
 2010-12-17  Pavel Feldman  <pfeldman@chromium.org>
 

trunk/WebCore/inspector/InspectorController.cpp

@@ -577 +577 @@
     releaseFrontendLifetimeAgents();
     m_timelineAgent.clear();
+    m_extraHeaders.clear();
 }
 
@@ -826 +827 @@
 
     request.setReportLoadTiming(true);
-    // Only enable raw headers if front-end is attached, as otherwise we may lack
-    // permissions to fetch the headers.
-    if (m_frontend)
+
+    if (m_frontend) {
+        // Only enable raw headers if front-end is attached, as otherwise we may lack
+        // permissions to fetch the headers.
         request.setReportRawHeaders(true);
 
-    if (m_extraHeaders) {
-        HTTPHeaderMap::const_iterator end = m_extraHeaders->end();
-        for (HTTPHeaderMap::const_iterator it = m_extraHeaders->begin(); it != end; ++it)
-            request.setHTTPHeaderField(it->first, it->second);
+        if (m_extraHeaders) {
+            HTTPHeaderMap::const_iterator end = m_extraHeaders->end();
+            for (HTTPHeaderMap::const_iterator it = m_extraHeaders->begin(); it != end; ++it)
+                request.setHTTPHeaderField(it->first, it->second);
+        }
     }
 
     bool isMainResource = m_mainResourceIdentifier == identifier;
-
     if (m_timelineAgent)
         m_timelineAgent->willSendResourceRequest(identifier, isMainResource, request);