From a5a1ebe34143df96b92c5db166110edac3abe863 Mon Sep 17 00:00:00 2001 From: Chad Burggraf Date: Fri, 2 Dec 2016 16:13:47 -0700 Subject: [PATCH] Remove all Google Closure Compiler annotations. See #7551. --- src/isomorphic/classic/class/ReactClass.js | 6 ----- .../classic/element/ReactDOMFactories.js | 4 --- .../classic/element/ReactElement.js | 1 - src/isomorphic/modern/class/ReactComponent.js | 4 --- .../modern/class/ReactNoopUpdateQueue.js | 2 -- src/renderers/art/ReactART.js | 4 --- .../dom/shared/escapeTextContentForBrowser.js | 2 -- .../SyntheticAnimationEvent.js | 1 - .../SyntheticClipboardEvent.js | 1 - .../SyntheticCompositionEvent.js | 1 - .../syntheticEvents/SyntheticDragEvent.js | 1 - .../syntheticEvents/SyntheticFocusEvent.js | 1 - .../syntheticEvents/SyntheticInputEvent.js | 1 - .../syntheticEvents/SyntheticKeyboardEvent.js | 1 - .../syntheticEvents/SyntheticMouseEvent.js | 1 - .../syntheticEvents/SyntheticTouchEvent.js | 1 - .../SyntheticTransitionEvent.js | 1 - .../syntheticEvents/SyntheticUIEvent.js | 1 - .../syntheticEvents/SyntheticWheelEvent.js | 1 - .../dom/stack/client/ReactDOMComponent.js | 3 --- src/renderers/dom/stack/client/ReactMount.js | 1 - .../stack/client/ReactReconcileTransaction.js | 2 -- .../stack/client/wrappers/ReactDOMSelect.js | 2 -- .../server/ReactServerRenderingTransaction.js | 2 -- .../stack/server/ReactServerUpdateQueue.js | 2 -- src/renderers/native/ReactNativeMount.js | 1 - .../native/ReactNativeReconcileTransaction.js | 2 -- .../native/createReactNativeComponentClass.js | 1 - .../shared/shared/event/EventPluginHub.js | 4 --- .../shared/event/EventPluginRegistry.js | 5 ---- .../eventPlugins/ResponderSyntheticEvent.js | 1 - .../shared/shouldUpdateReactComponent.js | 1 - .../reconciler/ReactCompositeComponent.js | 25 ------------------- .../stack/reconciler/ReactMultiChild.js | 14 ----------- .../shared/stack/reconciler/ReactOwner.js | 3 --- .../stack/reconciler/ReactReconciler.js | 2 -- .../stack/reconciler/ReactUpdateQueue.js | 2 -- .../shared/stack/reconciler/Transaction.js | 1 - .../reconciler/instantiateReactComponent.js | 1 - .../testing/ReactTestReconcileTransaction.js | 2 -- src/shared/types/checkReactTypeSpec.js | 1 - src/test/ReactTestUtils.js | 1 - 42 files changed, 114 deletions(-) diff --git a/src/isomorphic/classic/class/ReactClass.js b/src/isomorphic/classic/class/ReactClass.js index 475f08d7d81b..7d974e2bfba2 100644 --- a/src/isomorphic/classic/class/ReactClass.js +++ b/src/isomorphic/classic/class/ReactClass.js @@ -169,7 +169,6 @@ var ReactClassInterface: {[key: string]: SpecPolicy} = { * } * * @return {ReactComponent} - * @nosideeffects * @required */ render: 'DEFINE_ONCE', @@ -621,7 +620,6 @@ function mergeIntoWithNoDuplicateKeys(one, two) { * @param {function} one Function to invoke first. * @param {function} two Function to invoke second. * @return {function} Function that invokes the two argument functions. - * @private */ function createMergedResultFunction(one, two) { return function mergedResult() { @@ -645,7 +643,6 @@ function createMergedResultFunction(one, two) { * @param {function} one Function to invoke first. * @param {function} two Function to invoke second. * @return {function} Function that invokes the two argument functions. - * @private */ function createChainedFunction(one, two) { return function chainedFunction() { @@ -737,8 +734,6 @@ var ReactClassMixin = { /** * Checks whether or not this composite component is mounted. * @return {boolean} True if mounted, false otherwise. - * @protected - * @final */ isMounted: function() { return this.updater.isMounted(this); @@ -765,7 +760,6 @@ var ReactClass = { * * @param {object} spec Class specification (which must define `render`). * @return {function} Component constructor function. - * @public */ createClass: function(spec) { // To keep our warnings more understandable, we'll use a little hack here to diff --git a/src/isomorphic/classic/element/ReactDOMFactories.js b/src/isomorphic/classic/element/ReactDOMFactories.js index 4890fa676c0f..0fa6c0c6fa24 100644 --- a/src/isomorphic/classic/element/ReactDOMFactories.js +++ b/src/isomorphic/classic/element/ReactDOMFactories.js @@ -15,8 +15,6 @@ var ReactElement = require('ReactElement'); /** * Create a factory that creates HTML tag elements. - * - * @private */ var createDOMFactory = ReactElement.createFactory; if (__DEV__) { @@ -27,8 +25,6 @@ if (__DEV__) { /** * Creates a mapping from supported HTML tags to `ReactDOMComponent` classes. * This is also accessible via `React.DOM`. - * - * @public */ var ReactDOMFactories = { a: createDOMFactory('a'), diff --git a/src/isomorphic/classic/element/ReactElement.js b/src/isomorphic/classic/element/ReactElement.js index eb322debea47..b6358d6e04e8 100644 --- a/src/isomorphic/classic/element/ReactElement.js +++ b/src/isomorphic/classic/element/ReactElement.js @@ -373,7 +373,6 @@ ReactElement.cloneElement = function(element, config, children) { * See https://facebook.github.io/react/docs/react-api.html#isvalidelement * @param {?object} object * @return {boolean} True if `object` is a valid component. - * @final */ ReactElement.isValidElement = function(object) { return ( diff --git a/src/isomorphic/modern/class/ReactComponent.js b/src/isomorphic/modern/class/ReactComponent.js index b93da63e94cf..e7f90760e901 100644 --- a/src/isomorphic/modern/class/ReactComponent.js +++ b/src/isomorphic/modern/class/ReactComponent.js @@ -54,8 +54,6 @@ ReactComponent.prototype.isReactComponent = {}; * @param {object|function} partialState Next partial state or function to * produce next partial state to be merged with current state. * @param {?function} callback Called after state is updated. - * @final - * @protected */ ReactComponent.prototype.setState = function(partialState, callback) { invariant( @@ -82,8 +80,6 @@ ReactComponent.prototype.setState = function(partialState, callback) { * `componentWillUpdate` and `componentDidUpdate`. * * @param {?function} callback Called after update is complete. - * @final - * @protected */ ReactComponent.prototype.forceUpdate = function(callback) { this.updater.enqueueForceUpdate(this); diff --git a/src/isomorphic/modern/class/ReactNoopUpdateQueue.js b/src/isomorphic/modern/class/ReactNoopUpdateQueue.js index 49d4c72660a6..79b8028a201a 100644 --- a/src/isomorphic/modern/class/ReactNoopUpdateQueue.js +++ b/src/isomorphic/modern/class/ReactNoopUpdateQueue.js @@ -37,8 +37,6 @@ var ReactNoopUpdateQueue = { * Checks whether or not this composite component is mounted. * @param {ReactClass} publicInstance The instance we want to test. * @return {boolean} True if mounted, false otherwise. - * @protected - * @final */ isMounted: function(publicInstance) { return false; diff --git a/src/renderers/art/ReactART.js b/src/renderers/art/ReactART.js index 0f102926e31b..d3047ee555e4 100644 --- a/src/renderers/art/ReactART.js +++ b/src/renderers/art/ReactART.js @@ -100,7 +100,6 @@ const ContainerMixin = assign({}, ReactMultiChild, { * * @param {ReactComponent} child Component to move. * @param {number} toIndex Destination index of the element. - * @protected */ moveChild: function(child, afterNode, toIndex, lastIndex) { const childNode = child._mountImage; @@ -112,7 +111,6 @@ const ContainerMixin = assign({}, ReactMultiChild, { * * @param {ReactComponent} child Component to create. * @param {object} childNode ART node to insert. - * @protected */ createChild: function(child, afterNode, childNode) { child._mountImage = childNode; @@ -123,7 +121,6 @@ const ContainerMixin = assign({}, ReactMultiChild, { * Removes a child component. * * @param {ReactComponent} child Child to remove. - * @protected */ removeChild: function(child) { child._mountImage.eject(); @@ -144,7 +141,6 @@ const ContainerMixin = assign({}, ReactMultiChild, { * @param {?object} nextChildren. * @param {ReactReconcileTransaction} transaction * @internal - * @override {ReactMultiChild.updateChildren} */ updateChildren: function(nextChildren, transaction, context) { this._updateChildren(nextChildren, transaction, context); diff --git a/src/renderers/dom/shared/escapeTextContentForBrowser.js b/src/renderers/dom/shared/escapeTextContentForBrowser.js index 3735627f823b..64426462853d 100644 --- a/src/renderers/dom/shared/escapeTextContentForBrowser.js +++ b/src/renderers/dom/shared/escapeTextContentForBrowser.js @@ -39,7 +39,6 @@ // code copied and modified from escape-html /** * Module variables. - * @private */ var matchHtmlRegExp = /["'&<>]/; @@ -49,7 +48,6 @@ var matchHtmlRegExp = /["'&<>]/; * * @param {string} string The string to escape for inserting into HTML * @return {string} - * @public */ function escapeHtml(string) { diff --git a/src/renderers/dom/shared/syntheticEvents/SyntheticAnimationEvent.js b/src/renderers/dom/shared/syntheticEvents/SyntheticAnimationEvent.js index 17b6f368b179..35f14e8f65c8 100644 --- a/src/renderers/dom/shared/syntheticEvents/SyntheticAnimationEvent.js +++ b/src/renderers/dom/shared/syntheticEvents/SyntheticAnimationEvent.js @@ -28,7 +28,6 @@ var AnimationEventInterface = { * @param {object} dispatchConfig Configuration used to dispatch this event. * @param {string} dispatchMarker Marker identifying the event target. * @param {object} nativeEvent Native browser event. - * @extends {SyntheticEvent} */ function SyntheticAnimationEvent( dispatchConfig, diff --git a/src/renderers/dom/shared/syntheticEvents/SyntheticClipboardEvent.js b/src/renderers/dom/shared/syntheticEvents/SyntheticClipboardEvent.js index cd0d4b226435..127dde4b2e56 100644 --- a/src/renderers/dom/shared/syntheticEvents/SyntheticClipboardEvent.js +++ b/src/renderers/dom/shared/syntheticEvents/SyntheticClipboardEvent.js @@ -31,7 +31,6 @@ var ClipboardEventInterface = { * @param {object} dispatchConfig Configuration used to dispatch this event. * @param {string} dispatchMarker Marker identifying the event target. * @param {object} nativeEvent Native browser event. - * @extends {SyntheticUIEvent} */ function SyntheticClipboardEvent(dispatchConfig, dispatchMarker, nativeEvent, nativeEventTarget) { return SyntheticEvent.call(this, dispatchConfig, dispatchMarker, nativeEvent, nativeEventTarget); diff --git a/src/renderers/dom/shared/syntheticEvents/SyntheticCompositionEvent.js b/src/renderers/dom/shared/syntheticEvents/SyntheticCompositionEvent.js index 35b7f7ce18b7..905592bc419b 100644 --- a/src/renderers/dom/shared/syntheticEvents/SyntheticCompositionEvent.js +++ b/src/renderers/dom/shared/syntheticEvents/SyntheticCompositionEvent.js @@ -25,7 +25,6 @@ var CompositionEventInterface = { * @param {object} dispatchConfig Configuration used to dispatch this event. * @param {string} dispatchMarker Marker identifying the event target. * @param {object} nativeEvent Native browser event. - * @extends {SyntheticUIEvent} */ function SyntheticCompositionEvent( dispatchConfig, diff --git a/src/renderers/dom/shared/syntheticEvents/SyntheticDragEvent.js b/src/renderers/dom/shared/syntheticEvents/SyntheticDragEvent.js index f5d97420b33b..03b614c936d7 100644 --- a/src/renderers/dom/shared/syntheticEvents/SyntheticDragEvent.js +++ b/src/renderers/dom/shared/syntheticEvents/SyntheticDragEvent.js @@ -25,7 +25,6 @@ var DragEventInterface = { * @param {object} dispatchConfig Configuration used to dispatch this event. * @param {string} dispatchMarker Marker identifying the event target. * @param {object} nativeEvent Native browser event. - * @extends {SyntheticUIEvent} */ function SyntheticDragEvent(dispatchConfig, dispatchMarker, nativeEvent, nativeEventTarget) { return SyntheticMouseEvent.call(this, dispatchConfig, dispatchMarker, nativeEvent, nativeEventTarget); diff --git a/src/renderers/dom/shared/syntheticEvents/SyntheticFocusEvent.js b/src/renderers/dom/shared/syntheticEvents/SyntheticFocusEvent.js index e145b8a434ad..7f5c17d997bd 100644 --- a/src/renderers/dom/shared/syntheticEvents/SyntheticFocusEvent.js +++ b/src/renderers/dom/shared/syntheticEvents/SyntheticFocusEvent.js @@ -25,7 +25,6 @@ var FocusEventInterface = { * @param {object} dispatchConfig Configuration used to dispatch this event. * @param {string} dispatchMarker Marker identifying the event target. * @param {object} nativeEvent Native browser event. - * @extends {SyntheticUIEvent} */ function SyntheticFocusEvent(dispatchConfig, dispatchMarker, nativeEvent, nativeEventTarget) { return SyntheticUIEvent.call(this, dispatchConfig, dispatchMarker, nativeEvent, nativeEventTarget); diff --git a/src/renderers/dom/shared/syntheticEvents/SyntheticInputEvent.js b/src/renderers/dom/shared/syntheticEvents/SyntheticInputEvent.js index ba99c7c894c6..f906bbe43fb9 100644 --- a/src/renderers/dom/shared/syntheticEvents/SyntheticInputEvent.js +++ b/src/renderers/dom/shared/syntheticEvents/SyntheticInputEvent.js @@ -26,7 +26,6 @@ var InputEventInterface = { * @param {object} dispatchConfig Configuration used to dispatch this event. * @param {string} dispatchMarker Marker identifying the event target. * @param {object} nativeEvent Native browser event. - * @extends {SyntheticUIEvent} */ function SyntheticInputEvent( dispatchConfig, diff --git a/src/renderers/dom/shared/syntheticEvents/SyntheticKeyboardEvent.js b/src/renderers/dom/shared/syntheticEvents/SyntheticKeyboardEvent.js index ce71c17ca86f..b4d2e7020c3f 100644 --- a/src/renderers/dom/shared/syntheticEvents/SyntheticKeyboardEvent.js +++ b/src/renderers/dom/shared/syntheticEvents/SyntheticKeyboardEvent.js @@ -73,7 +73,6 @@ var KeyboardEventInterface = { * @param {object} dispatchConfig Configuration used to dispatch this event. * @param {string} dispatchMarker Marker identifying the event target. * @param {object} nativeEvent Native browser event. - * @extends {SyntheticUIEvent} */ function SyntheticKeyboardEvent(dispatchConfig, dispatchMarker, nativeEvent, nativeEventTarget) { return SyntheticUIEvent.call(this, dispatchConfig, dispatchMarker, nativeEvent, nativeEventTarget); diff --git a/src/renderers/dom/shared/syntheticEvents/SyntheticMouseEvent.js b/src/renderers/dom/shared/syntheticEvents/SyntheticMouseEvent.js index e8962b9adecb..403a8c0b7ef6 100644 --- a/src/renderers/dom/shared/syntheticEvents/SyntheticMouseEvent.js +++ b/src/renderers/dom/shared/syntheticEvents/SyntheticMouseEvent.js @@ -69,7 +69,6 @@ var MouseEventInterface = { * @param {object} dispatchConfig Configuration used to dispatch this event. * @param {string} dispatchMarker Marker identifying the event target. * @param {object} nativeEvent Native browser event. - * @extends {SyntheticUIEvent} */ function SyntheticMouseEvent(dispatchConfig, dispatchMarker, nativeEvent, nativeEventTarget) { return SyntheticUIEvent.call(this, dispatchConfig, dispatchMarker, nativeEvent, nativeEventTarget); diff --git a/src/renderers/dom/shared/syntheticEvents/SyntheticTouchEvent.js b/src/renderers/dom/shared/syntheticEvents/SyntheticTouchEvent.js index 38078a732c75..085098104267 100644 --- a/src/renderers/dom/shared/syntheticEvents/SyntheticTouchEvent.js +++ b/src/renderers/dom/shared/syntheticEvents/SyntheticTouchEvent.js @@ -34,7 +34,6 @@ var TouchEventInterface = { * @param {object} dispatchConfig Configuration used to dispatch this event. * @param {string} dispatchMarker Marker identifying the event target. * @param {object} nativeEvent Native browser event. - * @extends {SyntheticUIEvent} */ function SyntheticTouchEvent(dispatchConfig, dispatchMarker, nativeEvent, nativeEventTarget) { return SyntheticUIEvent.call(this, dispatchConfig, dispatchMarker, nativeEvent, nativeEventTarget); diff --git a/src/renderers/dom/shared/syntheticEvents/SyntheticTransitionEvent.js b/src/renderers/dom/shared/syntheticEvents/SyntheticTransitionEvent.js index 982410f78697..cc7212693851 100644 --- a/src/renderers/dom/shared/syntheticEvents/SyntheticTransitionEvent.js +++ b/src/renderers/dom/shared/syntheticEvents/SyntheticTransitionEvent.js @@ -28,7 +28,6 @@ var TransitionEventInterface = { * @param {object} dispatchConfig Configuration used to dispatch this event. * @param {string} dispatchMarker Marker identifying the event target. * @param {object} nativeEvent Native browser event. - * @extends {SyntheticEvent} */ function SyntheticTransitionEvent( dispatchConfig, diff --git a/src/renderers/dom/shared/syntheticEvents/SyntheticUIEvent.js b/src/renderers/dom/shared/syntheticEvents/SyntheticUIEvent.js index 16b5907e959b..adda7692aed8 100644 --- a/src/renderers/dom/shared/syntheticEvents/SyntheticUIEvent.js +++ b/src/renderers/dom/shared/syntheticEvents/SyntheticUIEvent.js @@ -48,7 +48,6 @@ var UIEventInterface = { * @param {object} dispatchConfig Configuration used to dispatch this event. * @param {string} dispatchMarker Marker identifying the event target. * @param {object} nativeEvent Native browser event. - * @extends {SyntheticEvent} */ function SyntheticUIEvent(dispatchConfig, dispatchMarker, nativeEvent, nativeEventTarget) { return SyntheticEvent.call(this, dispatchConfig, dispatchMarker, nativeEvent, nativeEventTarget); diff --git a/src/renderers/dom/shared/syntheticEvents/SyntheticWheelEvent.js b/src/renderers/dom/shared/syntheticEvents/SyntheticWheelEvent.js index eea628e33e1e..c278fb69b1f0 100644 --- a/src/renderers/dom/shared/syntheticEvents/SyntheticWheelEvent.js +++ b/src/renderers/dom/shared/syntheticEvents/SyntheticWheelEvent.js @@ -47,7 +47,6 @@ var WheelEventInterface = { * @param {object} dispatchConfig Configuration used to dispatch this event. * @param {string} dispatchMarker Marker identifying the event target. * @param {object} nativeEvent Native browser event. - * @extends {SyntheticMouseEvent} */ function SyntheticWheelEvent(dispatchConfig, dispatchMarker, nativeEvent, nativeEventTarget) { return SyntheticMouseEvent.call(this, dispatchConfig, dispatchMarker, nativeEvent, nativeEventTarget); diff --git a/src/renderers/dom/stack/client/ReactDOMComponent.js b/src/renderers/dom/stack/client/ReactDOMComponent.js index 0992b42f086f..65546d098d93 100644 --- a/src/renderers/dom/stack/client/ReactDOMComponent.js +++ b/src/renderers/dom/stack/client/ReactDOMComponent.js @@ -673,7 +673,6 @@ ReactDOMComponent.Mixin = { * Iterating over object properties is faster than iterating over arrays. * @see http://jsperf.com/obj-vs-arr-iteration * - * @private * @param {ReactReconcileTransaction|ReactServerRenderingTransaction} transaction * @param {object} props * @return {string} Markup of opening tag. @@ -732,7 +731,6 @@ ReactDOMComponent.Mixin = { /** * Creates markup for the content between the tags. * - * @private * @param {ReactReconcileTransaction|ReactServerRenderingTransaction} transaction * @param {object} props * @param {object} context @@ -914,7 +912,6 @@ ReactDOMComponent.Mixin = { * do not change for a given reconciliation. * TODO: Benchmark areas that can be improved with caching. * - * @private * @param {object} lastProps * @param {object} nextProps * @param {?DOMElement} node diff --git a/src/renderers/dom/stack/client/ReactMount.js b/src/renderers/dom/stack/client/ReactMount.js index b9c756ab179f..ab8f5e41c82d 100644 --- a/src/renderers/dom/stack/client/ReactMount.js +++ b/src/renderers/dom/stack/client/ReactMount.js @@ -166,7 +166,6 @@ function batchedMountComponentIntoNode( * * @param {ReactComponent} instance React component instance. * @param {DOMElement} container DOM element to unmount from. - * @final * @internal * @see {ReactMount.unmountComponentAtNode} */ diff --git a/src/renderers/dom/stack/client/ReactReconcileTransaction.js b/src/renderers/dom/stack/client/ReactReconcileTransaction.js index a14cb71cbfa6..160df575e88d 100644 --- a/src/renderers/dom/stack/client/ReactReconcileTransaction.js +++ b/src/renderers/dom/stack/client/ReactReconcileTransaction.js @@ -128,8 +128,6 @@ function ReactReconcileTransaction(useCreateElement) { var Mixin = { /** * @see Transaction - * @abstract - * @final * @return {array} List of operation wrap procedures. * TODO: convert to array */ diff --git a/src/renderers/dom/stack/client/wrappers/ReactDOMSelect.js b/src/renderers/dom/stack/client/wrappers/ReactDOMSelect.js index ef9033f9a538..72cb35c742ed 100644 --- a/src/renderers/dom/stack/client/wrappers/ReactDOMSelect.js +++ b/src/renderers/dom/stack/client/wrappers/ReactDOMSelect.js @@ -32,7 +32,6 @@ var valuePropNames = ['value', 'defaultValue']; /** * Validation function for `value` and `defaultValue`. - * @private */ function checkSelectPropTypes(inst, props) { var owner = inst._currentElement._owner; @@ -72,7 +71,6 @@ function checkSelectPropTypes(inst, props) { * @param {ReactDOMComponent} inst * @param {boolean} multiple * @param {*} propValue A stringable (with `multiple`, a list of stringables). - * @private */ function updateOptions(inst, multiple, propValue) { var options = ReactDOMComponentTree.getNodeFromInstance(inst).options; diff --git a/src/renderers/dom/stack/server/ReactServerRenderingTransaction.js b/src/renderers/dom/stack/server/ReactServerRenderingTransaction.js index 0a8936bbb790..1701fd776b4e 100644 --- a/src/renderers/dom/stack/server/ReactServerRenderingTransaction.js +++ b/src/renderers/dom/stack/server/ReactServerRenderingTransaction.js @@ -49,8 +49,6 @@ function ReactServerRenderingTransaction(renderToStaticMarkup) { var Mixin = { /** * @see Transaction - * @abstract - * @final * @return {array} Empty list of operation wrap procedures. */ getTransactionWrappers: function() { diff --git a/src/renderers/dom/stack/server/ReactServerUpdateQueue.js b/src/renderers/dom/stack/server/ReactServerUpdateQueue.js index b1e6a54c2ea5..d4d71276d128 100644 --- a/src/renderers/dom/stack/server/ReactServerUpdateQueue.js +++ b/src/renderers/dom/stack/server/ReactServerUpdateQueue.js @@ -51,8 +51,6 @@ class ReactServerUpdateQueue { * Checks whether or not this composite component is mounted. * @param {ReactClass} publicInstance The instance we want to test. * @return {boolean} True if mounted, false otherwise. - * @protected - * @final */ isMounted(publicInstance: ReactComponent): boolean { return false; diff --git a/src/renderers/native/ReactNativeMount.js b/src/renderers/native/ReactNativeMount.js index aa803a74bbae..e42ac760cd58 100644 --- a/src/renderers/native/ReactNativeMount.js +++ b/src/renderers/native/ReactNativeMount.js @@ -215,7 +215,6 @@ var ReactNativeMount = { * * @param {ReactComponent} instance React component instance. * @param {string} containerID ID of container we're removing from. - * @final * @internal * @see {ReactNativeMount.unmountComponentAtNode} */ diff --git a/src/renderers/native/ReactNativeReconcileTransaction.js b/src/renderers/native/ReactNativeReconcileTransaction.js index de34a657b5db..f83a7a8702aa 100644 --- a/src/renderers/native/ReactNativeReconcileTransaction.js +++ b/src/renderers/native/ReactNativeReconcileTransaction.js @@ -73,8 +73,6 @@ function ReactNativeReconcileTransaction() { var Mixin = { /** * @see Transaction - * @abstract - * @final * @return {array} List of operation wrap procedures. * TODO: convert to array */ diff --git a/src/renderers/native/createReactNativeComponentClass.js b/src/renderers/native/createReactNativeComponentClass.js index e5f113bf0bbe..1858586afb48 100644 --- a/src/renderers/native/createReactNativeComponentClass.js +++ b/src/renderers/native/createReactNativeComponentClass.js @@ -23,7 +23,6 @@ type ReactNativeBaseComponentViewConfig = { /** * @param {string} config iOS View configuration. - * @private */ var createReactNativeComponentClass = function( viewConfig: ReactNativeBaseComponentViewConfig diff --git a/src/renderers/shared/shared/event/EventPluginHub.js b/src/renderers/shared/shared/event/EventPluginHub.js index 402335427822..78a7e3a768c8 100644 --- a/src/renderers/shared/shared/event/EventPluginHub.js +++ b/src/renderers/shared/shared/event/EventPluginHub.js @@ -30,7 +30,6 @@ var eventQueue = null; * * @param {?object} event Synthetic event to be dispatched. * @param {boolean} simulated If the event is simulated (changes exn behavior) - * @private */ var executeDispatchesAndRelease = function(event, simulated) { if (event) { @@ -92,8 +91,6 @@ function shouldPreventMouseEvent(name, type, props) { * default, the listener is simply invoked. * * Each plugin that is injected into `EventsPluginHub` is immediately operable. - * - * @public */ var EventPluginHub = { @@ -104,7 +101,6 @@ var EventPluginHub = { /** * @param {array} InjectedEventPluginOrder - * @public */ injectEventPluginOrder: EventPluginRegistry.injectEventPluginOrder, diff --git a/src/renderers/shared/shared/event/EventPluginRegistry.js b/src/renderers/shared/shared/event/EventPluginRegistry.js index 1f09f23f83d2..58407d18c3de 100644 --- a/src/renderers/shared/shared/event/EventPluginRegistry.js +++ b/src/renderers/shared/shared/event/EventPluginRegistry.js @@ -41,8 +41,6 @@ var namesToPlugins: NamesToPlugins = {}; /** * Recomputes the plugin list using the injected plugins and plugin ordering. - * - * @private */ function recomputePluginOrdering(): void { if (!eventPluginOrder) { @@ -90,7 +88,6 @@ function recomputePluginOrdering(): void { * @param {object} dispatchConfig Dispatch configuration for the event. * @param {object} PluginModule Plugin publishing the event. * @return {boolean} True if the event was successfully published. - * @private */ function publishEventForPlugin( dispatchConfig: DispatchConfig, @@ -134,7 +131,6 @@ function publishEventForPlugin( * * @param {string} registrationName Registration name to add. * @param {object} PluginModule Plugin publishing the event. - * @private */ function publishRegistrationName( registrationName: string, @@ -294,7 +290,6 @@ var EventPluginRegistry = { /** * Exposed for unit testing. - * @private */ _resetEventPlugins: function(): void { eventPluginOrder = null; diff --git a/src/renderers/shared/shared/event/eventPlugins/ResponderSyntheticEvent.js b/src/renderers/shared/shared/event/eventPlugins/ResponderSyntheticEvent.js index f1cf77feacfc..f0e73cb2c2cd 100644 --- a/src/renderers/shared/shared/event/eventPlugins/ResponderSyntheticEvent.js +++ b/src/renderers/shared/shared/event/eventPlugins/ResponderSyntheticEvent.js @@ -28,7 +28,6 @@ var ResponderEventInterface = { * @param {object} dispatchConfig Configuration used to dispatch this event. * @param {string} dispatchMarker Marker identifying the event target. * @param {object} nativeEvent Native event. - * @extends {SyntheticEvent} */ function ResponderSyntheticEvent(dispatchConfig, dispatchMarker, nativeEvent, nativeEventTarget) { return SyntheticEvent.call(this, dispatchConfig, dispatchMarker, nativeEvent, nativeEventTarget); diff --git a/src/renderers/shared/shared/shouldUpdateReactComponent.js b/src/renderers/shared/shared/shouldUpdateReactComponent.js index 2c4a15542992..da93bbe0a484 100644 --- a/src/renderers/shared/shared/shouldUpdateReactComponent.js +++ b/src/renderers/shared/shared/shouldUpdateReactComponent.js @@ -20,7 +20,6 @@ * @param {?object} prevElement * @param {?object} nextElement * @return {boolean} True if the existing instance should be updated. - * @protected */ function shouldUpdateReactComponent(prevElement, nextElement) { var prevEmpty = prevElement === null || prevElement === false; diff --git a/src/renderers/shared/stack/reconciler/ReactCompositeComponent.js b/src/renderers/shared/stack/reconciler/ReactCompositeComponent.js index 2a15d71383f6..d64b917a38f1 100644 --- a/src/renderers/shared/stack/reconciler/ReactCompositeComponent.js +++ b/src/renderers/shared/stack/reconciler/ReactCompositeComponent.js @@ -118,21 +118,15 @@ function measureLifeCyclePerf(fn, debugID, timerType) { /** * An incrementing ID assigned to each component when it is mounted. This is * used to enforce the order in which `ReactUpdates` updates dirty components. - * - * @private */ var nextMountID = 1; -/** - * @lends {ReactCompositeComponent.prototype} - */ var ReactCompositeComponent = { /** * Base constructor for all composite component. * * @param {ReactElement} element - * @final * @internal */ construct: function(element) { @@ -175,7 +169,6 @@ var ReactCompositeComponent = { * @param {?object} hostContainerInfo * @param {?object} context * @return {?string} Rendered markup to be inserted into the DOM. - * @final * @internal */ mountComponent: function( @@ -555,7 +548,6 @@ var ReactCompositeComponent = { /** * Releases any resources allocated by `mountComponent`. * - * @final * @internal */ unmountComponent: function(safely, skipLifecycle) { @@ -630,7 +622,6 @@ var ReactCompositeComponent = { * * @param {object} context * @return {?object} - * @private */ _maskContext: function(context) { var Component = this._currentElement.type; @@ -651,7 +642,6 @@ var ReactCompositeComponent = { * * @param {object} context * @return {?object} - * @private */ _processContext: function(context) { var maskedContext = this._maskContext(context); @@ -671,7 +661,6 @@ var ReactCompositeComponent = { /** * @param {object} currentContext * @return {object} - * @private */ _processChildContext: function(currentContext) { var Component = this._currentElement.type; @@ -722,7 +711,6 @@ var ReactCompositeComponent = { * @param {object} typeSpecs Map of context field to a ReactPropType * @param {object} values Runtime values that need to be type-checked * @param {string} location e.g. "prop", "context", "child context" - * @private */ _checkContextTypes: function( typeSpecs, @@ -942,7 +930,6 @@ var ReactCompositeComponent = { * @param {?object} nextContext Next public object to set as context. * @param {ReactReconcileTransaction} transaction * @param {?object} unmaskedContext - * @private */ _performComponentUpdate: function( nextElement, @@ -1125,8 +1112,6 @@ var ReactCompositeComponent = { /** * Overridden in shallow rendering. - * - * @protected */ _replaceNodeWithMarkup: function(oldHostNode, nextMarkup, prevInstance) { ReactComponentEnvironment.replaceNodeWithMarkup( @@ -1136,9 +1121,6 @@ var ReactCompositeComponent = { ); }, - /** - * @protected - */ _renderValidatedComponentWithoutOwnerOrContext: function() { var inst = this._instance; var renderedElement; @@ -1166,9 +1148,6 @@ var ReactCompositeComponent = { return renderedElement; }, - /** - * @private - */ _renderValidatedComponent: function() { var renderedElement; if (__DEV__ || this._compositeType !== CompositeTypes.StatelessFunctional) { @@ -1200,8 +1179,6 @@ var ReactCompositeComponent = { * * @param {string} ref Reference name. * @param {component} component Component to store as `ref`. - * @final - * @private */ attachRef: function(ref, component) { var inst = this.getPublicInstance(); @@ -1229,8 +1206,6 @@ var ReactCompositeComponent = { * Detaches a reference name. * * @param {string} ref Name to dereference. - * @final - * @private */ detachRef: function(ref) { var refs = this.getPublicInstance().refs; diff --git a/src/renderers/shared/stack/reconciler/ReactMultiChild.js b/src/renderers/shared/stack/reconciler/ReactMultiChild.js index 18fa1e8d0bd9..8164f76a1dc6 100644 --- a/src/renderers/shared/stack/reconciler/ReactMultiChild.js +++ b/src/renderers/shared/stack/reconciler/ReactMultiChild.js @@ -28,7 +28,6 @@ var invariant = require('invariant'); * * @param {string} markup Markup that renders into an element. * @param {number} toIndex Destination index. - * @private */ function makeInsertMarkup(markup, afterNode, toIndex) { // NOTE: Null values reduce hidden classes. @@ -47,7 +46,6 @@ function makeInsertMarkup(markup, afterNode, toIndex) { * * @param {number} fromIndex Source index of the existing element. * @param {number} toIndex Destination index of the element. - * @private */ function makeMove(child, afterNode, toIndex) { // NOTE: Null values reduce hidden classes. @@ -65,7 +63,6 @@ function makeMove(child, afterNode, toIndex) { * Make an update for removing an element at an index. * * @param {number} fromIndex Index of the element to remove. - * @private */ function makeRemove(child, node) { // NOTE: Null values reduce hidden classes. @@ -83,7 +80,6 @@ function makeRemove(child, node) { * Make an update for setting the markup of a node. * * @param {string} markup Markup that renders into an element. - * @private */ function makeSetMarkup(markup) { // NOTE: Null values reduce hidden classes. @@ -101,7 +97,6 @@ function makeSetMarkup(markup) { * Make an update for setting the text content. * * @param {string} textContent Text content to set. - * @private */ function makeTextContent(textContent) { // NOTE: Null values reduce hidden classes. @@ -129,8 +124,6 @@ function enqueue(queue, update) { /** * Processes any enqueued updates. - * - * @private */ function processQueue(inst, updateQueue) { ReactComponentEnvironment.processChildrenUpdates( @@ -342,8 +335,6 @@ var ReactMultiChild = { /** * @param {?object} nextNestedChildrenElements Nested child element maps. * @param {ReactReconcileTransaction} transaction - * @final - * @protected */ _updateChildren: function(nextNestedChildrenElements, transaction, context) { var prevChildren = this._renderedChildren; @@ -447,7 +438,6 @@ var ReactMultiChild = { * @param {ReactComponent} child Component to move. * @param {number} toIndex Destination index of the element. * @param {number} lastIndex Last index visited of the siblings of `child`. - * @protected */ moveChild: function(child, afterNode, toIndex, lastIndex) { // If the index of `child` is less than `lastIndex`, then it needs to @@ -463,7 +453,6 @@ var ReactMultiChild = { * * @param {ReactComponent} child Component to create. * @param {string} mountImage Markup to insert. - * @protected */ createChild: function(child, afterNode, mountImage) { return makeInsertMarkup(mountImage, afterNode, child._mountIndex); @@ -473,7 +462,6 @@ var ReactMultiChild = { * Removes a child component. * * @param {ReactComponent} child Child to remove. - * @protected */ removeChild: function(child, node) { return makeRemove(child, node); @@ -488,7 +476,6 @@ var ReactMultiChild = { * @param {string} name Name of the child. * @param {number} index Index at which to insert the child. * @param {ReactReconcileTransaction} transaction - * @private */ _mountChildAtIndex: function( child, @@ -507,7 +494,6 @@ var ReactMultiChild = { * NOTE: This is part of `updateChildren` and is here for readability. * * @param {ReactComponent} child Component to unmount. - * @private */ _unmountChild: function(child, node) { var update = this.removeChild(child, node); diff --git a/src/renderers/shared/stack/reconciler/ReactOwner.js b/src/renderers/shared/stack/reconciler/ReactOwner.js index b6c67fffc767..1a27ba729597 100644 --- a/src/renderers/shared/stack/reconciler/ReactOwner.js +++ b/src/renderers/shared/stack/reconciler/ReactOwner.js @@ -23,7 +23,6 @@ import type { ReactInstance } from 'ReactInstanceType'; /** * @param {?object} object * @return {boolean} True if `object` is a valid owner. - * @final */ function isValidOwner(object: any): boolean { return !!( @@ -70,7 +69,6 @@ var ReactOwner = { * @param {ReactComponent} component Component to reference. * @param {string} ref Name by which to refer to the component. * @param {ReactOwner} owner Component on which to record the ref. - * @final * @internal */ addComponentAsRefTo: function( @@ -100,7 +98,6 @@ var ReactOwner = { * @param {ReactComponent} component Component to dereference. * @param {string} ref Name of the ref to remove. * @param {ReactOwner} owner Component on which the ref is recorded. - * @final * @internal */ removeComponentAsRefFrom: function( diff --git a/src/renderers/shared/stack/reconciler/ReactReconciler.js b/src/renderers/shared/stack/reconciler/ReactReconciler.js index 14a4191710d4..910aa8c7fda7 100644 --- a/src/renderers/shared/stack/reconciler/ReactReconciler.js +++ b/src/renderers/shared/stack/reconciler/ReactReconciler.js @@ -34,7 +34,6 @@ var ReactReconciler = { * @param {?object} the containing host component instance * @param {?object} info about the host container * @return {?string} Rendered markup to be inserted into the DOM. - * @final * @internal */ mountComponent: function( @@ -86,7 +85,6 @@ var ReactReconciler = { /** * Releases any resources allocated by `mountComponent`. * - * @final * @internal */ unmountComponent: function(internalInstance, safely, skipLifecycle) { diff --git a/src/renderers/shared/stack/reconciler/ReactUpdateQueue.js b/src/renderers/shared/stack/reconciler/ReactUpdateQueue.js index 1b555520a783..e4a9b3d98956 100644 --- a/src/renderers/shared/stack/reconciler/ReactUpdateQueue.js +++ b/src/renderers/shared/stack/reconciler/ReactUpdateQueue.js @@ -82,8 +82,6 @@ var ReactUpdateQueue = { * Checks whether or not this composite component is mounted. * @param {ReactClass} publicInstance The instance we want to test. * @return {boolean} True if mounted, false otherwise. - * @protected - * @final */ isMounted: function(publicInstance) { if (__DEV__) { diff --git a/src/renderers/shared/stack/reconciler/Transaction.js b/src/renderers/shared/stack/reconciler/Transaction.js index afa8bc9b38c3..a3509cb616ae 100644 --- a/src/renderers/shared/stack/reconciler/Transaction.js +++ b/src/renderers/shared/stack/reconciler/Transaction.js @@ -98,7 +98,6 @@ var TransactionImpl = { _isInTransaction: false, /** - * @abstract * @return {Array} Array of transaction wrappers. */ getTransactionWrappers: null, diff --git a/src/renderers/shared/stack/reconciler/instantiateReactComponent.js b/src/renderers/shared/stack/reconciler/instantiateReactComponent.js index 202768d8bba1..8f6bdae8c827 100644 --- a/src/renderers/shared/stack/reconciler/instantiateReactComponent.js +++ b/src/renderers/shared/stack/reconciler/instantiateReactComponent.js @@ -63,7 +63,6 @@ function isInternalComponentType(type) { * @param {ReactNode} node * @param {boolean} shouldHaveDebugID * @return {object} A new instance of the element's constructor. - * @protected */ function instantiateReactComponent(node, shouldHaveDebugID) { var instance; diff --git a/src/renderers/testing/ReactTestReconcileTransaction.js b/src/renderers/testing/ReactTestReconcileTransaction.js index 50585e08be10..3666ab400a08 100644 --- a/src/renderers/testing/ReactTestReconcileTransaction.js +++ b/src/renderers/testing/ReactTestReconcileTransaction.js @@ -68,8 +68,6 @@ function ReactTestReconcileTransaction(testOptions: TestRendererOptions) { var Mixin = { /** * @see Transaction - * @abstract - * @final * @return {array} List of operation wrap procedures. * TODO: convert to array */ diff --git a/src/shared/types/checkReactTypeSpec.js b/src/shared/types/checkReactTypeSpec.js index c9315a349cdf..1c34a438755d 100644 --- a/src/shared/types/checkReactTypeSpec.js +++ b/src/shared/types/checkReactTypeSpec.js @@ -45,7 +45,6 @@ var loggedTypeFailures = {}; * @param {string} componentName Name of the component for error messages. * @param {?object} element The React element that is being type-checked * @param {?number} debugID The React component instance that is being type-checked - * @private */ function checkReactTypeSpec( typeSpecs, diff --git a/src/test/ReactTestUtils.js b/src/test/ReactTestUtils.js index 50a50b95daea..1ad824dd2ab8 100644 --- a/src/test/ReactTestUtils.js +++ b/src/test/ReactTestUtils.js @@ -121,7 +121,6 @@ function findAllInRenderedFiberTreeInternal(fiber, test) { * * Todo: Support the entire DOM.scry query syntax. For now, these simple * utilities will suffice for testing purposes. - * @lends ReactTestUtils */ var ReactTestUtils = { renderIntoDocument: function(element) {