From 53053eb447686ac9a66b8035a958584ac2e40ecf Mon Sep 17 00:00:00 2001
From: Kevin Schaaf <kschaaf@google.com>
Date: Fri, 14 Apr 2017 17:59:21 -0700
Subject: [PATCH 1/9] Add 2.x hybrid affordances for stamping template content.
 Fixes #4536

---
 src/lib/template/dom-bind.html   | 12 +++++--
 src/lib/template/dom-if.html     | 13 ++++++--
 src/lib/template/dom-repeat.html |  7 ++++
 test/unit/dom-bind.html          | 37 +++++++++++++++++++--
 test/unit/dom-if.html            | 39 ++++++++++++++++++----
 test/unit/dom-repeat.html        | 55 +++++++++++++++++++++++---------
 6 files changed, 134 insertions(+), 29 deletions(-)

diff --git a/src/lib/template/dom-bind.html b/src/lib/template/dom-bind.html
index c9561c9105..a7cdf4bccd 100644
--- a/src/lib/template/dom-bind.html
+++ b/src/lib/template/dom-bind.html
@@ -108,8 +108,16 @@
     },
 
     _insertChildren: function() {
-      var parentDom = Polymer.dom(Polymer.dom(this).parentNode);
-      parentDom.insertBefore(this.root, this);
+      var refNode;
+      var parentNode = Polymer.dom(this).parentNode;
+      // Affordance for 2.x hybrid mode
+      if (parentNode.localName == this.is) {
+        refNode = parentNode;
+        parentNode = Polymer.dom(parentNode).parentNode;
+      } else {
+        refNode = this;
+      }
+      Polymer.dom(parentNode).insertBefore(this.root, refNode);
     },
 
     _removeChildren: function() {
diff --git a/src/lib/template/dom-if.html b/src/lib/template/dom-if.html
index dc018ba579..8952e8be5a 100644
--- a/src/lib/template/dom-if.html
+++ b/src/lib/template/dom-if.html
@@ -134,11 +134,18 @@
       var parentNode = Polymer.dom(this).parentNode;
       // Guard against element being detached while render was queued
       if (parentNode) {
-        var parent = Polymer.dom(parentNode);
+        var refNode;
+        // Affordance for 2.x hybrid mode
+        if (parentNode.localName == this.is) {
+          refNode = parentNode;
+          parentNode = Polymer.dom(parentNode).parentNode;
+        } else {
+          refNode = this;
+        }
         if (!this._instance) {
           this._instance = this.stamp();
           var root = this._instance.root;
-          parent.insertBefore(root, this);
+          Polymer.dom(parentNode).insertBefore(root, refNode);
         } else {
           var c$ = this._instance._children;
           if (c$ && c$.length) {
@@ -146,7 +153,7 @@
             var lastChild = Polymer.dom(this).previousSibling;
             if (lastChild !== c$[c$.length-1]) {
               for (var i=0, n; (i<c$.length) && (n=c$[i]); i++) {
-                parent.insertBefore(n, this);
+                Polymer.dom(parentNode).insertBefore(n, refNode);
               }
             }
           }
diff --git a/src/lib/template/dom-repeat.html b/src/lib/template/dom-repeat.html
index 07f56d9c2d..f8e3b2bdc3 100644
--- a/src/lib/template/dom-repeat.html
+++ b/src/lib/template/dom-repeat.html
@@ -710,6 +710,13 @@
       var beforeRow = this._instances[idx + 1];
       var beforeNode = beforeRow && !beforeRow.isPlaceholder ? beforeRow._children[0] : this;
       var parentNode = Polymer.dom(this).parentNode;
+      // Affordance for 2.x hybrid mode
+      if (parentNode.localName == this.is) {
+        if (beforeNode == this) {
+          beforeNode = parentNode;
+        }
+        parentNode = Polymer.dom(parentNode).parentNode;
+      }
       Polymer.dom(parentNode).insertBefore(inst.root, beforeNode);
       this._instances[idx] = inst;
       return inst;
diff --git a/test/unit/dom-bind.html b/test/unit/dom-bind.html
index 4539493f56..40e9d31b3a 100644
--- a/test/unit/dom-bind.html
+++ b/test/unit/dom-bind.html
@@ -21,7 +21,7 @@
   </template>
 
   <script>
-  /* global earlyDomBind earlyBoundChild decEl1 decEl2 decDomBind z container needsHost*/
+  /* global earlyDomBind earlyBoundChild decEl1 decEl2 decDomBind z container needsHost hybridDomBind hybridEl1 hybridEl2*/
     earlyDomBind.value = 'hi!';
   </script>
 
@@ -42,6 +42,13 @@
     <x-needs-host id="needsHost"></x-needs-host>
   </template>
 
+  <dom-bind>
+    <template is="dom-bind" id="hybridDomBind">
+      <x-basic value="{{value}}" id="hybridEl1"></x-basic>
+      <x-basic value="{{value}}" id="hybridEl2"></x-basic>
+    </template>
+  </dom-bind>
+
   <script>
 
     suite('dom-bind touched before upgrade', function() {
@@ -235,7 +242,7 @@
             Polymer.Settings.suppressTemplateNotifications ? 0 : 1);
           done();
         });
-      });        
+      });
 
       test('re-enable dom-change', function(done) {
         if (Polymer.Settings.suppressTemplateNotifications) {
@@ -255,6 +262,32 @@
 
     });
 
+    suite('hybrid dom-bind', function() {
+
+      var domBind;
+      var el1;
+      var el2;
+
+      setup(function() {
+        domBind = hybridDomBind;
+        el1 = hybridEl1;
+        el2 = hybridEl2;
+      });
+
+      test('value binds top-down', function() {
+        domBind.value = 'foo';
+        assert.equal(el1.value, 'foo');
+        assert.equal(el2.value, 'foo');
+      });
+
+      test('parent is parent of <dom-bind>', function() {
+        domBind.value = 'foo';
+        assert.equal(el1.parentNode, document.body);
+        assert.equal(el2.parentNode, document.body);
+      });
+
+    });
+
   </script>
 
   <link rel="import" href="dom-bind-elements2.html">
diff --git a/test/unit/dom-if.html b/test/unit/dom-if.html
index 39dfcfbf14..3666bde095 100644
--- a/test/unit/dom-if.html
+++ b/test/unit/dom-if.html
@@ -71,8 +71,33 @@
     <template is="dom-if" if id="toBeRemoved"><div id="shouldBeRemoved"></div></template>
   </div>
 
+  <!-- 2.x hybrid test -->
+  <dom-if>
+    <template is="dom-if" id="hybridDomIf" if restamp>
+      <div hybrid-stamped>[[prop]]</div>
+    </template>
+  </dom-if>
+
   <script>
-    /* global configured individual unconfigured1 unconfigured2 inDocumentContainer inDocumentIf structuredContainer structuredDomIf structuredDomBind outerContainer innerContainer shouldBeRemoved toBeRemoved removalContainer*/
+    /* global configured individual unconfigured1 unconfigured2 inDocumentContainer inDocumentIf structuredContainer structuredDomIf structuredDomBind outerContainer innerContainer shouldBeRemoved toBeRemoved removalContainer hybridDomIf*/
+
+    suite('hybrid dom-if', function() {
+
+      test('parent is parent of <dom-if>', function() {
+        var stamped = Array.from(document.querySelectorAll('[hybrid-stamped]'));
+        assert.equal(stamped.length, 1);
+        assert.equal(stamped[0].parentNode, document.body);
+      });
+
+      test('children removed correctly', function() {
+        hybridDomIf.if = false;
+        hybridDomIf.render();
+        var stamped = Array.from(document.querySelectorAll('[hybrid-stamped]'));
+        assert.equal(stamped.length, 0);
+      });
+
+    });
+
     suite('nested pre-configured dom-if', function() {
 
       test('parent scope binding', function() {
@@ -337,7 +362,7 @@
         var stamped = Polymer.dom(unconfigured1.root).querySelectorAll('*:not(template):not(span)');
         assert.equal(stamped.length, 3, 'total stamped count incorrect');
         stamped[0].prop = 'outer';
-        assert.equal(unconfigured1.domUpdateHandlerCount, 
+        assert.equal(unconfigured1.domUpdateHandlerCount,
           Polymer.Settings.suppressTemplateNotifications ? 0 : 1);
       });
 
@@ -352,7 +377,7 @@
         stamped.forEach(function(n) {
           assert.equal(getComputedStyle(n).display, 'none', 'node was not hidden but should have been');
         });
-        assert.equal(unconfigured1.domUpdateHandlerCount, 
+        assert.equal(unconfigured1.domUpdateHandlerCount,
           Polymer.Settings.suppressTemplateNotifications ? 0 : 1);
       });
 
@@ -368,7 +393,7 @@
         stamped.forEach(function(n) {
           assert.equal(getComputedStyle(n).display, 'inline', 'node was hidden but should not have been');
         });
-        assert.equal(unconfigured1.domUpdateHandlerCount, 
+        assert.equal(unconfigured1.domUpdateHandlerCount,
           Polymer.Settings.suppressTemplateNotifications ? 0 : 1);
       });
 
@@ -380,10 +405,10 @@
         CustomElements.takeRecords();
         CustomElements.takeRecords();
         var stamped = Polymer.dom(unconfigured1.root).querySelectorAll('*:not(template)');
-        assert.equal(unconfigured1.domUpdateHandlerCount, 
+        assert.equal(unconfigured1.domUpdateHandlerCount,
           Polymer.Settings.suppressTemplateNotifications ? 0 : 1);
         assert.equal(stamped.length, 0, 'total stamped count incorrect');
-        assert.equal(unconfigured1.domUpdateHandlerCount, 
+        assert.equal(unconfigured1.domUpdateHandlerCount,
           Polymer.Settings.suppressTemplateNotifications ? 0 : 1);
       });
 
@@ -398,7 +423,7 @@
         var stamped = Polymer.dom(unconfigured1.root).querySelectorAll('*:not(template):not(span)');
         assert.equal(stamped.length, 3, 'total stamped count incorrect');
         stamped[0].prop = 'outer';
-        assert.equal(unconfigured1.domUpdateHandlerCount, 
+        assert.equal(unconfigured1.domUpdateHandlerCount,
           Polymer.Settings.suppressTemplateNotifications ? 0 : 1);
       });
 
diff --git a/test/unit/dom-repeat.html b/test/unit/dom-repeat.html
index 97edd6b01f..744c516066 100644
--- a/test/unit/dom-repeat.html
+++ b/test/unit/dom-repeat.html
@@ -80,8 +80,15 @@ <h4>x-repeat-chunked</h4>
   <div id="inDocumentContainer">
   </div>
 
+  <!-- 2.x hybrid test -->
+  <dom-repeat>
+    <template is="dom-repeat" id="hybridDomRepeat" items='[{"prop":"a"},{"prop":"b"},{"prop":"c"}]'>
+      <div hybrid-stamped>[[item.prop]]</div>
+    </template>
+  </dom-repeat>
+
   <script>
-  /* global unconfigured1 unconfigured primitive limited inDocumentRepeater configured inDocumentContainer inDocumentContainerOrig unconfigured2 primitiveLarge simple:true model:true stamped:true chunked */
+  /* global unconfigured1 unconfigured primitive limited inDocumentRepeater configured inDocumentContainer inDocumentContainerOrig unconfigured2 primitiveLarge simple:true model:true stamped:true chunked hybridDomRepeat*/
 
     /*
       Expected:
@@ -100,7 +107,6 @@ <h4>x-repeat-chunked</h4>
       stamped[38] .. 3-3-3
     */
 
-
     suite('errors', function() {
 
       test('items must be array', function() {
@@ -830,7 +836,7 @@ <h4>x-repeat-chunked</h4>
           assert.equal(stamped2[39].itemaProp, 'new-1');
           assert.equal(stamped2[40].itemaProp, 'new-2');
           assert.equal(stamped2[41], undefined);
-          assert.equal(unconfigured1.domUpdateHandlerCount, 
+          assert.equal(unconfigured1.domUpdateHandlerCount,
             Polymer.Settings.suppressTemplateNotifications ? 0 : 1);
           done();
         });
@@ -854,7 +860,7 @@ <h4>x-repeat-chunked</h4>
           assert.equal(stamped2[38].indexb, 2);
           assert.equal(stamped2[38].indexc, 2);
           assert.equal(stamped2[39], undefined);
-          assert.equal(unconfigured1.domUpdateHandlerCount, 
+          assert.equal(unconfigured1.domUpdateHandlerCount,
             Polymer.Settings.suppressTemplateNotifications ? 0 : 1);
           done();
         });
@@ -3870,22 +3876,41 @@ <h4>x-repeat-chunked</h4>
 
       });
 
-      suite('notification suppression', function() {
+    }
 
-        test('re-enable dom-change', function() {
-          if (Polymer.Settings.suppressTemplateNotifications) {
-            unconfigured1.$.repeater.setAttribute('notify-dom-change', '');
-            unconfigured1.domUpdateHandlerCount = 0;
-            unconfigured.items = unconfigured.items.slice();
-            setTimeout(function() {
-              assert.equal(unconfigured1.domUpdateHandlerCount, 1);
-            });
-          }
+    suite('notification suppression', function() {
+
+      test('re-enable dom-change', function() {
+        if (Polymer.Settings.suppressTemplateNotifications) {
+          unconfigured1.$.repeater.setAttribute('notify-dom-change', '');
+          unconfigured1.domUpdateHandlerCount = 0;
+          unconfigured.items = unconfigured.items.slice();
+          setTimeout(function() {
+            assert.equal(unconfigured1.domUpdateHandlerCount, 1);
+          });
+        }
+      });
+
+    });
+
+    suite('hybrid dom-repeat', function() {
+
+      test('parent is parent of <dom-repeat>', function() {
+        var stamped = Array.from(document.querySelectorAll('[hybrid-stamped]'));
+        assert.equal(stamped.length, 3);
+        stamped.forEach(function(el) {
+          assert.equal(el.parentNode, document.body);
         });
+      });
 
+      test('children removed correctly', function() {
+        hybridDomRepeat.items = null;
+        hybridDomRepeat.render();
+        var stamped = Array.from(document.querySelectorAll('[hybrid-stamped]'));
+        assert.equal(stamped.length, 0);
       });
 
-    }
+    });
 
   </script>
 

From 4c87e1d96c849dfd0dcf56ba8b6285c540882f21 Mon Sep 17 00:00:00 2001
From: Kevin Schaaf <kschaaf@google.com>
Date: Fri, 14 Apr 2017 18:16:19 -0700
Subject: [PATCH 2/9] Add test for add & remove

---
 src/lib/template/dom-repeat.html | 17 +++++++++++++----
 test/unit/dom-repeat.html        | 17 +++++++++++++++--
 2 files changed, 28 insertions(+), 6 deletions(-)

diff --git a/src/lib/template/dom-repeat.html b/src/lib/template/dom-repeat.html
index f8e3b2bdc3..8a7f840bad 100644
--- a/src/lib/template/dom-repeat.html
+++ b/src/lib/template/dom-repeat.html
@@ -285,9 +285,18 @@
       // only perform attachment if the element was previously detached.
       if (this.__isDetached) {
         this.__isDetached = false;
-        var parent = Polymer.dom(Polymer.dom(this).parentNode);
+        var refNode;
+        var parentNode = Polymer.dom(this).parentNode;
+        // Affordance for 2.x hybrid mode
+        if (parentNode.localName == this.is) {
+          refNode = parentNode;
+          parentNode = Polymer.dom(parentNode).parentNode;
+        } else {
+          refNode = this;
+        }
+        var parent = Polymer.dom(parentNode);
         for (var i=0; i<this._instances.length; i++) {
-          this._attachInstance(i, parent);
+          this._attachInstance(i, parent, refNode);
         }
       }
     },
@@ -666,10 +675,10 @@
       }
     },
 
