ABP/Android JavaScript code

assets/js/XMLHttpRequest.jsm

 
 XMLHttpRequest = function()
 {
   this.headers = {};
   this.responseHeaders = {};
 };
 
 XMLHttpRequest.UNSENT = 0;
 XMLHttpRequest.OPEN = 1;
 XMLHttpRequest.HEADERS_RECEIVED = 2;
@@ skipping 57 matching lines @@
     this.readyState = XMLHttpRequest.DONE;
     this.onreadystatechange();
   },
   onreadystatechange: function(){},
   getResponseHeader: function(header)
   {
     if (this.readyState < XMLHttpRequest.LOADING)
       throw new Error("INVALID_STATE_ERR");
     else
     {
+      var lcHeader = header.toLowerCase();
       var returnedHeaders = [];
       for (var rHeader in this.responseHeaders)
       {
-        if (rHeader.match(new RegExp(header, "i")))

[Felix Dahlke, 2012/11/09 14:59:16]

I think we need to quote the header to use it in a regex, it may lead to weird
errors otherwise. Here's an example quote function that looks good to me:
http://stackoverflow.com/a/2593661

Alternatively, you could just convert both values to lower case before comparing
them.

[Andrey Novikov, 2012/11/12 09:33:12]

Done.

+        if (this.responseHeaders.hasOwnProperty(rHeader) && rHeader.toLowerCase() == lcHeader)
           returnedHeaders.push(this.responseHeaders[rHeader]);
       }
 
       if (returnedHeaders.length)
         return returnedHeaders.join(", ");
     }
 
     return null;
   },
   getAllResponseHeaders: function(header)
@@ skipping 68 matching lines @@
   status: -1,
   notificationCallbacks: {},
   loadFlags: 0,
   INHIBIT_CACHING: 0,
   VALIDATE_ALWAYS: 0,
   QueryInterface: function()
   {
     return this;
   }
 };