Skip to content

Commit 4eda393

Browse files
committed
Simplify.
1 parent 15c1241 commit 4eda393

1 file changed

Lines changed: 13 additions & 23 deletions

File tree

‎src/lib/template/dom-repeat.html‎

Lines changed: 13 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -204,19 +204,15 @@
204204
},
205205

206206
detached: function() {
207-
if (this._instances) {
208-
for (var i=0; i<this._instances.length; i++) {
209-
this._detachRow(i);
210-
}
207+
for (var i=0; i<this._instances.length; i++) {
208+
this._detachRow(i);
211209
}
212210
},
213211

214212
attached: function() {
215-
if (this._instances) {
216-
var parentNode = Polymer.dom(this).parentNode;
217-
for (var i=0; i<this._instances.length; i++) {
218-
Polymer.dom(parentNode).insertBefore(this._instances[i].root, this);
219-
}
213+
var parentNode = Polymer.dom(this).parentNode;
214+
for (var i=0; i<this._instances.length; i++) {
215+
Polymer.dom(parentNode).insertBefore(this._instances[i].root, this);
220216
}
221217
},
222218

@@ -567,10 +563,8 @@
567563

568564
// Implements extension point from Templatizer mixin
569565
_showHideChildren: function(hidden) {
570-
if (this._instances) {
571-
for (var i=0; i<this._instances.length; i++) {
572-
this._instances[i]._showHideChildren(hidden);
573-
}
566+
for (var i=0; i<this._instances.length; i++) {
567+
this._instances[i]._showHideChildren(hidden);
574568
}
575569
},
576570

@@ -607,22 +601,18 @@
607601
// Called as side-effect of a host property change, responsible for
608602
// notifying parent path change on each inst
609603
_forwardParentProp: function(prop, value) {
610-
if (this._instances) {
611-
this._instances.forEach(function(inst) {
612-
inst.__setProperty(prop, value, true);
613-
}, this);
614-
}
604+
this._instances.forEach(function(inst) {
605+
inst.__setProperty(prop, value, true);
606+
}, this);
615607
},
616608

617609
// Implements extension point from Templatizer
618610
// Called as side-effect of a host path change, responsible for
619611
// notifying parent path change on each inst
620612
_forwardParentPath: function(path, value) {
621-
if (this._instances) {
622-
this._instances.forEach(function(inst) {
623-
inst.notifyPath(path, value, true);
624-
}, this);
625-
}
613+
this._instances.forEach(function(inst) {
614+
inst.notifyPath(path, value, true);
615+
}, this);
626616
},
627617

628618
// Called as a side effect of a host items.<key>.<path> path change,

0 commit comments

Comments
 (0)