Skip to content

Commit 56fc630

Browse files
dulmandakhgrabbou
authored andcommittedFeb 27, 2019
SYSTEM_ALERT_WINDOW only in debug builds (#23504)
Summary: SYSTEM_ALERT_WINDOW permission is used only for debug builds to show draw overlay views or show warnings/errors. This permissions is unused in release builds, and there is an issue regarding removing unused permissions on Android build #5886 (comment). This PR removes SYSTEM_ALERT_WINDOW from all builds bug debug. [Android] [Changed] - SYSTEM_ALERT_WINDOW permissions available only in debug builds Pull Request resolved: #23504 Differential Revision: D14123045 Pulled By: cpojer fbshipit-source-id: 68829a774ff23c7cb2721076a9d3870405f48fea
1 parent dff3f60 commit 56fc630

File tree

3 files changed

+3
-3
lines changed

3 files changed

+3
-3
lines changed
 
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,5 @@
11
<manifest xmlns:android="http://schemas.android.com/apk/res/android" package="com.facebook.react">
22

3-
<uses-permission android:name="android.permission.SYSTEM_ALERT_WINDOW"/>
4-
53
<application />
64

75
</manifest>
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,8 @@
11
<?xml version="1.0" encoding="utf-8"?>
22
<manifest xmlns:android="http://schemas.android.com/apk/res/android"
33
xmlns:tools="http://schemas.android.com/tools">
4+
5+
<uses-permission android:name="android.permission.SYSTEM_ALERT_WINDOW"/>
6+
47
<application tools:targetApi="28" tools:ignore="GoogleAppIndexingWarning" android:networkSecurityConfig="@xml/react_native_config" />
58
</manifest>

‎template/android/app/src/main/AndroidManifest.xml

-1
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,6 @@
22
package="com.helloworld">
33

44
<uses-permission android:name="android.permission.INTERNET" />
5-
<uses-permission android:name="android.permission.SYSTEM_ALERT_WINDOW"/>
65

76
<application
87
android:name=".MainApplication"

0 commit comments

Comments
 (0)
Please sign in to comment.