Skip to content

Commit 8c42c01

Browse files
devon94facebook-github-bot
authored andcommitted
Fix debugging on android for 0.63 (react#29204)
Summary: Currently on react native 0.63-rc.0 and 0.63-rc.1 enabling debugging throws an exception. It looks like something may have been missed in unregistering JSDevSupport in this commit c20963e ![crash](https://user-images.githubusercontent.com/14797029/85500252-2acae400-b5b1-11ea-938a-674b55e649b2.gif) This should fix react#28746 and react#29136 ## Changelog [Android] [Fixed] - Fix crash when enabling debug Pull Request resolved: react#29204 Test Plan: To recreate the bug: npx react-native init RN063 --version 0.63.0-rc.1 react-native start react-native run-android Enable debug mode from react native dev menu After this commit, the crash no longer occurs ![non crash](https://user-images.githubusercontent.com/14797029/85500241-269ec680-b5b1-11ea-8cfe-85bfda4dd222.gif) Reviewed By: TheSavior Differential Revision: D22395406 Pulled By: RSNara fbshipit-source-id: 046df77ae1c1de96870fb46f409d59e7d6a68c0d
1 parent f2fdc1a commit 8c42c01

1 file changed

Lines changed: 1 addition & 5 deletions

File tree

ReactAndroid/src/main/java/com/facebook/react/DebugCorePackage.java

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,6 @@
1010
import com.facebook.react.bridge.NativeModule;
1111
import com.facebook.react.bridge.ReactApplicationContext;
1212
import com.facebook.react.devsupport.JSCHeapCapture;
13-
import com.facebook.react.devsupport.JSDevSupport;
1413
import com.facebook.react.module.annotations.ReactModule;
1514
import com.facebook.react.module.annotations.ReactModuleList;
1615
import com.facebook.react.module.model.ReactModuleInfo;
@@ -50,10 +49,7 @@ public ReactModuleInfoProvider getReactModuleInfoProvider() {
5049
return (ReactModuleInfoProvider) reactModuleInfoProviderClass.newInstance();
5150
} catch (ClassNotFoundException e) {
5251
// In OSS case, the annotation processor does not run. We fall back on creating this by hand
53-
Class<? extends NativeModule>[] moduleList =
54-
new Class[] {
55-
JSCHeapCapture.class, JSDevSupport.class,
56-
};
52+
Class<? extends NativeModule>[] moduleList = new Class[] {JSCHeapCapture.class};
5753

5854
final Map<String, ReactModuleInfo> reactModuleInfoMap = new HashMap<>();
5955
for (Class<? extends NativeModule> moduleClass : moduleList) {

0 commit comments

Comments
 (0)