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

BUG=23865

Review URL: http://codereview.chromium.org/341025

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@30534 0039d316-1c4b-4281-b951-d872f2087c98

2 files changed, 16 insertions, 13 deletions

diff --git a/chrome/test/data/extensions/api_test/connect/page.js b/chrome/test/data/extensions/api_test/connect/page.js
index d70f415..2f737f70 100644
--- a/chrome/test/data/extensions/api_test/connect/page.js
+++ b/chrome/test/data/extensions/api_test/connect/page.js
@@ -5,6 +5,6 @@ chrome.extension.sendRequest({step: 1}, function(response) {
 });
 
 // For test sendRequest.
-chrome.extension.onRequest.addListener(function(request, sendResponse) {
+chrome.extension.onRequest.addListener(function(request, sender, sendResponse) {
   sendResponse({success: (request.step2 == 1)});
 });
diff --git a/chrome/test/data/extensions/api_test/connect/test.html b/chrome/test/data/extensions/api_test/connect/test.html
index 31868b4..b108a8d 100644
--- a/chrome/test/data/extensions/api_test/connect/test.html
+++ b/chrome/test/data/extensions/api_test/connect/test.html
@@ -2,22 +2,25 @@
 chrome.test.runTests([
   // Tests receiving a request from a content script and responding.
   function onRequest() {
-    chrome.extension.onRequest.addListener(function(request, sendResponse) {
-      if (request.step == 1) {
-        // Step 1: Page should send another request for step 2.
-        sendResponse({nextStep: true});
-      } else {
-        // Step 2.
-        chrome.test.assertEq(request.step, 2);
-        sendResponse({});
-        chrome.test.succeed();
-      } 
-    });
+    chrome.extension.onRequest.addListener(
+      function(request, sender, sendResponse) {
+        chrome.test.assertTrue("url" in sender.tab, "no tab available.");
+        chrome.test.assertEq(sender.id, location.host);
+        if (request.step == 1) {
+          // Step 1: Page should send another request for step 2.
+          sendResponse({nextStep: true});
+        } else {
+          // Step 2.
+          chrome.test.assertEq(request.step, 2);
+          sendResponse({});
+          chrome.test.succeed();
+        } 
+      });
   },
   // Tests sending a request to a tab and receiving a response.
   function sendRequest() {
     chrome.tabs.getSelected(null, function(tab) {
-      chrome.test.log('Selected tab: ' + tab.url);
+      chrome.test.log("Selected tab: " + tab.url);
       chrome.tabs.sendRequest(tab.id, {step2: 1}, function(response) {
         chrome.test.assertTrue(response.success);
         chrome.test.succeed();