Bug 19103

Summary: Web inspector never shows the query part of URIs
Product: WebKit Reporter: Derk-Jan Hartman <hartman.wiki>
Component: Web Inspector (Deprecated)Assignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: jberlin, timothy
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Attachments:
Description Flags
Patch to add a tooltip with the full URL with query parameters to each resource in the Resources Panel timothy: commit-queue-

Description Derk-Jan Hartman 2008-05-16 11:20:50 PDT
When you are in "Resources", none of the files show the "query" part of the URIs. Not in the sidebar, and not in the titlebar. This is especially problematic if you load a load of dynamic generated files. for instance, i'm now looking at the CSS/JS contents in order to identify those files, because they are all named index.php atm :(

Suggestion is: add query part to the filename in the titlebar, and secondly, add a tooltip that shows the entire filename in the sidebar. Or simply show the query part there as well (though i guess it might become messy a bit quicker in the sidebar).
Comment 1 Timothy Hatcher 2008-05-25 13:57:30 PDT
We now show the full URL when you click a resource. But we should add a tooltip.
Comment 2 Jessie Berlin 2009-11-21 16:15:25 PST
Created attachment 43661 [details]
Patch to add a tooltip with the full URL with query parameters to each resource in the Resources Panel
Comment 3 Timothy Hatcher 2009-11-21 17:11:29 PST
Comment on attachment 43661 [details]
Patch to add a tooltip with the full URL with query parameters to each resource in the Resources Panel


> +        this.tooltip = this.resource.url.escapeHTML();

You don't want escapeHTML here. This sets the title property in the DOM, so escaping isn't needed (and would cause & to show as &amp;.

Fix that before landing.
Comment 4 Jessie Berlin 2009-11-21 19:06:28 PST
Comment on attachment 43661 [details]
Patch to add a tooltip with the full URL with query parameters to each resource in the Resources Panel

committed with the change Tim requested in
http://trac.webkit.org/changeset/51290