Bug 99145

Summary: Web Inspector: Show request body source in request headers view.
Product: WebKit Reporter: Vsevolod Vlasov <vsevik>
Component: Web Inspector (Deprecated)Assignee: Vsevolod Vlasov <vsevik>
Status: RESOLVED FIXED    
Severity: Normal CC: apavlov, bweinstein, joepeck, keishi, loislo, pfeldman, pmuellr, rik, webkit.review.bot, yurys
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Attachments:
Description Flags
screenshot
none
Patch none

Description Vsevolod Vlasov 2012-10-12 00:20:02 PDT
User report: 

Currently on the Network -> Headers tab, you can see the form data submitted on a POST request. However, unlike with Request Headers and Response Headers, there isn't a "view source" option to see this form data in the raw.

This option would be helpful, as it's (a) easier to see subtle syntax issues this way. 

I ran into this problem today, where a piece of form data had quotes around it when it shouldn't have. Since you often see quotes around string data to distinguish them from other datatypes (e.g., JSON), this didn't immediately stand out. I spent a few hours troubleshooting, thinking the problem was somewhere else, until I used Firebug with Firefox, and viewing the raw data immediately made the quotes stand out.
Comment 1 Vsevolod Vlasov 2012-10-12 00:29:03 PDT
Created attachment 168370 [details]
screenshot
Comment 2 Vsevolod Vlasov 2012-10-12 00:30:22 PDT
Created attachment 168371 [details]
Patch
Comment 3 WebKit Review Bot 2012-10-12 00:33:39 PDT
Attachment 168371 [details] did not pass style-queue:

Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/WebCore/ChangeLog', u'Source/WebCor..." exit_code: 1
Source/WebCore/ChangeLog:25:  Need whitespace between colon and description  [changelog/filechangedescriptionwhitespace] [5]
Total errors found: 1 in 4 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 4 WebKit Review Bot 2012-10-12 02:20:37 PDT
Comment on attachment 168371 [details]
Patch

Clearing flags on attachment: 168371

Committed r131161: <http://trac.webkit.org/changeset/131161>
Comment 5 WebKit Review Bot 2012-10-12 02:20:41 PDT
All reviewed patches have been landed.  Closing bug.