autocomplete.js clean up

Mostly good,

• $("#autocomplete") needs to be cached, and since it's an ID it doesn't make sense to keep .each,
• typeof value === "undefined", it's comparing to a string.

You're sure about if (this.selected === false)? Looks like you reversed the condition, $(false).removeClass('selected') will not work.

Looks good to me, but needed a reroll, please confirm it's still good droplet.

It's rtbc for me, check I didn't mess up your patch anyway :)

The patch includes #784670: autocomplete.js can iterate over functions in Array objects with it too.

how stupid is making a patch aginst D7 here :þ

+++ b/core/misc/autocomplete.jsundefined
@@ -30,9 +30,11 @@ Drupal.behaviors.autocomplete = {
+  var $autocomplete = $('#autocomplete');
+  if($autocomplete.length !== 0) {

Missing space between "if" and "(".

sorry about that

Committed/pushed to 8.x.

Moving to 7.x for backport.

Just a simple reroll from D8 patch.

Can you clarify the browsers this has been tested with? in Drupal 7, we need to see testing in IE 6 & 7.

On my schedule for tomorrow night

We don't backport cleanup JS patches. No automated testing and not enough manpower.

Actually the hasownproperty would fix a bug linked in #5 so i don't think we need to drop anything from it. It just needs testing. I don't have the time ATM, I can't rtbc it and nobody picked it up since April.

By all mean if you have people to test it, get that in that'd be a good thing.

autocomplete_clean_up.patch queued for re-testing.