Fixed "Hide placeholders of blocked elements" feature

Send a message to the content script if an element was blocked, it can then try to remove the element

[Thomas Greiner, 2012-10-26 14:21:54]

some minor loop-optimizations which are useful for websites with a lot of images
on them

http://codereview.adblockplus.org/8684066/diff/1/include.preload.js
File include.preload.js (right):

http://codereview.adblockplus.org/8684066/diff/1/include.preload.js#newcode64
include.preload.js:64: for (var i = 0; i < elements.length; i++)
don't access elements.length on each iteration (create a variable for it before
the loop)

http://codereview.adblockplus.org/8684066/diff/1/include.preload.js#newcode66
include.preload.js:66: remove.push(elements[i]);
we could break after we found one element

http://codereview.adblockplus.org/8684066/diff/1/include.preload.js#newcode68
include.preload.js:68: for (var i = 0; i < remove.length; i++)
not as important as with elements.length but still not a bad thing to do:
don't access remove.length on each iteration (create a variable for it before
the loop)

[Wladimir Palant, 2012-10-26 14:34:23]

http://codereview.adblockplus.org/8684066/diff/1/include.preload.js
File include.preload.js (right):

http://codereview.adblockplus.org/8684066/diff/1/include.preload.js#newcode66
include.preload.js:66: remove.push(elements[i]);
On 2012/10/26 14:21:54, Thomas Greiner wrote:
> we could break after we found one element

I don't think we should, it isn't necessarily a single element.

[Thomas Greiner, 2012-10-26 14:35:59]

LGTM