{"issue":{"title":"Highly cross-site linked websites causes 100% cpu usage and memory allocation fluctuations","user":{"id":1240282,"login":"jsamuel","url":"https://api.github.com/users/jsamuel","gravatar_id":"bc1d195cedb959a3946572b08ce9e7c7","avatar_url":"https://secure.gravatar.com/avatar/bc1d195cedb959a3946572b08ce9e7c7?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png"},"milestone":null,"created_at":"2011-12-22T18:54:29Z","labels":[{"name":"extension","url":"https://api.github.com/repos/RequestPolicy/requestpolicy/labels/extension","color":"ededed"}],"state":"open","updated_at":"2011-12-22T18:54:31Z","id":2640436,"pull_request":{"patch_url":null,"html_url":null,"diff_url":null},"html_url":"https://github.com/RequestPolicy/requestpolicy/issues/127","closed_at":null,"body":"    imported trac ticket\n    created: 2010-07-21 11:13:14\n    reporter: gonefish\n\nThe performance of RequestPolicy in my browser can become very poor on certain sites that rely on cross-site linking.  An especially poor one is [http://motls.blogspot.com/]. Firefox will run cause cpu to run 100% continously, and memory allocation will fluctuate between 50MB-100MB. I've tried older version of RequestPolicy all the way back to 0.5.09 (oldest compatible with Firefox) and same result. The example url has as far back as I know always cause serious performance problems since I've started using RequestPolicy.\r\n\r\nIf this is a general problem, I hope the example site can be used for locating/tuning the performance problems.\r\n\r\n'''Setup'''\r\nOS: Windows 7 32-bit\r\nFirefox: 3.6.6\r\nFirefox-addons: Only RequestPolicy (and Flash disabled)","number":127,"url":"https://api.github.com/repos/RequestPolicy/requestpolicy/issues/127","comments":1,"assignee":null},"comment":{"user":{"id":1240282,"login":"jsamuel","url":"https://api.github.com/users/jsamuel","gravatar_id":"bc1d195cedb959a3946572b08ce9e7c7","avatar_url":"https://secure.gravatar.com/avatar/bc1d195cedb959a3946572b08ce9e7c7?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png"},"created_at":"2011-12-22T18:54:31Z","updated_at":"2011-12-22T18:54:31Z","id":3253100,"body":"    imported trac comment\n    created: 2010-07-21 11:54:42\n    author: justin\n\nHi, it's a bug on this website. The website tries to load an image from an img tag's onerror handler, and they try to remove the error handler to prevent infinite error handling if the backup image also fails, but the code is incorrect. That is, the clustmaps site realized that if their site went down, for example, all of the sites using the code they provided would go into this infinite loop. So, they tried to remove the error handler the first time it runs so that it can't get called again. However, they didn't get it right.\r\n\r\nFor example, the following is one of the culprits (there may be more, I didn't look any further):\r\n\r\n{{{\r\n<img alt=\"Locations of visitors to this page\" onerror=\"this.onError=null; this.src='http://clustrmaps.com/images/clustrmaps-back-soon.jpg'; document.getElementById('clustrMapsLink').href='http://clustrmaps.com'\" title=\"Locations of visitors to this page\" height=\"106\" src=\"http://www2.clustrmaps.com/counter/index2.php?url=http://motls.blogspot.com\" id=\"clustrMapsImg\" style=\"border:1px solid;\" width=\"160\"/>\r\n}}}\r\n\r\nThe site can solve this by changing:\r\n\r\n{{{\r\nonerror=\"this.onError=null;\r\n}}}\r\n\r\nto\r\n\r\n{{{\r\nonerror=\"this.onerror=null;\r\n}}}\r\n\r\nThat is, it's `this.onerror` (all lowercase), not `this.onError`. Maybe this code works in IE but not Firefox?\r\n\r\nI've emailed clustrmaps to let them know they should fix this.","url":"https://api.github.com/repos/RequestPolicy/requestpolicy/issues/comments/3253100"},"action":"created"}