From 4ce08461f070b0f45e556c1564d5a48c0059a529 Mon Sep 17 00:00:00 2001
From: Hongfei Ding <lannka@users.noreply.github.com>
Date: Wed, 31 Jan 2018 09:41:02 -0800
Subject: [PATCH] Removes the check for custom-template declaration (#13165)

* Removes the check for custom-template declaration

* remove test
---
 src/service/template-impl.js     | 24 ------------------------
 test/functional/test-template.js |  7 -------
 2 files changed, 31 deletions(-)

diff --git a/src/service/template-impl.js b/src/service/template-impl.js
index e4ac2aefca74..91e982c9737a 100644
--- a/src/service/template-impl.js
+++ b/src/service/template-impl.js
@@ -289,7 +289,6 @@ export class Templates {
       return this.templateClassMap_[type];
     }
 
-    this.checkTemplateDeclared_(element, type);
     let aResolve;
     const promise = new Promise((resolve, unusedReject) => {
       aResolve = resolve;
@@ -299,29 +298,6 @@ export class Templates {
     return promise;
   }
 
-
-  /**
-   * Checks that the template type has actually been declared by a
-   * `<script custom-template=$type>` tag in the head.
-   * @param {!Element} element
-   * @param {string} type
-   * @private
-   */
-  checkTemplateDeclared_(element, type) {
-    if (!this.declaredTemplates_) {
-      this.declaredTemplates_ = this.win_.Object.create(null);
-      const scriptTags = this.win_.document.querySelectorAll(
-          'script[custom-template]');
-      for (let i = 0; i < scriptTags.length; i++) {
-        this.declaredTemplates_[scriptTags[i].getAttribute(
-            'custom-template')] = true;
-      }
-    }
-    user().assert(this.declaredTemplates_[type],
-        'Template must be declared for %s as <script custom-template=%s>',
-        element, type);
-  }
-
   /**
    * Registers an extended template. This function should typically be called
    * through the registerTemplate method on the AMP runtime.
diff --git a/test/functional/test-template.js b/test/functional/test-template.js
index b3deaf3c81c9..121735272e0d 100644
--- a/test/functional/test-template.js
+++ b/test/functional/test-template.js
@@ -87,13 +87,6 @@ describes.fakeWin('Template', {}, env => {
     }).to.throw(/Duplicate template type/);
   });
 
-  it('should fail render if template is not declared', () => {
-    const templateElement = createTemplateElement();
-    expect(() => {
-      templates.renderTemplate(templateElement, {value: 0});
-    }).to.throw(/Template must be declared/);
-  });
-
   it('should block render until template registered', () => {
     templates.declaredTemplates_ = undefined;
     const templateElement = createTemplateElement();