Bug 39848 - Web Inspector: [REGRESSION] Query parameters are not displayed in the resources headers section.
Summary: Web Inspector: [REGRESSION] Query parameters are not displayed in the resourc...
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: Pavel Feldman
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-05-27 10:35 PDT by Pavel Feldman
Modified: 2010-05-27 15:53 PDT (History)
8 users (show)

See Also:

Attachments
[PATCH] Proposed fix. (1.76 KB, patch)
2010-05-27 10:47 PDT, Pavel Feldman 		no flags Details | Formatted Diff | Diff
patch for manual test case (6.52 KB, patch)
2010-05-27 11:17 PDT, Patrick Mueller 		no flags 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 Pavel Feldman 2010-05-27 10:35:37 PDT 
Patch to follow.
Comment 1 Pavel Feldman 2010-05-27 10:47:19 PDT 
Created attachment 57257 [details]
[PATCH] Proposed fix.
Comment 2 Patrick Mueller 2010-05-27 11:12:51 PDT 
I was looking at this via Bug 39711 this morning.  Guess we can close that.

My fix was to add a call to this._refreshQueryString() in _refreshURL().  That worked for WebKit nightlies.  

I've also patched the WebCore/manual-tests/inspector/display-form-data.html test case to work better than it used to.  It unfortunately can't really be run from a file:// url, and so would probably work better as a LayoutTest.
Comment 3 Patrick Mueller 2010-05-27 11:17:13 PDT 
Created attachment 57261 [details]
patch for manual test case

current patch I was playing with - you won't want the ResourceView.js bit that's also in here, the rest is a patch to the manual test case
Comment 4 WebKit Commit Bot 2010-05-27 15:53:03 PDT 
Comment on attachment 57257 [details]
[PATCH] Proposed fix.

Clearing flags on attachment: 57257

Committed r60327: <http://trac.webkit.org/changeset/60327>
Comment 5 WebKit Commit Bot 2010-05-27 15:53:09 PDT 
All reviewed patches have been landed.  Closing bug.