fix(android): Normalize start and end args

[uqmessias]

Summary

Fixes #18579

Normalize start and end arguments when onSelectionChange is
dispatched on Android.

It just applies a fix sent by @TheSavior (Thanks, by the way 😄)

Changelog

[Android] [Fixed] - fix(android): Normalize start and end args

Test Plan

I followed the steps inside react-native/RNTester/README.md in order to build RN from the source for Android. Opened the RNTester App (before applying the following diff) and testing it manually which the result is the following gif:

Diff used for manually test it

diff --git a/RNTester/js/TextInputExample.android.js b/RNTester/js/TextInputExample.android.js
index edeb83e..f172907 100644
--- a/RNTester/js/TextInputExample.android.js
+++ b/RNTester/js/TextInputExample.android.js
@@ -342,6 +342,50 @@ class SelectionExample extends React.Component<
   }
 }
 
+const testStyles = StyleSheet.create({
+  container: {
+    alignItems: 'center',
+    backgroundColor: '#fff',
+    flex: 1,
+    justifyContent: 'center',
+  },
+  textInput: {
+    marginTop: 100,
+    borderColor: 'black',
+    borderWidth: 1,
+    height: 40,
+    width: 150,
+  },
+});
+class TestAppStyleTI extends React.Component<
+  any,
+  {start?: number, end?: number},
+> {
+  constructor(props: any) {
+    super(props);
+    this.state = {start: 0, end: 0};
+  }
+
+  onSelectionChange = (event: any) => {
+    const selection = event.nativeEvent.selection;
+    this.setState({start: selection.start, end: selection.end});
+  };
+
+  render() {
+    return (
+      <View style={testStyles.container}>
+        <Text>{`Start: ${this.state.start}, End: ${this.state.end}`}</Text>
+        <TextInput
+          style={testStyles.textInput}
+          onSelectionChange={this.onSelectionChange}
+          value="initial text"
+          multiline={true}
+        />
+      </View>
+    );
+  }
+}
+
 class AutogrowingTextInputExample extends React.Component<{}> {
   constructor(props) {
     super(props);
@@ -449,6 +493,12 @@ exports.title = '<TextInput>';
 exports.description = 'Single and multi-line text inputs.';
 exports.examples = [
   {
+    title: 'TestAppStyleTI',
+    render: function() {
+      return <TestAppStyleTI />;
+    },
+  },
+  {
     title: 'Auto-focus',
     render: function() {
       return (

Result

[cpojer]

Thanks!

[facebook-github-bot]

@cpojer is landing this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

[react-native-bot]

This pull request was successfully merged by @uqmessias in 2ad3bb2.

^{When will my fix make it into a release? | Upcoming Releases}