Change the extension.onRequest even to include the sender info.

Change the extension.onRequest even to include the sender info.

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

[Aaron Boodman, 2009-10-29 01:37:36]

http://codereview.chromium.org/341025/diff/1/6
File chrome/test/data/extensions/api_test/connect/test.html (right):

http://codereview.chromium.org/341025/diff/1/6#newcode5
Line 5: chrome.extension.onRequest.addListener(function(info) {
I realize this is all personal, but to me the nesting of this parameter is
getting unwieldy.

I would prefer to change the signature to requestCallback(data, details,
sendResponse).

[Matt Perry, 2009-10-29 22:30:23]

OK, went with 3 args as you suggested.

[Aaron Boodman, 2009-10-29 23:45:44]

lgtm