Adding 'blob:' and 'filesystem:' URLs as secure sources for extensions.

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 35 matching lines @@
     // We might need to relax this whitelist over time.
     if (source == "'self'" ||
         source == "'none'" ||
         source == "http://127.0.0.1" ||
         LowerCaseEqualsASCII(source, "http://localhost") ||
         StartsWithASCII(source, "http://127.0.0.1:", false) ||
         StartsWithASCII(source, "http://localhost:", false) ||
         StartsWithASCII(source, "https://", true) ||
         StartsWithASCII(source, "chrome://", true) ||
         StartsWithASCII(source, "chrome-extension://", true) ||
-        StartsWithASCII(source, "chrome-extension-resource:", true)) {
+        StartsWithASCII(source, "chrome-extension-resource:", true) ||
+        StartsWithASCII(source, "blob:", true) ||
+        StartsWithASCII(source, "filesystem:", true)) {

[abarth-chromium, 2012/08/24 14:55:00]

I suspect this should be LowerCaseEqualsASCII.  "blob://example.com" doesn't
make much sense.

       continue;
     }
 
     return false;
   }
 
   return true;  // Empty values default to 'none', which is secure.
 }
 
 // Returns true if |directive_name| matches |status.directive_name|.
@@ skipping 99 matching lines @@
       }
     }
   }
 
   return seen_sandbox;
 }
 
 }  // csp_validator
 
 }  // extensions