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...
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
Depends on:
Reported: 2010-05-27 10:35 PDT by Pavel Feldman
Modified: 2010-05-27 15:53 PDT (History)
8 users (show)

See Also:

[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

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.