[Webkit-unassigned] [Bug 22920] Inspector Request Headers Should Show Data/Variables/Parameters Sent With Request

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=22920


Patrick Mueller <pmuellr at yahoo.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #38829|                            |review?
               Flag|                            |
  Attachment #38550|0                           |1
        is obsolete|                            |




--- Comment #27 from Patrick Mueller <pmuellr at yahoo.com>  2009-08-31 12:14:32 PDT ---
Created an attachment (id=38829)
 --> (https://bugs.webkit.org/attachment.cgi?id=38829)
proposed patch - 2009/08/31

All of the issues from the previous review have been attended to, except for
the first:

in WebCore/inspector/front-end/Resource.js:
> +    this.requestMethod = requestMethod || "";
> +    this.requestFormData = requestFormData || "";

> Is this needed, can they be null/undefined?

It's not clear to me that they will always be string values, they might be null
at some point along the way.  Code further down in the processing expects these
to be string values, so seems best to initialize them to an empty string here. 
Empty string turns out to be a good default value for both of these.

Regarding the encoded/decoded versions of the form data; the initial display is
now decoded.  A hover help over a form data item displays: "Double-Click to
toggle between URL encoded and decoded formats".  Double-clicking results in
toggling between encoded and decoded.  Initially I was thinking of doing this
with a single click, but realized (without trying it), that this would likely
interfere with trying to select the items if you wanted to copy them to the
clipboard.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.