Remove all TODOs

This commit is contained in:
Steven Orvell 2015-12-07 12:49:45 -08:00
parent f356ba894e
commit 6467ae11c4

View File

@ -73,7 +73,6 @@ subject to an additional IP rights grant found at http://polymer.github.io/PATEN
},
detached: function() {
// TODO(kschaaf): add logic to re-stamp in attached?
if (!this.parentNode) {
this._teardownInstance();
}
@ -81,7 +80,7 @@ subject to an additional IP rights grant found at http://polymer.github.io/PATEN
attached: function() {
if (this.if && this.ctor) {
// TODO(sorvell): should not be async, but node can be attached
// NOTE: ideally should not be async, but node can be attached
// when shady dom is in the act of distributing/composing so push it out
this.async(this._ensureInstance);
}
@ -123,10 +122,8 @@ subject to an additional IP rights grant found at http://polymer.github.io/PATEN
if (parentNode) {
var parent = Polymer.dom(parentNode);
if (!this._instance) {
// TODO(sorvell): pickup stamping logic from x-repeat
this._instance = this.stamp();
var root = this._instance.root;
// TODO(sorvell): this incantation needs to be simpler.
parent.insertBefore(root, this);
} else {
var c$ = this._instance._children;