Compile third_party/jstemplate with Closure Compiler

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 805 matching lines @@
 
 /**
  * Helps to implement the transclude attribute, and is the initial
  * call to get hold of a template from its ID.
  *
  * If the ID is not present in the DOM, and opt_loadHtmlFn is specified, this
  * function will call that function and add the result to the DOM, before
  * returning the template.
  *
  * @param {string} name The ID of the HTML element used as template.
- * @param {Function} opt_loadHtmlFn A function which, when called, will return
+ * @param {Function=} opt_loadHtmlFn A function which, when called, will return
  *   HTML that contains an element whose ID is 'name'.
  *
  * @return {Element|null} The DOM node of the template. (Only element nodes
  * can be found by ID, hence it's a Element.)
  */
 function jstGetTemplate(name, opt_loadHtmlFn) {
   var doc = document;
   var section;
   if (opt_loadHtmlFn) {
     section = jstLoadTemplateIfNotPresent(doc, name, opt_loadHtmlFn);
@@ skipping 16 matching lines @@
  *
  * @param {string} name The ID of the HTML element used as template.
  * @param {Function} opt_loadHtmlFn A function which, when called, will return
  *   HTML that contains an element whose ID is 'name'.
  *
  * @return {Element} The DOM node of the template. (Only element nodes
  * can be found by ID, hence it's a Element.)
  */
 function jstGetTemplateOrDie(name, opt_loadHtmlFn) {
   var x = jstGetTemplate(name, opt_loadHtmlFn);
-  check(x !== null);
+  if (x === null) {
+    throw new Error('jstGetTemplate() returned null');
+  }
   return /** @type Element */(x);

[Dan Beam, 2014/09/26 02:23:09]

shouldn't this be @type {Element} ?

[Vitaly Pavlenko, 2014/09/26 18:31:39]

It works anyways, but that's a good nit. Done.

 }
 
 
 /**
  * If an element with id 'name' is not present in the document, call loadHtmlFn
  * and insert the result into the DOM.
  *
  * @param {Document} doc
  * @param {string} name
  * @param {Function} loadHtmlFn A function that returns HTML to be inserted
  * into the DOM.
- * @param {string} opt_target The id of a DOM object under which to attach the
+ * @param {string=} opt_target The id of a DOM object under which to attach the
  *   HTML once it's inserted.  An object with this id is created if it does not
  *   exist.
  * @return {Element} The node whose id is 'name'
  */
 function jstLoadTemplateIfNotPresent(doc, name, loadHtmlFn, opt_target) {
   var section = domGetElementById(doc, name);
   if (section) {
     return section;
   }
   // Load any necessary HTML and try again.
@@ skipping 62 matching lines @@
 
 /**
  * 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) {
 };
-
-
-/**
- * Retrieve the processing logs.
- * @return {Array.<string>} The processing logs.
- */
-JstProcessor.prototype.getLogs = function() {
-  return this.logs_;
-};