Don't wrap values with ReactTextComponent in traverseAllChildren (cloneWithProps)

Fix for #1962

Moves wrapping with `ReactTextComponent` to `flattenChildren` instead, where it belongs.

Also very minor code fix-up. Also worth mentioning that the flattening done by `cloneWIthProps` messes with the reactids, so while the idea is to avoid unnecessary allocations I assume, it does affect outcome (i.e, you can't conditionally use `cloneWithProps` without all components remounting).
Can you remove the require of ReactTextComponent at the top? We aren't using it anymore.