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

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?)