Merge pull request #2751 from zpao/eslint-fixup

Eslint fixup

src/addons/ReactComponentWithPureRenderMixin.js

@@ -9,7 +9,7 @@
 * @providesModule ReactComponentWithPureRenderMixin
 */
 
-"use strict";
+'use strict';
 
 var shallowEqual = require('shallowEqual');
 

src/addons/ReactRAFBatchingStrategy.js

@@ -10,7 +10,9 @@
  * @typechecks static-only
  */
 
-"use strict";
+/* global requestAnimationFrame:true */
+
+'use strict';
 
 var ExecutionEnvironment = require('ExecutionEnvironment');
 var ReactUpdates = require('ReactUpdates');

src/addons/__tests__/update-test.js

@@ -9,7 +9,7 @@
  * @emails react-core
  */
 
-"use strict";
+'use strict';
 
 var update = require('update');
 

src/addons/link/LinkedStateMixin.js

@@ -10,7 +10,7 @@
  * @typechecks static-only
  */
 
-"use strict";
+'use strict';
 
 var ReactLink = require('ReactLink');
 var ReactStateSetters = require('ReactStateSetters');

src/addons/link/ReactLink.js

@@ -10,7 +10,7 @@
  * @typechecks static-only
  */
 
-"use strict";
+'use strict';
 
 /**
  * ReactLink encapsulates a common pattern in which a component wants to modify

src/addons/link/__tests__/LinkedStateMixin-test.js

@@ -9,7 +9,7 @@
  * @emails react-core
  */
 
-"use strict";
+'use strict';
 
 /*jshint evil:true */
 

src/addons/link/__tests__/ReactLinkPropTypes-test.js

@@ -9,7 +9,7 @@
  * @emails react-core
  */
 
-"use strict";
+'use strict';
 
 var emptyFunction = require('emptyFunction');
 var LinkPropTypes = require('ReactLink').PropTypes;
@@ -40,7 +40,7 @@ function typeCheckPass(declaration, value) {
     'testComponent',
     ReactPropTypeLocations.prop
   );
