Fix code hint provider selection

Commit 5878608 replaced a number of instances of `$.each` with `Array.forEach`. But those functions are not interchangeable: only the former supports early break-out. Both the CodeHintManager and the CodeHintList relied on this feature. In particular, that commit broke 1) CodeHintManager provider selection, by always choosing the last provider instead of stopping at the first, highest-priority provider; and 2) the CodeHintList, by adding all the available hints instead of of just the first `maxResults` hints. The first issue also breaks Edge Web Fonts in particular.

This pull reverts just those two changes. I haven't checked all the other replacements from that commit to see if there are other related problems. (Want to have a look, @TomMalbran?)