Merge pull request #909 from JordanMilne/master

Throw if a.protocol unsupported, fix comment
honestbleeps · Apr 12, 2014 · 5c778ab · 5c778ab
2 parents 4a5573a + a85653e
commit 5c778ab
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/lib/HTMLPasteurizer.js b/lib/HTMLPasteurizer.js
@@ -75,7 +75,7 @@
                 if(node.protocol !== undefined) {
                     var scrubbedProto = node.protocol.replace(SCHEME_FILTER, "");
 
-                    // Only allow non-whitelisted schemes unless the document was served via
+                    // Only allow whitelisted schemes unless the document was served via
                     // the same scheme.
                     if(config.schemeWhitelist.indexOf(scrubbedProto) === -1 &&
                        scrubbedProto !== document.location.protocol) {
@@ -84,6 +84,7 @@
                 } else {
                     // TODO: Handle UAs that don't support a.protocol?
                     // we may need to bundle URL.js.
+                    throw "Pasteurizer: a.protocol unsupported.";
                 }
             }
         }
@@ -201,4 +202,4 @@
 			return real_parseFromString.apply(this, arguments);
 		}
 	};
-}(DOMParser));
+}(DOMParser));