Whitelisting `127.0.0.1` and `localhost` for HTTP in extensions' CSP.

chrome/common/extensions/csp_validator.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/common/extensions/csp_validator.h"
 
 #include "base/string_split.h"
 #include "base/string_tokenizer.h"
 #include "base/string_util.h"
 
@@ skipping 28 matching lines @@
   while (tokenizer.GetNext()) {
     std::string source = tokenizer.token();
     StringToLowerASCII(&source);
 
     if (EndsWith(source, "*", true))
       return false;
 
     // We might need to relax this whitelist over time.
     if (source == "'self'" ||
         source == "'none'" ||
+        StartsWithASCII(source, "http://127.0.0.1", true) ||
+        StartsWithASCII(source, "http://localhost", true) ||

[levin, 2012/08/13 11:07:23]

Does this allow http://localhost.mydomain.com/.... ?
If so, then this should be restricted further or not allowed.

         StartsWithASCII(source, "https://", true) ||
         StartsWithASCII(source, "chrome://", true) ||
         StartsWithASCII(source, "chrome-extension://", true) ||
         StartsWithASCII(source, "chrome-extension-resource:", true)) {
       continue;
     }
 
     return false;
   }
 
@@ skipping 103 matching lines @@
       }
     }
   }
 
   return seen_sandbox;
 }
 
 }  // csp_validator
 
 }  // extensions