Permalink
Browse files

Fix for some broken sites that have a Location header that points to …

…the same URL
  • Loading branch information...
1 parent 30ab2fc commit 7cf8e9e5a3d9b7de1bf15bad3c1324bf599e420f @brewt committed Jan 31, 2013
Showing with 1 addition and 1 deletion.
  1. +1 −1 src/common/component.js
@@ -92,7 +92,7 @@ YSLOW.Component.prototype.populateProperties = function (resolveRedirect, ignore
// check location
// bookmarklet and har already handle redirects
- if (that.headers.location && resolveRedirect) {
+ if (that.headers.location && resolveRedirect && that.headers.location !== that.url) {
// Add a new component.
comp = that.parent.addComponentNoDuplicate(that.headers.location,
(that.type !== 'redirect' ? that.type : 'unknown'), that.url);

0 comments on commit 7cf8e9e

Please sign in to comment.