Bug 32551 – Web Inspector: Do not lose HTTP redirect information, display it in the Resources Panel.

• Status: RESOLVED
• Resolution: FIXED
• Component: Web Inspector (Deprecated)

[Pavel Feldman]

Currently we lose all the intermediate redirects / headers. We should not.

[Pavel Feldman]

Created attachment 44865 [details]
[IMAGE] Navigating to gmail.com when not logged in now results in 3 entries.

[Pavel Feldman]

Created attachment 44867 [details]
[PATCH] Proposed change (contains drive-by fix removing createScriptObject - no need in that).

[WebKit Review Bot]

style-queue ran check-webkit-style on attachment 44867 [details] without any errors.

[Timothy Hatcher]

Comment on attachment 44867 [details]
[PATCH] Proposed change (contains drive-by fix removing createScriptObject - no need in that).


> +    if (resource != m_mainResource && windowVisible())
>          resource->updateScriptObject(m_frontend.get());

Where does the main resource get updated?

[Pavel Feldman]

Committing to http://svn.webkit.org/repository/webkit/trunk ...
	M	WebCore/ChangeLog
	M	WebCore/inspector/InspectorController.cpp
	M	WebCore/inspector/InspectorFrontend.cpp
	M	WebCore/inspector/InspectorFrontend.h
	M	WebCore/inspector/InspectorResource.cpp
	M	WebCore/inspector/InspectorResource.h
	M	WebCore/inspector/front-end/inspector.js
Committed r52154

[Pavel Feldman]

(In reply to comment #4)
> (From update of attachment 44867 [details])
> 
> > +    if (resource != m_mainResource && windowVisible())
> >          resource->updateScriptObject(m_frontend.get());
> 
> Where does the main resource get updated?

It did not matter that much - I just wanted main resource not to be touched until the commit.