Remove deprecated removeEventListener() calls

[TheWirv]

PR Checklist

• The PR title follows our guidelines: https://github.com/proyecto26/react-native-inappbrowser/blob/main/CONTRIBUTING.md#pull-request-process.
• There is an issue for the bug/feature this PR is for. To avoid wasting your time, it's best to open a suggestion issue first and wait for approval before working on it.
• All existing tests are passing
• Tests for the changes are included

What is the current behavior?

For React Native's Linking and AppState APIs, the deprecated removeEventListener() calls had been used before.

What is the new behavior?

Both Linking.addEventListener() and AppState.addEventListener() return an event subscription object containing a corresponding remove() method. They are being used in utils.js now.

Fixes/Implements/Closes #296.
No breaking changes

@jdnichollsc I am a TypeScript person and have never worked with flow before. Please check the flow types whether they are still correct.

[jdnichollsc]

Thanks for your contribution mate, please let me check this ASAP! <3

[radko93]

@jdnichollsc would it be possible to merge that? Please not this is a breaking change for older RN versions.

[jdnichollsc]

Hello folks, thanks for this awesome contribution, do you know if this works for previous versions of RN?

Thanks for your help!

[radko93]

@jdnichollsc This PR is not backwards compatible (at least not all the way to older versions), but it is possible. I've made a similar PR recently https://github.com/AgoraIO-Community/react-native-agora/pull/414/files

[TheWirv]

@radko93 @jdnichollsc Hey guys, please check out the PR's current state after my newest commit. This should make it backwards-compatible, I think.

[mr-rick]

I have tested your latest commit and I'm getting these errors...

I/ViewRootImpl@97b629a[MainActivity]: ViewPostIme pointer 0
I/ViewRootImpl@97b629a[MainActivity]: ViewPostIme pointer 1
I/ViewRootImpl@97b629a[MainActivity]: ViewPostIme pointer 0
I/ViewRootImpl@97b629a[MainActivity]: ViewPostIme pointer 1
I/ViewRootImpl@97b629a[MainActivity]: ViewPostIme pointer 0
I/ViewRootImpl@97b629a[MainActivity]: ViewPostIme pointer 1
I/ViewRootImpl@97b629a[MainActivity]: ViewPostIme pointer 0
I/ViewRootImpl@97b629a[MainActivity]: ViewPostIme pointer 1```

A quick google search says..... "The reason is because you miss to set listener to your button."

btn.setOnClickListener(this);

I'm on react native 66.3

[jdnichollsc]

@TheWirv hey mate, do you have any idea about those warnings? 🔝

[TheWirv]

No, absolutely not, sorry. :/

[jdnichollsc]

No problem, let me try and thanks for your help mate! <3

[ajaxjiang96]

Is their a release plan for this patch?

[jdnichollsc]

I'm creating a new Release 3.7.0, stay tuned! <3