Skip to content

Commit

Permalink
Oops. Was only overriding resolveURL() for img@srcset, but source@src…
Browse files Browse the repository at this point in the history
…set needs the same override.
  • Loading branch information
shogun70 committed Dec 17, 2013
1 parent 2a61377 commit fe52fc2
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion HTMLDecor.js
Original file line number Diff line number Diff line change
Expand Up @@ -998,7 +998,7 @@ forEach(words("link@<href script@<src img@<longDesc,<src,srcset iframe@<longDesc

urlAttrs['script']['src'].mustResolve = true;

urlAttrs['img']['srcset'].resolveURL = function(urlSet, baseURL, force) { // img@srcset will never be neutralized
function resolveSrcset(urlSet, baseURL, force) { // img@srcset will never be neutralized
if (!force) return urlSet;
var urlList = urlSet.split(/\s*,\s*/); // WARN this assumes URLs don't contain ','
forEach(urlList, function(urlDesc, i) {
Expand All @@ -1007,6 +1007,9 @@ urlAttrs['img']['srcset'].resolveURL = function(urlSet, baseURL, force) { // img
return urlList.join(', ');
}

urlAttrs['img']['srcset'].resolveURL = resolveSrcset;
urlAttrs['source']['srcset'].resolveURL = resolveSrcset;

urlAttrs['a']['ping'].resolveURL = function(urlSet, baseURL, force) { // a@ping will never be neutralized
if (!force) return urlSet;
var urlList = urlSet.split(/\s+/);
Expand Down

0 comments on commit fe52fc2

Please sign in to comment.