fix: NativeEventEmitter warnings for android (#2975)

Summary:
With React Native v0.65+, `NativeEventEmitter` shows non-null argument warnings. Issue https://github.com/facebook/flipper/issues/2707
```
`new NativeEventEmitter()` was called with a non-null argument without the required `addListener` method.
`new NativeEventEmitter()` was called with a non-null argument without the required `removeListeners` method.
```

## Changelog
https://github.com/facebook/flipper/issues/2707 - Fix NativeEventEmitter warnings for android

Pull Request resolved: https://github.com/facebook/flipper/pull/2975

Test Plan:
- Add `redux-flipper` plugin to `ReactNativeFlipperExample` project

## Screenshots

#### Before
![Screenshot_1634241069](https://user-images.githubusercontent.com/20144332/137386556-79bbad7e-74b9-4343-ad45-26119557142a.png)

#### After
![Screenshot_1634241159](https://user-images.githubusercontent.com/20144332/137386572-54c74ca5-a70a-4fd4-9644-d5e15f817128.png)

Reviewed By: passy

Differential Revision: D32243694

Pulled By: mweststrate

fbshipit-source-id: 21e490a275875268fea982ac1e855ca4bed02bc8
This commit is contained in:
Archit Raj
2021-11-08 04:22:03 -08:00
committed by Facebook GitHub Bot
parent 5047f4532e
commit d33041cd9e

View File

@@ -78,6 +78,16 @@ public class FlipperModule extends ReactContextBaseJavaModule {
mManager.respondError(responderId, data);
}
@ReactMethod
public void addListener(String eventName) {
// Set up any upstream listeners or background tasks as necessary
}
@ReactMethod
public void removeListeners(Integer count) {
// Remove upstream listeners, stop unnecessary background tasks
}
void sendJSEvent(String eventName, WritableMap params) {
final ReactApplicationContext context = getReactApplicationContextIfActiveOrWarn();
if (context != null) {