Update from https://crrev.com/317530

third_party/jstemplate/jstemplate.js

 // Copyright 2006 Google Inc.
 //
 // Licensed under the Apache License, Version 2.0 (the "License");
 // you may not use this file except in compliance with the License.
 // You may obtain a copy of the License at
 //
 // http://www.apache.org/licenses/LICENSE-2.0
 //
 // Unless required by applicable law or agreed to in writing, software
 // distributed under the License is distributed on an "AS IS" BASIS,
@@ skipping 534 matching lines @@
  * element in the array, with the array element being the context
  * object. If the array is empty, or the value is undefined, then the
  * current template node is dropped. If the value is not an array,
  * then it is just made the context object.
  *
  * @param {JsEvalContext} context The current evaluation context.
  *
  * @param {Element} template The currently processed node of the template.
  *
  * @param {Function} select The javascript expression to evaluate.
- *
- * @notypecheck FIXME(hmitchell): See OCL6434950. instance and value need
- * type checks.
  */
 JstProcessor.prototype.jstSelect_ = function(context, template, select) {
   var me = this;
 
   var value = context.jsexec(select, template);
 
   // Enable reprocessing: if this template is reprocessed, then only
   // fill the section instance here. Otherwise do the cardinal
   // processing of a new template.
   var instance = domGetAttribute(template, ATT_instance);
@@ skipping 11 matching lines @@
   // The expression value instanceof Array is occasionally false for
   // arrays, seen in Firefox. Thus we recognize an array as an object
   // which is not null that has a length property. Notice that this
   // also matches input data with a length property, so this property
   // name should be avoided in input data.
   var multiple = isArray(value);
   var count = multiple ? jsLength(value) : 1;
   var multipleEmpty = (multiple && count == 0);
 
   if (multiple) {
+    value = /** @type Array */(value);
     if (multipleEmpty) {
       // For an empty array, keep the first template instance and mark
       // it last. Remove all other template instances.
       if (!instance) {
         domSetAttribute(template, ATT_instance, STRING_asteriskzero);
         displayNone(template);
       } else {
         domRemoveNode(template);
       }
 
@@ skipping 332 matching lines @@
 
 /**
  * Sets the jsinstance attribute on a node according to its context.
  *
  * @param {Element} template The template DOM node to set the instance
  * attribute on.
  *
  * @param {Array} values The current input context, the array of
  * values of which the template node will render one instance.
  *
- * @param {number} index The index of this template node in values.
+ * @param {number|string} index The index of this template node in values.
  */
 function jstSetInstance(template, values, index) {
   if (index == jsLength(values) - 1) {
     domSetAttribute(template, ATT_instance, CHAR_asterisk + index);
   } else {
     domSetAttribute(template, ATT_instance, STRING_empty + index);
   }
 }
 
 
 /**
  * Log the current state.
  * @param {string} caller An identifier for the caller of .log_.
  * @param {Element} template The template node being processed.
  * @param {Object} jstAttributeValues The jst attributes of the template node.
  */
 JstProcessor.prototype.logState_ = function(
     caller, template, jstAttributeValues) {
 };