Ensure detached cannot be called when disble-upgrade is set

src/mini/ready.html

@@ -32,6 +32,7 @@
 (function() {
 
   var baseAttachedCallback = Polymer.Base.attachedCallback;
+  var baseDetachedCallback = Polymer.Base.detachedCallback;
 
   Polymer.Base._addFeature({
 
@@ -175,6 +176,23 @@
       } else {
         this._attachedPending = true;
       }
+    },
+
+    /**
+     * Polymer library implementation of the Custom Elements `detachedCallback`.
+     *
+     * Note, users should not override `detachedCallback`, and instead should
+     * implement the `detached` method on Polymer elements to receive
+     * detached-time callbacks.
+     *
+     * @protected
+     */
+    detachedCallback: function() {
+      if (this._readied) {
+        baseDetachedCallback.call(this);
+      } else {
+        this._attachedPending = false;
+      }
     }
 
   });

test/unit/element-disable-upgrade.html

@@ -32,12 +32,44 @@
   </script>
 </dom-module>
 
+<dom-module id="x-attach">
+  <template>
+    <style>
+    :host {
+      display: block;
+    }
+    </style>
+    <div id="child">Live!</div>
+  </template>
+  <script>
+  HTMLImports.whenReady(function() {
+    Polymer({
+      is: 'x-attach',
+      attached: function() {
+        this._connectLog = this._connectLog || [];
+        this._connectLog.push('attached');
+      },
+      detached: function() {
+        this._connectLog = this._connectLog || [];
+        this._connectLog.push('detached');
+      }
+    });
+  });
+  </script>
+</dom-module>
+
 <test-fixture id="simple">
   <template>
     <x-lazy disable-upgrade></x-lazy>
   </template>
 </test-fixture>
 
+<test-fixture id="attach">
+  <template>
+    <x-attach disable-upgrade></x-attach>
+  </template>
+</test-fixture>
+
 <dom-module id="x-complicated-child">
   <script>
   HTMLImports.whenReady(function() {
@@ -141,6 +173,59 @@
       assert.ok(el.$.child);
     });
   });
+  suite('disableUpgrade attach/detach', function() {
+    var el;
+    setup(function() {
+      el = fixture('attach');
+    });
+    test('attached does not fire when element is not yet enabled', function() {
+      assert.notOk(el._connectLog);
+      el.removeAttribute('disable-upgrade');
+      assert.equal(el._connectLog.length, 1);
+      assert.equal(el._connectLog[0], 'attached');
+    });
+    test('attached/detached do not fire when element is not yet enabled', function() {
+      el.parentNode.removeChild(el);
+      Polymer.dom.flush();
+      assert.notOk(el._connectLog);
+      el.removeAttribute('disable-upgrade');
+      assert.notOk(el._connectLog);
+    });
+    test('attached/detached do not fire when element is detached/attached when not yet enabled', function() {
+      var parent = el.parentNode;
+      parent.removeChild(el);
+      Polymer.dom.flush();
+      assert.notOk(el._connectLog);
+      parent.appendChild(el);
+      Polymer.dom.flush();
+      assert.notOk(el._connectLog);
+      assert.notOk(el._connectLog);
+      el.removeAttribute('disable-upgrade');
+      assert.equal(el._connectLog.length, 1);
+      assert.equal(el._connectLog[0], 'attached');
+    });
+
+    test('attached/detached fire as expected after element is enabled', function() {
+      var parent = el.parentNode;
+      parent.removeChild(el);
+      Polymer.dom.flush();
+      assert.notOk(el._connectLog);
+      parent.appendChild(el);
+      Polymer.dom.flush();
+      assert.notOk(el._connectLog);
+      el.removeAttribute('disable-upgrade');
+      assert.equal(el._connectLog.length, 1);
+      assert.equal(el._connectLog[0], 'attached');
+      parent.removeChild(el);
+      Polymer.dom.flush();
+      assert.equal(el._connectLog.length, 2);
+      assert.equal(el._connectLog[1], 'detached');
+      parent.appendChild(el);
+      Polymer.dom.flush();
+      assert.equal(el._connectLog.length, 3);
+      assert.equal(el._connectLog[2], 'attached');
+    });
+  });
   suite('disableUpgrade and Databinding', function() {
     test('binding to disable-upgrade with true', function() {
       var el = fixture('databind-lazy');