Minor for-loop optimizations for base.js

Problem/Motivation

When reading through the views code I came accross the
core/modules/views/js/base.js
which contains optimizable for loops:

for(var i in pairs) {  
  if (typeof(pairs[i]) === 'string') {
    ..
  }
}

Which could be done faster with less code
for(var i = 0; i<pairs.length; i++) { .. }
Or even faster but maybe a little bit harder to read
for(var i = pairs.length; i>=0; i--) { .. }

However the real problem is that this loop might cause unintended behaviour if the global Array.prototype is modified:

Array.prototype.hello = "world";
pairs = [];
for(var i in pairs) {  
  if (typeof(pairs[i]) === 'string') {
    console.log(i);
  }
}

So the typeof check is not needed as String.prototype.split always returns an array of strings:
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global...

Some of the above points also apply for the second loop:

for (var i in chars) {
  if (href.indexOf(chars[i]) > -1) {
    ..
  }
}

Proposed resolution

Use for(var i = 0; i<pairs.length; i++) { .. } and remove if (typeof(pairs[i]) === 'string') { so
* it's faster
* requires less code
* is safer