-  expect(error).toBe(undefined);
+  expect(error).toBe(null);
 }
 
 describe('ReactLink', function() {

src/addons/transitions/ReactCSSTransitionGroup.js

@@ -10,7 +10,7 @@
  * @providesModule ReactCSSTransitionGroup
  */
 
-"use strict";
+'use strict';
 
 var React = require('React');
 

src/addons/transitions/ReactCSSTransitionGroupChild.js

@@ -10,7 +10,7 @@
  * @providesModule ReactCSSTransitionGroupChild
  */
 
-"use strict";
+'use strict';
 
 var React = require('React');
 
@@ -64,7 +64,9 @@ var ReactCSSTransitionGroupChild = React.createClass({
 
       // Usually this optional callback is used for informing an owner of
       // a leave animation and telling it to remove the child.
-      finishCallback && finishCallback();
+      if (finishCallback) {
+        finishCallback();
+      }
     };
 
     ReactTransitionEvents.addEndEventListener(node, endListener);

src/addons/transitions/ReactTransitionChildMapping.js

@@ -10,7 +10,7 @@
  * @providesModule ReactTransitionChildMapping
  */
 
-"use strict";
+'use strict';
 
 var ReactChildren = require('ReactChildren');
 

src/addons/transitions/ReactTransitionEvents.js

@@ -9,7 +9,7 @@
  * @providesModule ReactTransitionEvents
  */
 
-"use strict";
+'use strict';
 
 var ExecutionEnvironment = require('ExecutionEnvironment');
 

src/addons/transitions/ReactTransitionGroup.js

@@ -9,7 +9,7 @@
  * @providesModule ReactTransitionGroup
  */
 
-"use strict";
+'use strict';
 
 var React = require('React');
 var ReactTransitionChildMapping = require('ReactTransitionChildMapping');

src/addons/transitions/__tests__/ReactCSSTransitionGroup-test.js

@@ -9,7 +9,7 @@
  * @emails react-core
  */
 
-"use strict";
+'use strict';
 
 var React;
 var ReactCSSTransitionGroup;

src/addons/transitions/__tests__/ReactTransitionChildMapping-test.js

@@ -9,7 +9,7 @@
  * @emails react-core
  */
 
-"use strict";
+'use strict';
 
 var React;
 var ReactTransitionChildMapping;

src/addons/transitions/__tests__/ReactTransitionGroup-test.js

@@ -9,7 +9,7 @@
  * @emails react-core
  */
 
-"use strict";
+'use strict';
 
 var React;
 var ReactTransitionGroup;

src/addons/update.js

@@ -9,7 +9,7 @@
  * @providesModule update
  */
 
-"use strict";
+'use strict';
 
 var assign = require('Object.assign');
 var keyOf = require('keyOf');

src/browser/ClientReactRootIndex.js

@@ -10,7 +10,7 @@
  * @typechecks
  */
 
-"use strict";
+'use strict';
 
 var nextReactRootIndex = 0;
 

src/browser/ReactBrowserEventEmitter.js

@@ -10,7 +10,7 @@
  * @typechecks static-only
  */
 
-"use strict";
+'use strict';
 
 var EventConstants = require('EventConstants');
 var EventPluginHub = require('EventPluginHub');
@@ -126,7 +126,7 @@ var topEventMapping = {
 /**
  * To ensure no conflicts with other potential React instances on the page
  */
-var topListenersIDKey = "_reactListenersID" + String(Math.random()).slice(2);
+var topListenersIDKey = '_reactListenersID' + String(Math.random()).slice(2);
 
 function getListeningForDocument(mountAt) {
   // In IE8, `mountAt` is a host object and doesn't have `hasOwnProperty`

src/browser/ReactDOM.js

@@ -10,7 +10,7 @@
  * @typechecks static-only
  */
 
-"use strict";
+'use strict';
 
 var ReactElement = require('ReactElement');
 var ReactElementValidator = require('ReactElementValidator');

src/browser/ReactPutListenerQueue.js

@@ -9,7 +9,7 @@
  * @providesModule ReactPutListenerQueue
  */
 
-"use strict";
+'use strict';
 
 var PooledClass = require('PooledClass');
 var ReactBrowserEventEmitter = require('ReactBrowserEventEmitter');

src/browser/ReactReconcileTransaction.js

@@ -10,7 +10,7 @@
  * @typechecks static-only
  */
 
-"use strict";
+'use strict';
 
 var CallbackQueue = require('CallbackQueue');
 var PooledClass = require('PooledClass');

src/browser/ReactWithAddons.js

@@ -16,7 +16,7 @@
  * place to live inside React core.
  */
 
-"use strict";
+'use strict';
 
 var LinkedStateMixin = require('LinkedStateMixin');
 var React = require('React');

src/browser/__tests__/ReactBrowserEventEmitter-test.js

@@ -9,7 +9,7 @@
  * @emails react-core
  */
 
-"use strict";
+'use strict';
 
 require('mock-modules')
     .dontMock('EventPluginHub')

src/browser/__tests__/ReactDOM-test.js

@@ -11,7 +11,7 @@
 
 /*jslint evil: true */
 
-"use strict";
+'use strict';
 
 var React = require('React');
 var ReactTestUtils = require('ReactTestUtils');

src/browser/__tests__/ReactWebWorker-test.js

@@ -11,7 +11,7 @@
 
 /*jslint evil: true */
 
-"use strict";
+'use strict';
 
 describe('ReactWebWorker', function() {
   ;(typeof Worker == 'undefined' ? xit : it)('can run React in a web worker', function() {

src/browser/__tests__/findDOMNode-test.js

@@ -9,7 +9,7 @@
  * @emails react-core
  */
 
-"use strict";
+'use strict';
 
 var React = require('React');
 var ReactTestUtils = require('ReactTestUtils');
@@ -48,4 +48,3 @@ describe('findDOMNode', function() {
   });
 
 });
-

src/browser/eventPlugins/AnalyticsEventPluginFactory.js

@@ -74,7 +74,7 @@
  * usage.
  */
 
-"use strict";
+'use strict';
 
 var ExecutionEnvironment = require('ExecutionEnvironment');
 
@@ -152,7 +152,7 @@ function extractEvents(
     return null;
   }
 
-  var analyticsEventsArr = analyticsEventsStr.split(",");
+  var analyticsEventsArr = analyticsEventsStr.split(',');
   if (!analyticsData.hasOwnProperty(analyticsID)) {
     initAnalyticsDataForID(analyticsID, analyticsEventsArr);
   }

src/browser/eventPlugins/BeforeInputEventPlugin.js

@@ -10,7 +10,7 @@
  * @typechecks static-only
  */
 
-"use strict";
+'use strict';
 
 var EventConstants = require('EventConstants');
 var EventPropagators = require('EventPropagators');
@@ -256,7 +256,7 @@ function extractCompositionEvent(
   }
 
   if (!eventType) {
-    return;
+    return null;
   }
 
   if (useFallbackCompositionData) {
@@ -427,7 +427,7 @@ function extractBeforeInputEvent(
   // If no characters are being inserted, no BeforeInput event should
   // be fired.
   if (!chars) {
-    return;
+    return null;
   }
 
   var event = SyntheticInputEvent.getPooled(
@@ -490,7 +490,7 @@ var BeforeInputEventPlugin = {
         topLevelTarget,
         topLevelTargetID,
         nativeEvent
-      ),
+      )
     ];
   }
 };

src/browser/eventPlugins/ChangeEventPlugin.js

@@ -9,7 +9,7 @@
  * @providesModule ChangeEventPlugin
  */
 
-"use strict";
+'use strict';
 
 var EventConstants = require('EventConstants');
 var EventPluginHub = require('EventPluginHub');

src/browser/eventPlugins/DefaultEventPluginOrder.js

@@ -9,7 +9,7 @@
  * @providesModule DefaultEventPluginOrder
  */
 
-"use strict";
+'use strict';
 
  var keyOf = require('keyOf');
 

src/browser/eventPlugins/EnterLeaveEventPlugin.js

@@ -10,7 +10,7 @@
  * @typechecks static-only
  */
 
-"use strict";
+'use strict';
 
 var EventConstants = require('EventConstants');
 var EventPropagators = require('EventPropagators');

src/browser/eventPlugins/FallbackCompositionState.js

@@ -10,7 +10,7 @@
  * @typechecks static-only
  */
 
-"use strict";
+'use strict';
 
 var PooledClass = require('PooledClass');
 
@@ -58,19 +58,21 @@ assign(FallbackCompositionState.prototype, {
       return this._fallbackText;
     }
 
-    var endValue = this.getText();
+    var start;
     var startValue = this._startText;
     var startLength = startValue.length;
+    var end;
+    var endValue = this.getText();
     var endLength = endValue.length;
 
-    for (var start = 0; start < startLength; start++) {
+    for (start = 0; start < startLength; start++) {
       if (startValue[start] !== endValue[start]) {
         break;
       }
     }
 
     var minEnd = startLength - start;
-    for (var end = 1; end <= minEnd; end++) {
+    for (end = 1; end <= minEnd; end++) {
       if (startValue[startLength - end] !== endValue[endLength - end]) {
         break;
       }

src/browser/eventPlugins/MobileSafariClickEventPlugin.js

@@ -10,7 +10,7 @@
  * @typechecks static-only
  */
 
-"use strict";
+'use strict';
 
 var EventConstants = require('EventConstants');
 

src/browser/eventPlugins/ResponderEventPlugin.js

@@ -9,7 +9,7 @@
  * @providesModule ResponderEventPlugin
  */
 
-"use strict";
+'use strict';
 
 var EventConstants = require('EventConstants');
 var EventPluginUtils = require('EventPluginUtils');

src/browser/eventPlugins/SelectEventPlugin.js

@@ -9,7 +9,7 @@
  * @providesModule SelectEventPlugin
  */
 
-"use strict";
+'use strict';
 
 var EventConstants = require('EventConstants');
 var EventPropagators = require('EventPropagators');
@@ -94,8 +94,8 @@ function constructSelectEvent(nativeEvent) {
   // won't dispatch.
   if (mouseDown ||
       activeElement == null ||
-      activeElement != getActiveElement()) {
-    return;
+      activeElement !== getActiveElement()) {
+    return null;
   }
 
   // Only fire when selection has actually changed.

src/browser/eventPlugins/SimpleEventPlugin.js

@@ -9,7 +9,7 @@
  * @providesModule SimpleEventPlugin
  */
 
-"use strict";
+'use strict';
 
 var EventConstants = require('EventConstants');
 var EventPluginUtils = require('EventPluginUtils');

src/browser/eventPlugins/TapEventPlugin.js

@@ -10,7 +10,7 @@
  * @typechecks static-only
  */
 
-"use strict";
+'use strict';
 
 var EventConstants = require('EventConstants');
 var EventPluginUtils = require('EventPluginUtils');
@@ -117,7 +117,7 @@ var TapEventPlugin = {
       usedTouchTime = Date.now();
     } else {
       if (usedTouch && (Date.now() - usedTouchTime < TOUCH_DELAY)) {
-        return;
+        return null;
       }
     }
     var event = null;

src/browser/eventPlugins/__tests__/AnalyticsEventPlugin-test.js

@@ -9,7 +9,7 @@
  * @emails react-core
  */
 
-"use strict";
+'use strict';
 
 var mocks = require('mocks');
 

src/browser/eventPlugins/__tests__/EnterLeaveEventPlugin-test.js

@@ -11,7 +11,7 @@
 
 /*jslint evil: true */
 
-"use strict";
+'use strict';
 
 var EnterLeaveEventPlugin;
 var EventConstants;

src/browser/eventPlugins/__tests__/FallbackCompositionState-test.js

@@ -9,7 +9,7 @@
  * @emails react-core
  */
 
-"use strict";
+'use strict';
 
 describe('FallbackCompositionState', function() {
   var FallbackCompositionState;

src/browser/eventPlugins/__tests__/ResponderEventPlugin-test.js

@@ -9,7 +9,7 @@
  * @emails react-core
  */
 
-"use strict";
+'use strict';
 
 var EventPluginHub;
 var EventConstants;

src/browser/findDOMNode.js

@@ -10,7 +10,7 @@
  * @typechecks static-only
  */
 
-"use strict";
+'use strict';
 var ReactInstanceMap = require('ReactInstanceMap');
 var ReactMount = require('ReactMount');
 
@@ -35,9 +35,9 @@ function findDOMNode(componentOrElement) {
   }
   invariant(
     componentOrElement.render == null ||
-    typeof(componentOrElement.render) !== 'function',
-    'Component (with keys: %s) contains `render` method '
-    +'but is not mounted in the DOM',
+    typeof componentOrElement.render !== 'function',
+    'Component (with keys: %s) contains `render` method ' +
+    'but is not mounted in the DOM',
     Object.keys(componentOrElement)
   );
   invariant(

src/browser/server/ReactMarkupChecksum.js

@@ -9,7 +9,7 @@
  * @providesModule ReactMarkupChecksum
  */
 
-"use strict";
+'use strict';
 
 var adler32 = require('adler32');
 

src/browser/server/ReactServerRendering.js

@@ -9,7 +9,7 @@
  * @typechecks static-only
  * @providesModule ReactServerRendering
  */
-"use strict";
+'use strict';
 
 var ReactElement = require('ReactElement');
 var ReactInstanceHandles = require('ReactInstanceHandles');
@@ -38,7 +38,8 @@ function renderToString(element) {
 
     return transaction.perform(function() {
       var componentInstance = instantiateReactComponent(element, null);
-      var markup = componentInstance.mountComponent(id, transaction, emptyObject);
+      var markup =
+        componentInstance.mountComponent(id, transaction, emptyObject);
       return ReactMarkupChecksum.addChecksumToMarkup(markup);
     }, null);
   } finally {

src/browser/server/ReactServerRenderingTransaction.js

@@ -10,7 +10,7 @@
  * @typechecks
  */
 
-"use strict";
+'use strict';
 
 var PooledClass = require('PooledClass');
 var CallbackQueue = require('CallbackQueue');

src/browser/server/ServerReactRootIndex.js

@@ -10,7 +10,7 @@
  * @typechecks
  */
 
-"use strict";
+'use strict';
 
 /**
  * Size of the reactRoot ID space. We generate random numbers for React root

src/browser/server/__tests__/ReactServerRendering-test.js

@@ -11,7 +11,7 @@
 
 /*jslint evil: true */
 
-"use strict";
+'use strict';
 
 require('mock-modules')
   .dontMock('ExecutionEnvironment')

src/browser/syntheticEvents/SyntheticClipboardEvent.js

@@ -10,7 +10,7 @@
  * @typechecks static-only
  */
 
-"use strict";
+'use strict';
 
 var SyntheticEvent = require('SyntheticEvent');
 
@@ -41,4 +41,3 @@ function SyntheticClipboardEvent(dispatchConfig, dispatchMarker, nativeEvent) {
 SyntheticEvent.augmentClass(SyntheticClipboardEvent, ClipboardEventInterface);
 
 module.exports = SyntheticClipboardEvent;
-

src/browser/syntheticEvents/SyntheticCompositionEvent.js

@@ -10,7 +10,7 @@
  * @typechecks static-only
  */
 
-"use strict";
+'use strict';
 
 var SyntheticEvent = require('SyntheticEvent');
 
@@ -41,4 +41,3 @@ SyntheticEvent.augmentClass(
 );
 
 module.exports = SyntheticCompositionEvent;
-

src/browser/syntheticEvents/SyntheticDragEvent.js

@@ -10,7 +10,7 @@
  * @typechecks static-only
  */
 
-"use strict";
+'use strict';
 
 var SyntheticMouseEvent = require('SyntheticMouseEvent');
 

src/browser/syntheticEvents/SyntheticEvent.js

@@ -10,7 +10,7 @@
  * @typechecks static-only
  */
 
-"use strict";
+'use strict';
 
 var PooledClass = require('PooledClass');
 
@@ -88,13 +88,21 @@ assign(SyntheticEvent.prototype, {
   preventDefault: function() {
     this.defaultPrevented = true;
     var event = this.nativeEvent;
-    event.preventDefault ? event.preventDefault() : event.returnValue = false;
+    if (event.preventDefault) {
+      event.preventDefault();
+    } else {
+      event.returnValue = false;
+    }
     this.isDefaultPrevented = emptyFunction.thatReturnsTrue;
   },
 
   stopPropagation: function() {
     var event = this.nativeEvent;
-    event.stopPropagation ? event.stopPropagation() : event.cancelBubble = true;
+    if (event.stopPropagation) {
+      event.stopPropagation();
+    } else {
+      event.cancelBubble = true;
+    }
     this.isPropagationStopped = emptyFunction.thatReturnsTrue;
   },
 

src/browser/syntheticEvents/SyntheticFocusEvent.js

@@ -10,7 +10,7 @@
  * @typechecks static-only
  */
 
-"use strict";
+'use strict';
 
 var SyntheticUIEvent = require('SyntheticUIEvent');
 

src/browser/syntheticEvents/SyntheticInputEvent.js

@@ -10,7 +10,7 @@
  * @typechecks static-only
  */
 
-"use strict";
+'use strict';
 
 var SyntheticEvent = require('SyntheticEvent');
 
@@ -42,4 +42,3 @@ SyntheticEvent.augmentClass(
 );
 
 module.exports = SyntheticInputEvent;
-

src/browser/syntheticEvents/SyntheticKeyboardEvent.js

@@ -10,7 +10,7 @@
  * @typechecks static-only
  */
 
-"use strict";
+'use strict';
 
 var SyntheticUIEvent = require('SyntheticUIEvent');
 

src/browser/syntheticEvents/SyntheticMouseEvent.js

@@ -10,7 +10,7 @@
  * @typechecks static-only
  */
 
-"use strict";
+'use strict';
 
 var SyntheticUIEvent = require('SyntheticUIEvent');
 var ViewportMetrics = require('ViewportMetrics');

src/browser/syntheticEvents/SyntheticTouchEvent.js

@@ -10,7 +10,7 @@
  * @typechecks static-only
  */
 
-"use strict";
+'use strict';
 
 var SyntheticUIEvent = require('SyntheticUIEvent');
 

src/browser/syntheticEvents/SyntheticUIEvent.js

@@ -10,7 +10,7 @@
  * @typechecks static-only
  */
 
-"use strict";
+'use strict';
 
 var SyntheticEvent = require('SyntheticEvent');
 

src/browser/syntheticEvents/SyntheticWheelEvent.js

@@ -10,7 +10,7 @@
  * @typechecks static-only
  */
 
-"use strict";
+'use strict';
 
 var SyntheticMouseEvent = require('SyntheticMouseEvent');
 

src/browser/syntheticEvents/__tests__/SyntheticEvent-test.js

@@ -9,7 +9,7 @@
  * @emails react-core
  */
 
-"use strict";
+'use strict';
 
 var SyntheticEvent;
 

src/browser/syntheticEvents/__tests__/SyntheticWheelEvent-test.js

@@ -9,7 +9,7 @@
  * @emails react-core
  */
 
-"use strict";
+'use strict';
 
 var SyntheticWheelEvent;
 

src/browser/ui/React.js

@@ -11,7 +11,7 @@
 
 /* globals __REACT_DEVTOOLS_GLOBAL_HOOK__*/
 
-"use strict";
+'use strict';
 
 var DOMPropertyOperations = require('DOMPropertyOperations');
 var EventPluginUtils = require('EventPluginUtils');

src/browser/ui/ReactBrowserComponentMixin.js

@@ -9,7 +9,7 @@
  * @providesModule ReactBrowserComponentMixin
  */
 
-"use strict";
+'use strict';
 
 var findDOMNode = require('findDOMNode');
 

src/browser/ui/ReactComponentBrowserEnvironment.js

@@ -11,7 +11,7 @@
 
 /*jslint evil: true */
 
-"use strict";
+'use strict';
 
 var ReactDOMIDOperations = require('ReactDOMIDOperations');
 var ReactMount = require('ReactMount');

src/browser/ui/ReactDOMComponent.js

@@ -10,7 +10,9 @@
  * @typechecks static-only
  */
 
-"use strict";
+/* global hasOwnProperty:true */
+
+'use strict';
 
 var CSSPropertyOperations = require('CSSPropertyOperations');
 var DOMProperty = require('DOMProperty');
@@ -73,9 +75,9 @@ function assertValidProps(props) {
     if (props.contentEditable && props.children != null) {
       console.warn(
         'A component is `contentEditable` and contains `children` managed by ' +
-        'React. It is now your responsibility to guarantee that none of those '+
-        'nodes are unexpectedly modified or duplicated. This is probably not ' +
-        'intentional.'
+        'React. It is now your responsibility to guarantee that none of ' +
+        'those nodes are unexpectedly modified or duplicated. This is ' +
+        'probably not intentional.'
       );
     }
   }

src/browser/ui/ReactDOMIDOperations.js

@@ -12,7 +12,7 @@
 
 /*jslint evil: true */
 
-"use strict";
+'use strict';
 
 var CSSPropertyOperations = require('CSSPropertyOperations');
 var DOMChildrenOperations = require('DOMChildrenOperations');

src/browser/ui/ReactDOMSelection.js

@@ -9,7 +9,7 @@
  * @providesModule ReactDOMSelection
  */
 
-"use strict";
+'use strict';
 
 var ExecutionEnvironment = require('ExecutionEnvironment');
 

src/browser/ui/ReactDOMTextComponent.js

@@ -10,7 +10,7 @@
  * @typechecks static-only
  */
 
-"use strict";
+'use strict';
 
 var DOMPropertyOperations = require('DOMPropertyOperations');
 var ReactComponentBrowserEnvironment =

src/browser/ui/ReactDefaultInjection.js

@@ -9,7 +9,7 @@
  * @providesModule ReactDefaultInjection
  */
 
-"use strict";
+'use strict';
 
 var BeforeInputEventPlugin = require('BeforeInputEventPlugin');
 var ChangeEventPlugin = require('ChangeEventPlugin');

src/browser/ui/ReactEventListener.js

@@ -10,7 +10,7 @@
  * @typechecks static-only
  */
 
-"use strict";
+'use strict';
 
 var EventListener = require('EventListener');
 var ExecutionEnvironment = require('ExecutionEnvironment');
@@ -123,7 +123,7 @@ var ReactEventListener = {
   trapBubbledEvent: function(topLevelType, handlerBaseName, handle) {
     var element = handle;
     if (!element) {
-      return;
+      return null;
     }
     return EventListener.listen(
       element,
@@ -145,7 +145,7 @@ var ReactEventListener = {
   trapCapturedEvent: function(topLevelType, handlerBaseName, handle) {
     var element = handle;
     if (!element) {
-      return;
+      return null;
     }
     return EventListener.capture(
       element,

src/browser/ui/ReactInjection.js

@@ -9,7 +9,7 @@
  * @providesModule ReactInjection
  */
 
-"use strict";
+'use strict';
 
 var DOMProperty = require('DOMProperty');
 var EventPluginHub = require('EventPluginHub');

src/browser/ui/ReactInputSelection.js

@@ -9,7 +9,7 @@
  * @providesModule ReactInputSelection
  */
 
-"use strict";
+'use strict';
 
 var ReactDOMSelection = require('ReactDOMSelection');
 

src/browser/ui/ReactMount.js

@@ -9,7 +9,7 @@
  * @providesModule ReactMount
  */
 
-"use strict";
+'use strict';
 
 var DOMProperty = require('DOMProperty');
 var ReactBrowserEventEmitter = require('ReactBrowserEventEmitter');
@@ -435,7 +435,9 @@ var ReactMount = {
       container,
       shouldReuseMarkup
     ).getPublicInstance();
-    callback && callback.call(component);
+    if (callback) {
+      callback.call(component);
+    }
     return component;
   },
 

src/browser/ui/__tests__/ReactDOMComponent-test.js

@@ -11,7 +11,7 @@
 
 /*jslint evil: true */
 
-"use strict";
+'use strict';
 
 var assign = require('Object.assign');
 var mocks = require('mocks');

src/browser/ui/__tests__/ReactDOMIDOperations-test.js

@@ -11,7 +11,7 @@
 
 /*jslint evil: true */
 
-"use strict";
+'use strict';
 
 describe('ReactDOMIDOperations', function() {
   var DOMPropertyOperations = require('DOMPropertyOperations');

src/browser/ui/__tests__/ReactDOMTextComponent-test.js

@@ -9,7 +9,7 @@
  * @emails react-core
  */
 
-"use strict";
+'use strict';
 
 var React;
 

src/browser/ui/__tests__/ReactMount-test.js

@@ -9,7 +9,7 @@
  * @emails react-core
  */
 
-"use strict";
+'use strict';
 
 var mocks = require('mocks');
 

src/browser/ui/__tests__/ReactMountDestruction-test.js

@@ -9,7 +9,7 @@
  * @emails react-core
  */
 
-"use strict";
+'use strict';
 
 var React = require('React');
 

src/browser/ui/__tests__/ReactRenderDocument-test.js

@@ -11,7 +11,7 @@
 
 /*jslint evil: true */
 
-"use strict";
+'use strict';
 
 var React;
 var ReactInstanceMap;

src/browser/ui/dom/CSSProperty.js

@@ -9,7 +9,7 @@
  * @providesModule CSSProperty
  */
 
-"use strict";
+'use strict';
 
 /**
  * CSS properties which accept numbers but are not in units of "px".

src/browser/ui/dom/CSSPropertyOperations.js

@@ -10,7 +10,7 @@
  * @typechecks static-only
  */
 
-"use strict";
+'use strict';
 
 var CSSProperty = require('CSSProperty');
 var ExecutionEnvironment = require('ExecutionEnvironment');

src/browser/ui/dom/DOMChildrenOperations.js

@@ -10,7 +10,7 @@
  * @typechecks static-only
  */
 
-"use strict";
+'use strict';
 
 var Danger = require('Danger');
 var ReactMultiChildUpdateTypes = require('ReactMultiChildUpdateTypes');
@@ -100,7 +100,8 @@ var DOMChildrenOperations = {
     // List of children that will be moved or removed.
     var updatedChildren = null;
 
-    for (var i = 0; update = updates[i]; i++) {
+    for (var i = 0; i < updates.length; i++) {
+      update = updates[i];
       if (update.type === ReactMultiChildUpdateTypes.MOVE_EXISTING ||
           update.type === ReactMultiChildUpdateTypes.REMOVE_NODE) {
         var updatedIndex = update.fromIndex;
@@ -112,7 +113,7 @@ var DOMChildrenOperations = {
           'processUpdates(): Unable to find child %s of element. This ' +
           'probably means the DOM was unexpectedly mutated (e.g., by the ' +
           'browser), usually due to forgetting a <tbody> when using tables, ' +
-          'nesting tags like <form>, <p>, or <a>, or using non-SVG elements '+
+          'nesting tags like <form>, <p>, or <a>, or using non-SVG elements ' +
           'in an <svg> parent. Try inspecting the child nodes of the element ' +
           'with React ID `%s`.',
           updatedIndex,
@@ -137,7 +138,8 @@ var DOMChildrenOperations = {
       }
     }
 
-    for (var k = 0; update = updates[k]; k++) {
+    for (var k = 0; k < updates.length; k++) {
+      update = updates[k];
       switch (update.type) {
         case ReactMultiChildUpdateTypes.INSERT_MARKUP:
           insertChildAt(

src/browser/ui/dom/DOMProperty.js

@@ -12,7 +12,7 @@
 
 /*jslint bitwise: true */
 
-"use strict";
+'use strict';
 
 var invariant = require('invariant');
 

src/browser/ui/dom/DOMPropertyOperations.js

@@ -10,7 +10,7 @@
  * @typechecks static-only
  */
 
-"use strict";
+'use strict';
 
 var DOMProperty = require('DOMProperty');
 

src/browser/ui/dom/Danger.js

@@ -12,7 +12,7 @@
 
 /*jslint evil: true, sub: true */
 
-"use strict";
+'use strict';
 
 var ExecutionEnvironment = require('ExecutionEnvironment');
 
@@ -82,7 +82,8 @@ var Danger = {
       // This for-in loop skips the holes of the sparse array. The order of
       // iteration should follow the order of assignment, which happens to match
       // numerical index order, but we don't rely on that.
-      for (var resultIndex in markupListByNodeName) {
+      var resultIndex;
+      for (resultIndex in markupListByNodeName) {
         if (markupListByNodeName.hasOwnProperty(resultIndex)) {
           var markup = markupListByNodeName[resultIndex];
 
@@ -103,8 +104,8 @@ var Danger = {
         emptyFunction // Do nothing special with <script> tags.
       );
 
-      for (i = 0; i < renderNodes.length; ++i) {
-        var renderNode = renderNodes[i];
+      for (var j = 0; j < renderNodes.length; ++j) {
+        var renderNode = renderNodes[j];
         if (renderNode.hasAttribute &&
             renderNode.hasAttribute(RESULT_INDEX_ATTR)) {
 
@@ -124,7 +125,7 @@ var Danger = {
 
         } else if (__DEV__) {
           console.error(
-            "Danger: Discarding unexpected node:",
+            'Danger: Discarding unexpected node:',
             renderNode
           );
         }

src/browser/ui/dom/HTMLDOMPropertyConfig.js

@@ -11,7 +11,7 @@
 
 /*jslint bitwise: true*/
 
-"use strict";
+'use strict';
 
 var DOMProperty = require('DOMProperty');
 var ExecutionEnvironment = require('ExecutionEnvironment');
@@ -160,12 +160,17 @@ var HTMLDOMPropertyConfig = {
     /**
      * Non-standard Properties
      */
-    autoCapitalize: null, // Supported in Mobile Safari for keyboard hints
-    autoCorrect: null, // Supported in Mobile Safari for keyboard hints
-    itemProp: MUST_USE_ATTRIBUTE, // Microdata: http://schema.org/docs/gs.html
-    itemScope: MUST_USE_ATTRIBUTE | HAS_BOOLEAN_VALUE, // Microdata: http://schema.org/docs/gs.html
-    itemType: MUST_USE_ATTRIBUTE, // Microdata: http://schema.org/docs/gs.html
-    property: null // Supports OG in meta tags
+    // autoCapitalize and autoCorrect are supported in Mobile Safari for
+    // keyboard hints.
+    autoCapitalize: null,
+    autoCorrect: null,
+    // itemProp, itemScope, itemType are for Microdata support. See
+    // http://schema.org/docs/gs.html
+    itemProp: MUST_USE_ATTRIBUTE,
+    itemScope: MUST_USE_ATTRIBUTE | HAS_BOOLEAN_VALUE,
+    itemType: MUST_USE_ATTRIBUTE,
+    // property is supported for OpenGraph in meta tags.
+    property: null
   },
   DOMAttributeNames: {
     acceptCharset: 'accept-charset',

src/browser/ui/dom/SVGDOMPropertyConfig.js

@@ -11,7 +11,7 @@
 
 /*jslint bitwise: true*/
 
-"use strict";
+'use strict';
 
 var DOMProperty = require('DOMProperty');
 

src/browser/ui/dom/ViewportMetrics.js

@@ -9,7 +9,7 @@
  * @providesModule ViewportMetrics
  */
 
-"use strict";
+'use strict';
 
 var ViewportMetrics = {
 

src/browser/ui/dom/__tests__/CSSProperty-test.js

@@ -11,7 +11,7 @@
 
 /*jslint evil: true */
 
-"use strict";
+'use strict';
 
 describe('CSSProperty', function() {
   var CSSProperty;

src/browser/ui/dom/__tests__/CSSPropertyOperations-test.js

@@ -11,7 +11,7 @@
 
 /*jslint evil: true */
 
-"use strict";
+'use strict';
 
 var React = require('React');
 

src/browser/ui/dom/__tests__/DOMPropertyOperations-test.js

@@ -9,7 +9,7 @@
  * @emails react-core
  */
 
-"use strict";
+'use strict';
 
 describe('DOMPropertyOperations', function() {
   var DOMPropertyOperations;

src/browser/ui/dom/__tests__/Danger-test.js

@@ -10,7 +10,7 @@
  */
 /*jslint evil: true */
 
-"use strict";
+'use strict';
 
 var React = require('React');
 var instantiateReactComponent = require('instantiateReactComponent');

src/browser/ui/dom/__tests__/getNodeForCharacterOffset-test.js

@@ -11,7 +11,7 @@
 
 /*jslint evil: true */
 
-"use strict";
+'use strict';
 
 var getTestDocument = require('getTestDocument');
 

src/browser/ui/dom/components/AutoFocusMixin.js

@@ -10,7 +10,7 @@
  * @typechecks static-only
  */
 
-"use strict";
+'use strict';
 
 var focusNode = require('focusNode');
 

src/browser/ui/dom/components/LinkedValueUtils.js

@@ -10,7 +10,7 @@
  * @typechecks static-only
  */
 
-"use strict";
+'use strict';
 
 var ReactPropTypes = require('ReactPropTypes');
 
@@ -81,7 +81,7 @@ var LinkedValueUtils = {
             props.onChange ||
             props.readOnly ||
             props.disabled) {
-          return;
+          return null;
         }
         return new Error(
           'You provided a `value` prop to a form field without an ' +
@@ -95,7 +95,7 @@ var LinkedValueUtils = {
             props.onChange ||
             props.readOnly ||
             props.disabled) {
-          return;
+          return null;
         }
         return new Error(
           'You provided a `checked` prop to a form field without an ' +

src/browser/ui/dom/components/LocalEventTrapMixin.js

@@ -9,7 +9,7 @@
  * @providesModule LocalEventTrapMixin
  */
 
-"use strict";
+'use strict';
 
 var ReactBrowserEventEmitter = require('ReactBrowserEventEmitter');
 

src/browser/ui/dom/components/ReactDOMButton.js

@@ -9,7 +9,7 @@
  * @providesModule ReactDOMButton
  */
 
-"use strict";
+'use strict';
 
 var AutoFocusMixin = require('AutoFocusMixin');
 var ReactBrowserComponentMixin = require('ReactBrowserComponentMixin');

src/browser/ui/dom/components/ReactDOMForm.js

@@ -9,7 +9,7 @@
  * @providesModule ReactDOMForm
  */
 
-"use strict";
+'use strict';
 
 var EventConstants = require('EventConstants');
 var LocalEventTrapMixin = require('LocalEventTrapMixin');

src/browser/ui/dom/components/ReactDOMImg.js

@@ -9,7 +9,7 @@
  * @providesModule ReactDOMImg
  */
 
-"use strict";
+'use strict';
 
 var EventConstants = require('EventConstants');
 var LocalEventTrapMixin = require('LocalEventTrapMixin');

src/browser/ui/dom/components/ReactDOMInput.js

@@ -9,7 +9,7 @@
  * @providesModule ReactDOMInput
  */
 
-"use strict";
+'use strict';
 
 var AutoFocusMixin = require('AutoFocusMixin');
 var DOMPropertyOperations = require('DOMPropertyOperations');

src/browser/ui/dom/components/ReactDOMOption.js

@@ -9,7 +9,7 @@
  * @providesModule ReactDOMOption
  */
 
-"use strict";
+'use strict';
 
 var ReactBrowserComponentMixin = require('ReactBrowserComponentMixin');
 var ReactClass = require('ReactClass');