fix(FlattenedNodesObserver): do not fail on node without children

Polymer · Dec 19, 2017 · 09bb6cd · 09bb6cd
1 parent 8196483
commit 09bb6cd
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 2 deletions.
diff --git a/lib/utils/flattened-nodes-observer.html b/lib/utils/flattened-nodes-observer.html
@@ -130,7 +130,7 @@
     connect() {
       if (isSlot(this._target)) {
         this._listenSlots([this._target]);
-      } else {
+      } else if (this._target.children) {
         this._listenSlots(this._target.children);
         if (window.ShadyDOM) {
           this._shadyChildrenObserver =
@@ -159,7 +159,7 @@
     disconnect() {
       if (isSlot(this._target)) {
         this._unlistenSlots([this._target]);
-      } else {
+      } else if (this._target.children) {
         this._unlistenSlots(this._target.children);
         if (window.ShadyDOM && this._shadyChildrenObserver) {
           ShadyDOM.unobserveChildren(this._shadyChildrenObserver);

diff --git a/test/unit/flattened-nodes-observer.html b/test/unit/flattened-nodes-observer.html
@@ -1039,6 +1039,16 @@
       document.body.removeChild(host);
     });
 
+    test('should not fail on node without children', function() {
+      var recorded;
+      var el = document;
+      var observer = new Polymer.FlattenedNodesObserver(el, function(info) {
+        recorded = info;
+      });
+      assert.equal(recorded, null);
+      observer.disconnect();
+    });
+
   });
 
 </script>