Add read-only access to POST data for webRequest's onBeforeRequest

chrome/common/extensions/api/web_request.json

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 [
   {
     "namespace": "webRequest",
     "types": [
       {
         "id": "RequestFilter",
@@ skipping 88 matching lines @@
         "parameters": [
           {
             "type": "object",
             "name": "details",
             "properties": {
               "requestId": {"type": "string", "description": "The ID of the request. Request IDs are unique within a browser session. As a result, they could be used to relate different events of the same request."},
               "url": {"type": "string"},
               "method": {"type": "string", "description": "Standard HTTP method."},
               "frameId": {"type": "integer", "description": "The value 0 indicates that the request happens in the main frame; a positive value indicates the ID of a subframe in which the request happens. If the document of a (sub-)frame is loaded (<code>type</code> is <code>main_frame</code> or <code>sub_frame</code>), <code>frameId</code> indicates the ID of this frame, not the ID of the outer frame. Frame IDs are unique within a tab."},
               "parentFrameId": {"type": "integer", "description": "ID of frame that wraps the frame which sent the request. Set to -1 if no parent frame exists."},
+              "experimentalPostData": {

[Matt Perry, 2012/07/23 21:19:01]

Drop the "experimental" prefix. We're moving away from that convention.

[battre, 2012/07/24 09:28:53]

Don't you want to indicate to developers that this feature may change? I am
concerned that we might not get it right in the first try.

[Matt Perry, 2012/07/24 20:46:57]

The description indicates that. Also, the feature is restricted to the dev
channel. I think that's enough, and is the convention we're moving towards.

[vabr (Chromium), 2012/07/30 16:05:57]

I removed the 'experimental' prefix based on the above discussion.

+                "type": "object",
+                "optional": true,
+                "description": "Experimental feature. Container for representations of data sent by the request via POST method. Only provided if extraInfoSpec contains requestPostData.",
+                "properties": {
+                  "formData": {
+                    "type": "object",
+                    "optional": true,
+                    "description": "If the POST data is a sequence of key-value pairs, either in multipart/form-data, or application/x-www-form-urlencoded, this dictionary is present and for each key contains the list of all values for that key. If the data is in another encoding, or if it was malformed, the dictionary is not present. It is also not present if the form upload was chunked. Example value of this dictionary is {'key': ['value1', 'value2']}.",
+                    "properties": {},
+                    "additionalProperties": {
+                      "type": "array",
+                      "items": { "type": "string" }
+                    }
+                  }
+                }
+              },
               "tabId": {"type": "integer", "description": "The ID of the tab in which the request takes place. Set to -1 if the request isn't related to a tab."},
               "type": {"type": "string", "enum": ["main_frame", "sub_frame", "stylesheet", "script", "image", "object", "xmlhttprequest", "other"], "description": "How the requested resource will be used."},
               "timeStamp": {"type": "number", "description": "The time when this signal is triggered, in milliseconds since the epoch."}
             }
           }
         ],
         "extraParameters": [
           {
             "$ref": "RequestFilter",
             "name": "filter",
             "description": "A set of filters that restricts the events that will be sent to this listener."
           },
           {
             "type": "array",
             "optional": true,
             "name": "extraInfoSpec",
             "description": "Array of extra information that should be passed to the listener function.",
             "items": {
               "type": "string",
-              "enum": ["blocking"]
+              "enum": ["blocking", "requestPostData"]

[Matt Perry, 2012/07/23 21:19:01]

imo "request" is unnecessary here. There's only 1 kind of "postData", so I think
that would be a fine name.

[vabr (Chromium), 2012/07/30 16:05:57]

Done.

             }
           }
         ],
         "returns": {
           "$ref": "BlockingResponse",
           "description": "If \"blocking\" is specified in the \"extraInfoSpec\" parameter, the event listener should return an object of this type.",
           "optional": true
         }
       },
       {
@@ skipping 338 matching lines @@
           {
             "$ref": "RequestFilter",
             "name": "filter",
             "description": "A set of filters that restricts the events that will be sent to this listener."
           }
         ]
       }
     ]
   }
 ]