Bug 67120 - Web Inspector: wrong script display name when there is a slash in its URL query parameters
Summary: Web Inspector: wrong script display name when there is a slash in its URL que...
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: Yury Semikhatsky
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-08-29 04:56 PDT by Yury Semikhatsky
Modified: 2011-08-30 03:56 PDT (History)
10 users (show)

See Also:

Attachments
Patch (4.31 KB, patch)
2011-08-29 06:41 PDT, Yury Semikhatsky 		pfeldman: review+
Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Yury Semikhatsky 2011-08-29 04:56:28 PDT 
From user report:

I see Chrome v15 (Chromium 15.0.861.0) has added some funky options for grouping the list of scripts by folder (enabled by default, see script view options to disable it).

But it seems the URL parsing for this feature is a little buggy... it doesn't cope properly with query parameters that include a slash, so a JS file

 /somepath/script.js?version=123&referer=anotherpath/file.html

will be shown as being a file called "file.html" in a folder "/somepath/script.js?version=123&referer=anotherpath/"

[ This actually happens whether the "scripts by folder" option is enabled or not, but it worked properly in v14, so I'm sort of pointing the finger at the grouping change :) ]
Comment 1 Yury Semikhatsky 2011-08-29 06:41:17 PDT 
Created attachment 105484 [details]
Patch
Comment 2 Yury Semikhatsky 2011-08-30 03:56:39 PDT 
Committed r94067: <http://trac.webkit.org/changeset/94067>