Bug 19103 - Web inspector never shows the query part of URIs
Summary: Web inspector never shows the query part of URIs
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Inspector (Deprecated) (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-05-16 11:20 PDT by Derk-Jan Hartman
Modified: 2009-11-21 19:06 PST (History)
2 users (show)

See Also:


Attachments
Patch to add a tooltip with the full URL with query parameters to each resource in the Resources Panel (1.28 KB, patch)
2009-11-21 16:15 PST, Jessie Berlin
timothy: commit-queue-
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
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 &.

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