Remove DOM nesting warning for whitespace

  * Identify if content of ReactDOMTextComponent is only whitespace
  * Fixed misc. lint errors

src/renderers/dom/client/validateDOMNesting.js

@@ -327,8 +327,8 @@ if (__DEV__) {
     var parentInfo = ancestorInfo.current;
     var parentTag = parentInfo && parentInfo.tag;
 
-    var invalidParent =
-      isTagValidWithParent(childTag, parentTag) ? null : parentInfo;
+    var invalidParent = childInstance._isWhitespace ?
+      null : isTagValidWithParent(childTag, parentTag) ? null : parentInfo;
     var invalidAncestor =
       invalidParent ? null : findInvalidAncestorForTag(childTag, ancestorInfo);
     var problematic = invalidParent || invalidAncestor;

src/renderers/dom/shared/ReactDOMTextComponent.js

@@ -48,6 +48,11 @@ var ReactDOMTextComponent = function(text) {
   this._mountIndex = 0;
   this._closingComment = null;
   this._commentNodes = null;
+
+  if (__DEV__) {
+    // validateDOMNesting uses this:
+    this._isWhitespace = typeof text === 'string' && text.trim().length === 0;
+  }
 };
 
 Object.assign(ReactDOMTextComponent.prototype, {

src/renderers/dom/shared/__tests__/ReactDOMComponent-test.js

@@ -724,7 +724,7 @@ describe('ReactDOMComponent', function() {
     });
 
     it('should work error event on <source> element', function() {
-      spyOn(console, 'error');  
+      spyOn(console, 'error');
       var container = document.createElement('div');
       ReactDOM.render(
         <video>
@@ -1215,21 +1215,29 @@ describe('ReactDOMComponent', function() {
       });
       var Foo = React.createClass({
         render: function() {
-          return <table><Row /> </table>;
+          return <table><Row /></table>;
         },
       });
       ReactTestUtils.renderIntoDocument(<Foo />);
 
-      expect(console.error.calls.count()).toBe(2);
+      expect(console.error.calls.count()).toBe(1);
       expect(console.error.calls.argsFor(0)[0]).toBe(
         'Warning: validateDOMNesting(...): <tr> cannot appear as a child of ' +
         '<table>. See Foo > table > Row > tr. Add a <tbody> to your code to ' +
         'match the DOM tree generated by the browser.'
       );
-      expect(console.error.calls.argsFor(1)[0]).toBe(
-        'Warning: validateDOMNesting(...): #text cannot appear as a child ' +
-        'of <table>. See Foo > table > #text.'
-      );
+    });
+
+    it('does not warn for arbitrary table whitespace', () => {
+      spyOn(console, 'error');
+      var Foo = React.createClass({
+        render: function() {
+          return <table><tbody><tr /> </tbody></table>;
+        },
+      });
+      ReactTestUtils.renderIntoDocument(<Foo />);
+
+      expect(console.error.calls.count()).toBe(0);
     });
 
     it('gives useful context in warnings', () => {

src/renderers/native/NativeMethodsMixin.js

@@ -217,6 +217,6 @@ function mountSafeCallback(
     }
     return callback.apply(context, arguments);
   };
-};
+}
 
 module.exports = NativeMethodsMixin;

src/renderers/shared/__tests__/ReactPerf-test.js

@@ -467,5 +467,5 @@ describe('ReactPerf', function() {
     expect(console.error.calls.count()).toBe(1);
 
     __DEV__ = true;
-  })
+  });
 });