Merge pull request #3815 from neojski/isMounted-should-return-boolean

IsMounted should always return a boolean.

src/classic/class/ReactClass.js

@@ -761,10 +761,11 @@ var ReactClassMixin = {
       }
     }
     var internalInstance = ReactInstanceMap.get(this);
-    return (
-      internalInstance &&
-      internalInstance !== ReactLifeCycle.currentlyMountingInstance
-    );
+    if (internalInstance) {
+      return internalInstance !== ReactLifeCycle.currentlyMountingInstance;
+    } else {
+      return false;
+    }
   },
 
   /**

src/core/__tests__/ReactComponentLifeCycle-test.js

@@ -287,6 +287,23 @@ describe('ReactComponentLifeCycle', function() {
     );
   });
 
+  it('isMounted should return false when unmounted', function () {
+    var Component = React.createClass({
+      render: function() {
+        return <div/>;
+      }
+    });
+
+    var container = document.createElement('div');
+    var instance = React.render(<Component />, container);
+
+    expect(instance.isMounted()).toBe(true);
+
+    React.unmountComponentAtNode(container);
+
+    expect(instance.isMounted()).toBe(false);
+  });
+
   it('warns if findDOMNode is used inside render', function() {
     spyOn(console, 'error');
     var Component = React.createClass({