Merge pull request #4841 from Polymer/exclude-style-script-bindings

Exclude script and style tags for parsing bindings

lib/mixins/template-stamp.html

@@ -256,6 +256,9 @@
        * @param {!NodeInfo} nodeInfo Node metadata for current template.
        */
       static _parseTemplateChildNodes(root, templateInfo, nodeInfo) {
+        if (root.localName === 'script' || root.localName === 'style') {
+          return;
+        }
         for (let node=root.firstChild, parentIndex=0, next; node; node=next) {
           // Wrap templates
           if (node.localName == 'template') {

test/unit/property-effects-elements.html

@@ -68,6 +68,18 @@
         literal1 {{cpnd1}} literal2 {{cpnd2}}{{cpnd3.prop}} literal3 {{computeCompound(cpnd4, cpnd5, 'literalComputed')}} literal4
       </div>
       <span id="boundWithDash">{{objectWithDash.binding-with-dash}}</span>
+      <script id="scriptWithBinding">
+        /* eslint-disable no-unused-vars */
+        function foo() {
+          return "We have a {{binding}} here";
+        }
+        /* eslint-enable no-unused-vars */
+      </script>
+      <style id="styleWithBinding">
+        :host {
+          content: '[[binding]]'
+        }
+      </style>
   </template>
   <script>
     let ComputingBehavior = {

test/unit/property-effects.html

@@ -386,6 +386,21 @@
     assert.equal(el.lateBoundObserver.firstCall.args[0], 'late');
   });
 
+  test('does not parse bindings inside <script>', function() {
+    assert.include(el.$.scriptWithBinding.textContent, "{{binding}}");
+  });
+
+  test('does not parse bindings inside <style>', function() {
+    var style = el.shadowRoot.querySelector('style');
+    // native shadow dom
+    if (style) {
+      assert.include(style.textContent, "[[binding]]");
+    // shady dom
+    } else {
+      assert.include(document.querySelector('[scope="x-basic"]').textContent, "[[binding]]");
+    }
+  });
+
 });
 
 suite('computed bindings with dynamic functions', function() {