Fix crash with webRequest.event.addListener when provided an invalid URL

chrome/test/data/extensions/api_test/webrequest/api/test.html

 <script>
 chrome.test.runTests([
   function addListeners() {
     chrome.experimental.webRequest.onBeforeRequest.addListener(
         function(details) {});
     chrome.experimental.webRequest.onBeforeSendHeaders.addListener(
         function(details) {});
     chrome.experimental.webRequest.onRequestSent.addListener(
         function(details) {});
     chrome.experimental.webRequest.onBeforeRedirect.addListener(
         function(details) {});
     chrome.experimental.webRequest.onResponseStarted.addListener(
         function(details) {});
     chrome.experimental.webRequest.onCompleted.addListener(
         function(details) {});
     chrome.experimental.webRequest.onErrorOccurred.addListener(
         function(details) {});
     chrome.test.succeed();
   },
 
   // Tests that the extra parameters to addListener are checked for invalid
   // values.
   function specialEvents() {
     var goodFilter = {urls: ["http://*.google.com/*"]};
     var goodExtraInfo = ["blocking"];
-    var badFilter = {broken: 42};
-    var badExtraInfo = ["garbage"];
-    // Works for onBeforeSendHeaders only:
-    var headersExtraInfo = ["requestHeaders"];
     chrome.experimental.webRequest.onBeforeRequest.addListener(
-      function(details) {},
-      goodFilter, goodExtraInfo);
+        function(details) {},
+        goodFilter, goodExtraInfo);
+
+    // Try a bad RequestFilter.
     try {
       chrome.experimental.webRequest.onBeforeRequest.addListener(
-        function(details) {},
-        badFilter, goodExtraInfo);
+          function(details) {},
+          {badFilter: 42}, goodExtraInfo);
       chrome.test.fail();
     } catch (e) {
       chrome.test.assertTrue(e.message.search("Invalid value") >= 0);
     }
+
+    // Try a bad ExtraInfoSpec.
     try {
       chrome.experimental.webRequest.onBeforeRequest.addListener(
-        function(details) {},
-        goodFilter, badExtraInfo);
+          function(details) {},
+          goodFilter, ["badExtraInfo"]);
       chrome.test.fail();
     } catch (e) {
       chrome.test.assertTrue(e.message.search("Invalid value") >= 0);
     }
+
+    // This extraInfoSpec should only work for onBeforeSendHeaders.
+    var headersExtraInfo = ["requestHeaders"];
     chrome.experimental.webRequest.onBeforeSendHeaders.addListener(
-      function(details) {},
-      goodFilter, headersExtraInfo);
+        function(details) {},
+        goodFilter, headersExtraInfo);
     try {
       chrome.experimental.webRequest.onBeforeRequest.addListener(
-        function(details) {},
-        goodFilter, headersExtraInfo);
+          function(details) {},
+          goodFilter, headersExtraInfo);
       chrome.test.fail();
     } catch (e) {
       chrome.test.assertTrue(e.message.search("Invalid value") >= 0);
     }
+
+    // Try a bad URL pattern. The error happens asynchronously. We're just
+    // verifying that the browser doesn't crash.
+    chrome.experimental.webRequest.onBeforeRequest.addListener(
+        function(details) {},
+        {urls: ["badpattern://*"]});

[battre, 2011/04/27 09:45:31]

optional: do you want to send a removeListener instantly after the addListener?

[Matt Perry, 2011/04/27 19:45:59]

Done.

+
     chrome.test.succeed();
   },
 ]);
 </script>