-    _attachInstance: function(idx, parent) {
+    _attachInstance: function(idx, parent, refNode) {
       var inst = this._instances[idx];
       if (!inst.isPlaceholder) {
-        parent.insertBefore(inst.root, this);
+        parent.insertBefore(inst.root, refNode);
       }
     },
 
diff --git a/test/unit/dom-repeat.html b/test/unit/dom-repeat.html
index 744c516066..7ff3540dc9 100644
--- a/test/unit/dom-repeat.html
+++ b/test/unit/dom-repeat.html
@@ -81,14 +81,14 @@ <h4>x-repeat-chunked</h4>
   </div>
 
   <!-- 2.x hybrid test -->
-  <dom-repeat>
+  <dom-repeat id="hybridDomRepeatWrapper">
     <template is="dom-repeat" id="hybridDomRepeat" items='[{"prop":"a"},{"prop":"b"},{"prop":"c"}]'>
       <div hybrid-stamped>[[item.prop]]</div>
     </template>
   </dom-repeat>
 
   <script>
-  /* global unconfigured1 unconfigured primitive limited inDocumentRepeater configured inDocumentContainer inDocumentContainerOrig unconfigured2 primitiveLarge simple:true model:true stamped:true chunked hybridDomRepeat*/
+  /* global unconfigured1 unconfigured primitive limited inDocumentRepeater configured inDocumentContainer inDocumentContainerOrig unconfigured2 primitiveLarge simple:true model:true stamped:true chunked hybridDomRepeat hybridDomRepeatWrapper*/
 
     /*
       Expected:
@@ -3903,6 +3903,19 @@ <h4>x-repeat-chunked</h4>
         });
       });
 
+      test('remove & re-add works correctly', function() {
+        var wrapper = hybridDomRepeatWrapper;
+        Polymer.dom(document.body).removeChild(wrapper);
+        Polymer.dom(document.body).appendChild(wrapper);
+        CustomElements.takeRecords();
+        hybridDomRepeat.render();
+        var stamped = Array.from(document.querySelectorAll('[hybrid-stamped]'));
+        assert.equal(stamped.length, 3);
+        stamped.forEach(function(el) {
+          assert.equal(el.parentNode, document.body);
+        });
+      });
+
       test('children removed correctly', function() {
         hybridDomRepeat.items = null;
         hybridDomRepeat.render();

From a2825650549317b82df8dcd095f3ce64729ea863 Mon Sep 17 00:00:00 2001
From: Kevin Schaaf <kschaaf@google.com>
Date: Fri, 14 Apr 2017 18:19:38 -0700
Subject: [PATCH 3/9] Add dom-if test for add/remove.

---
 test/unit/dom-if.html | 15 +++++++++++++--
 1 file changed, 13 insertions(+), 2 deletions(-)

diff --git a/test/unit/dom-if.html b/test/unit/dom-if.html
index 3666bde095..ef4a1baddd 100644
--- a/test/unit/dom-if.html
+++ b/test/unit/dom-if.html
@@ -72,14 +72,14 @@
   </div>
 
   <!-- 2.x hybrid test -->
-  <dom-if>
+  <dom-if id="hybridDomIfWrapper">
     <template is="dom-if" id="hybridDomIf" if restamp>
       <div hybrid-stamped>[[prop]]</div>
     </template>
   </dom-if>
 
   <script>
-    /* global configured individual unconfigured1 unconfigured2 inDocumentContainer inDocumentIf structuredContainer structuredDomIf structuredDomBind outerContainer innerContainer shouldBeRemoved toBeRemoved removalContainer hybridDomIf*/
+    /* global configured individual unconfigured1 unconfigured2 inDocumentContainer inDocumentIf structuredContainer structuredDomIf structuredDomBind outerContainer innerContainer shouldBeRemoved toBeRemoved removalContainer hybridDomIf hybridDomIfWrapper*/
 
     suite('hybrid dom-if', function() {
 
@@ -89,6 +89,17 @@
         assert.equal(stamped[0].parentNode, document.body);
       });
 
+      test('remove & re-add works correctly', function() {
+        var wrapper = hybridDomIfWrapper;
+        Polymer.dom(document.body).removeChild(wrapper);
+        Polymer.dom(document.body).appendChild(wrapper);
+        CustomElements.takeRecords();
+        hybridDomIf.render();
+        var stamped = Array.from(document.querySelectorAll('[hybrid-stamped]'));
+        assert.equal(stamped.length, 1);
+        assert.equal(stamped[0].parentNode, document.body);
+      });
+
       test('children removed correctly', function() {
         hybridDomIf.if = false;
         hybridDomIf.render();

From 2722532b51cf7b18f6d9e59454f70af8282ed7c6 Mon Sep 17 00:00:00 2001
From: Kevin Schaaf <kschaaf@google.com>
Date: Fri, 14 Apr 2017 18:42:03 -0700
Subject: [PATCH 4/9] Fix dom-if detachment

---
 src/lib/template/dom-if.html |  8 ++++++--
 test/unit/dom-if.html        | 17 ++++++++++++-----
 test/unit/dom-repeat.html    | 23 ++++++++++++++++++++---
 3 files changed, 38 insertions(+), 10 deletions(-)

diff --git a/src/lib/template/dom-if.html b/src/lib/template/dom-if.html
index 8952e8be5a..01dce89f90 100644
--- a/src/lib/template/dom-if.html
+++ b/src/lib/template/dom-if.html
@@ -82,8 +82,12 @@
     },
 
     detached: function() {
-      if (!this.parentNode ||
-          (this.parentNode.nodeType == Node.DOCUMENT_FRAGMENT_NODE &&
+      var parentNode = this.parentNode;
+      if (parentNode && parentNode.localName == this.is) {
+        parentNode = Polymer.dom(parentNode).parentNode;
+      }
+      if (!parentNode ||
+          (parentNode.nodeType == Node.DOCUMENT_FRAGMENT_NODE &&
            (!Polymer.Settings.hasShadow ||
             !(this.parentNode instanceof ShadowRoot)))) {
         this._teardownInstance();
diff --git a/test/unit/dom-if.html b/test/unit/dom-if.html
index ef4a1baddd..fc98e3a221 100644
--- a/test/unit/dom-if.html
+++ b/test/unit/dom-if.html
@@ -89,15 +89,22 @@
         assert.equal(stamped[0].parentNode, document.body);
       });
 
-      test('remove & re-add works correctly', function() {
+      test('remove & re-add works correctly', function(done) {
         var wrapper = hybridDomIfWrapper;
+        var domIf = hybridDomIf;
         Polymer.dom(document.body).removeChild(wrapper);
-        Polymer.dom(document.body).appendChild(wrapper);
         CustomElements.takeRecords();
-        hybridDomIf.render();
+        domIf.render();
         var stamped = Array.from(document.querySelectorAll('[hybrid-stamped]'));
-        assert.equal(stamped.length, 1);
-        assert.equal(stamped[0].parentNode, document.body);
+        assert.equal(stamped.length, 0);
+        Polymer.dom(removalContainer).appendChild(wrapper);
+        CustomElements.takeRecords();
+        setTimeout(function() {
+          stamped = Array.from(document.querySelectorAll('[hybrid-stamped]'));
+          assert.equal(stamped.length, 1);
+          assert.equal(stamped[0].parentNode, removalContainer);
+          done();
+        });
       });
 
       test('children removed correctly', function() {
diff --git a/test/unit/dom-repeat.html b/test/unit/dom-repeat.html
index 7ff3540dc9..fb1b841b31 100644
--- a/test/unit/dom-repeat.html
+++ b/test/unit/dom-repeat.html
@@ -3898,21 +3898,38 @@ <h4>x-repeat-chunked</h4>
       test('parent is parent of <dom-repeat>', function() {
         var stamped = Array.from(document.querySelectorAll('[hybrid-stamped]'));
         assert.equal(stamped.length, 3);
-        stamped.forEach(function(el) {
+        stamped.forEach(function(el, i) {
           assert.equal(el.parentNode, document.body);
+          assert.equal(el.textContent, hybridDomRepeat.items[i].prop);
         });
       });
 
       test('remove & re-add works correctly', function() {
         var wrapper = hybridDomRepeatWrapper;
         Polymer.dom(document.body).removeChild(wrapper);
-        Polymer.dom(document.body).appendChild(wrapper);
         CustomElements.takeRecords();
         hybridDomRepeat.render();
         var stamped = Array.from(document.querySelectorAll('[hybrid-stamped]'));
+        assert.equal(stamped.length, 0);
+        Polymer.dom(document.body).appendChild(wrapper);
+        CustomElements.takeRecords();
+        hybridDomRepeat.render();
+        stamped = Array.from(document.querySelectorAll('[hybrid-stamped]'));
         assert.equal(stamped.length, 3);
-        stamped.forEach(function(el) {
+        stamped.forEach(function(el, i) {
+          assert.equal(el.parentNode, document.body);
+          assert.equal(el.textContent, hybridDomRepeat.items[i].prop);
+        });
+      });
+
+      test('insertion works correctly', function() {
+        hybridDomRepeat.splice('items', 1, 0, {prop: 'new'});
+        hybridDomRepeat.render();
+        var stamped = Array.from(document.querySelectorAll('[hybrid-stamped]'));
+        assert.equal(stamped.length, 4);
+        stamped.forEach(function(el, i) {
           assert.equal(el.parentNode, document.body);
+          assert.equal(el.textContent, hybridDomRepeat.items[i].prop);
         });
       });
 

From ee3b9a698d05174f2d78f17452bfe9f7cadf7001 Mon Sep 17 00:00:00 2001
From: Kevin Schaaf <kschaaf@google.com>
Date: Mon, 17 Apr 2017 10:45:09 -0700
Subject: [PATCH 5/9] Capture hybridDomRepeat.

---
 test/unit/dom-repeat.html | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/test/unit/dom-repeat.html b/test/unit/dom-repeat.html
index fb1b841b31..2abba7dc40 100644
--- a/test/unit/dom-repeat.html
+++ b/test/unit/dom-repeat.html
@@ -3905,20 +3905,21 @@ <h4>x-repeat-chunked</h4>
       });
 
       test('remove & re-add works correctly', function() {
+        var domRepeat = hybridDomRepeat;
         var wrapper = hybridDomRepeatWrapper;
         Polymer.dom(document.body).removeChild(wrapper);
         CustomElements.takeRecords();
-        hybridDomRepeat.render();
+        domRepeat.render();
         var stamped = Array.from(document.querySelectorAll('[hybrid-stamped]'));
         assert.equal(stamped.length, 0);
         Polymer.dom(document.body).appendChild(wrapper);
         CustomElements.takeRecords();
-        hybridDomRepeat.render();
+        domRepeat.render();
         stamped = Array.from(document.querySelectorAll('[hybrid-stamped]'));
         assert.equal(stamped.length, 3);
         stamped.forEach(function(el, i) {
           assert.equal(el.parentNode, document.body);
-          assert.equal(el.textContent, hybridDomRepeat.items[i].prop);
+          assert.equal(el.textContent, domRepeat.items[i].prop);
         });
       });
 

From 2e4290f8381b53d692bd15da948e43b78ff48efc Mon Sep 17 00:00:00 2001
From: Kevin Schaaf <kschaaf@google.com>
Date: Mon, 17 Apr 2017 10:53:22 -0700
Subject: [PATCH 6/9] Use local `parentNode`

---
 src/lib/template/dom-if.html | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/lib/template/dom-if.html b/src/lib/template/dom-if.html
index 01dce89f90..9d699a22c7 100644
--- a/src/lib/template/dom-if.html
+++ b/src/lib/template/dom-if.html
@@ -89,7 +89,7 @@
       if (!parentNode ||
           (parentNode.nodeType == Node.DOCUMENT_FRAGMENT_NODE &&
            (!Polymer.Settings.hasShadow ||
-            !(this.parentNode instanceof ShadowRoot)))) {
+            !(parentNode instanceof ShadowRoot)))) {
         this._teardownInstance();
       }
     },

From b7f009922c3e88df6eac82eb032ddef0d8ef4e33 Mon Sep 17 00:00:00 2001
From: Kevin Schaaf <kschaaf@google.com>
Date: Mon, 17 Apr 2017 11:10:55 -0700
Subject: [PATCH 7/9] Ensure optimization uses hybrid parentNode

---
 src/lib/template/dom-if.html | 18 +++++++++---------
 1 file changed, 9 insertions(+), 9 deletions(-)

diff --git a/src/lib/template/dom-if.html b/src/lib/template/dom-if.html
index 9d699a22c7..c06ddef99f 100644
--- a/src/lib/template/dom-if.html
+++ b/src/lib/template/dom-if.html
@@ -135,17 +135,17 @@
     },
 
     _ensureInstance: function() {
+      var refNode;
       var parentNode = Polymer.dom(this).parentNode;
+      // Affordance for 2.x hybrid mode
+      if (parentNode && parentNode.localName == this.is) {
+        refNode = parentNode;
+        parentNode = Polymer.dom(parentNode).parentNode;
+      } else {
+        refNode = this;
+      }
       // Guard against element being detached while render was queued
       if (parentNode) {
-        var refNode;
-        // Affordance for 2.x hybrid mode
-        if (parentNode.localName == this.is) {
-          refNode = parentNode;
-          parentNode = Polymer.dom(parentNode).parentNode;
-        } else {
-          refNode = this;
-        }
         if (!this._instance) {
           this._instance = this.stamp();
           var root = this._instance.root;
@@ -154,7 +154,7 @@
           var c$ = this._instance._children;
           if (c$ && c$.length) {
             // Detect case where dom-if was re-attached in new position
-            var lastChild = Polymer.dom(this).previousSibling;
+            var lastChild = Polymer.dom(refNode).previousSibling;
             if (lastChild !== c$[c$.length-1]) {
               for (var i=0, n; (i<c$.length) && (n=c$[i]); i++) {
                 Polymer.dom(parentNode).insertBefore(n, refNode);

From 646dce69e48b310bc5f7cffafdf952beaee673fc Mon Sep 17 00:00:00 2001
From: Kevin Schaaf <kschaaf@google.com>
Date: Mon, 17 Apr 2017 12:10:26 -0700
Subject: [PATCH 8/9] Remove use of ES6 API.

---
 test/unit/dom-repeat.html | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/test/unit/dom-repeat.html b/test/unit/dom-repeat.html
index 2abba7dc40..b8e143a88c 100644
--- a/test/unit/dom-repeat.html
+++ b/test/unit/dom-repeat.html
@@ -3896,7 +3896,7 @@ <h4>x-repeat-chunked</h4>
     suite('hybrid dom-repeat', function() {
 
       test('parent is parent of <dom-repeat>', function() {
-        var stamped = Array.from(document.querySelectorAll('[hybrid-stamped]'));
+        var stamped = [].slice.call(document.querySelectorAll('[hybrid-stamped]'));
         assert.equal(stamped.length, 3);
         stamped.forEach(function(el, i) {
           assert.equal(el.parentNode, document.body);
@@ -3910,12 +3910,12 @@ <h4>x-repeat-chunked</h4>
         Polymer.dom(document.body).removeChild(wrapper);
         CustomElements.takeRecords();
         domRepeat.render();
-        var stamped = Array.from(document.querySelectorAll('[hybrid-stamped]'));
+        var stamped = [].slice.call(document.querySelectorAll('[hybrid-stamped]'));
         assert.equal(stamped.length, 0);
         Polymer.dom(document.body).appendChild(wrapper);
         CustomElements.takeRecords();
         domRepeat.render();
-        stamped = Array.from(document.querySelectorAll('[hybrid-stamped]'));
+        stamped = [].slice.call(document.querySelectorAll('[hybrid-stamped]'));
         assert.equal(stamped.length, 3);
         stamped.forEach(function(el, i) {
           assert.equal(el.parentNode, document.body);
@@ -3926,7 +3926,7 @@ <h4>x-repeat-chunked</h4>
       test('insertion works correctly', function() {
         hybridDomRepeat.splice('items', 1, 0, {prop: 'new'});
         hybridDomRepeat.render();
-        var stamped = Array.from(document.querySelectorAll('[hybrid-stamped]'));
+        var stamped = [].slice.call(document.querySelectorAll('[hybrid-stamped]'));
         assert.equal(stamped.length, 4);
         stamped.forEach(function(el, i) {
           assert.equal(el.parentNode, document.body);
@@ -3937,7 +3937,7 @@ <h4>x-repeat-chunked</h4>
       test('children removed correctly', function() {
         hybridDomRepeat.items = null;
         hybridDomRepeat.render();
-        var stamped = Array.from(document.querySelectorAll('[hybrid-stamped]'));
+        var stamped = [].slice.call(document.querySelectorAll('[hybrid-stamped]'));
         assert.equal(stamped.length, 0);
       });
 

From 960106572ebb82021ef88865294df7f4a134d49d Mon Sep 17 00:00:00 2001
From: Kevin Schaaf <kschaaf@google.com>
Date: Mon, 17 Apr 2017 12:47:09 -0700
Subject: [PATCH 9/9] Remove use of ES6 API.

---
 test/unit/dom-if.html | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/test/unit/dom-if.html b/test/unit/dom-if.html
index fc98e3a221..33e8d80e21 100644
--- a/test/unit/dom-if.html
+++ b/test/unit/dom-if.html
@@ -84,7 +84,7 @@
     suite('hybrid dom-if', function() {
 
       test('parent is parent of <dom-if>', function() {
-        var stamped = Array.from(document.querySelectorAll('[hybrid-stamped]'));
+        var stamped = [].slice.call(document.querySelectorAll('[hybrid-stamped]'));
         assert.equal(stamped.length, 1);
         assert.equal(stamped[0].parentNode, document.body);
       });
@@ -95,12 +95,12 @@
         Polymer.dom(document.body).removeChild(wrapper);
         CustomElements.takeRecords();
         domIf.render();
-        var stamped = Array.from(document.querySelectorAll('[hybrid-stamped]'));
+        var stamped = [].slice.call(document.querySelectorAll('[hybrid-stamped]'));
         assert.equal(stamped.length, 0);
         Polymer.dom(removalContainer).appendChild(wrapper);
         CustomElements.takeRecords();
         setTimeout(function() {
-          stamped = Array.from(document.querySelectorAll('[hybrid-stamped]'));
+          stamped = [].slice.call(document.querySelectorAll('[hybrid-stamped]'));
           assert.equal(stamped.length, 1);
           assert.equal(stamped[0].parentNode, removalContainer);
           done();
@@ -110,7 +110,7 @@
       test('children removed correctly', function() {
         hybridDomIf.if = false;
         hybridDomIf.render();
-        var stamped = Array.from(document.querySelectorAll('[hybrid-stamped]'));
+        var stamped = [].slice.call(document.querySelectorAll('[hybrid-stamped]'));
         assert.equal(stamped.length, 0);
       });