Add the definition of the webRequest and webNavigation APIs.

Add the definition of the webRequest and webNavigation APIs.

The webRequest API doesn't support the planned filter and extraInfoSpec parameters yet, these will be added later.

BUG=50943
TEST=ExtensionApiTest.WebRequest,ExtensionApiTest.WebNavigation

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=54723

[jochen (gone - plz use gerrit), 2010-08-02 14:24:38]

please review.

[yzshen, 2010-08-02 21:07:54]

Hi, Jochen.

Thanks for sending this out so quickly!

http://codereview.chromium.org/3061041/diff/1/5
File chrome/common/extensions/api/extension_api.json (right):

http://codereview.chromium.org/3061041/diff/1/5#newcode3009
chrome/common/extensions/api/extension_api.json:3009: "requestId": {"type":
"integer", "description": "The ID of the request to retrieve the document of
this navigation. Note that this event is fired prior to the corresponding
chrome.webRequest.onBeforeRequest."},
Currently I think it should be chrome.experimental.webRequest.onBeforeRequest

http://codereview.chromium.org/3061041/diff/1/5#newcode3027
chrome/common/extensions/api/extension_api.json:3027: "transitionType": {"type":
"string", "description": "Cause of the navigation. Set to null for subframe
navigations. Please refer to the transition types below for detailed
explanation."},
The transition types are not defined below.

http://codereview.chromium.org/3061041/diff/1/5#newcode3120
chrome/common/extensions/api/extension_api.json:3120: "tabId": {"type":
"integer", "description": "The ID of the tab in which the request takes place.
Set to null if the request isn't related to a tab."},
There is a "method" property in the design doc. Is that also postponed to later
CLs?

http://codereview.chromium.org/3061041/diff/1/5#newcode3121
chrome/common/extensions/api/extension_api.json:3121: "type": {"type": "string",
"description": "Please refer to the request types below for detailed
explanation."},
Request types are not defined below.

[Matt Perry, 2010-08-03 00:57:59]

LGTM

[jochen (gone - plz use gerrit), 2010-08-03 08:38:25]

http://codereview.chromium.org/3061041/diff/1/5
File chrome/common/extensions/api/extension_api.json (right):

http://codereview.chromium.org/3061041/diff/1/5#newcode3009
chrome/common/extensions/api/extension_api.json:3009: "requestId": {"type":
"integer", "description": "The ID of the request to retrieve the document of
this navigation. Note that this event is fired prior to the corresponding
chrome.webRequest.onBeforeRequest."},
On 2010/08/02 21:07:54, yzshen wrote:
> Currently I think it should be chrome.experimental.webRequest.onBeforeRequest

Done.

http://codereview.chromium.org/3061041/diff/1/5#newcode3027
chrome/common/extensions/api/extension_api.json:3027: "transitionType": {"type":
"string", "description": "Cause of the navigation. Set to null for subframe
navigations. Please refer to the transition types below for detailed
explanation."},
On 2010/08/02 21:07:54, yzshen wrote:
> The transition types are not defined below.

Done.

http://codereview.chromium.org/3061041/diff/1/5#newcode3120
chrome/common/extensions/api/extension_api.json:3120: "tabId": {"type":
"integer", "description": "The ID of the tab in which the request takes place.
Set to null if the request isn't related to a tab."},
On 2010/08/02 21:07:54, yzshen wrote:
> There is a "method" property in the design doc. Is that also postponed to
later
> CLs?

Done.

http://codereview.chromium.org/3061041/diff/1/5#newcode3121
chrome/common/extensions/api/extension_api.json:3121: "type": {"type": "string",
"description": "Please refer to the request types below for detailed
explanation."},
On 2010/08/02 21:07:54, yzshen wrote:
> Request types are not defined below.

Done.