Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit eabd2120 authored by Kevin Jeon's avatar Kevin Jeon
Browse files

Remove unnecessary permissions from Traceur 

This change removes the privapp allowlist from Traceur, which had
previously explicitly granted the DUMP, QUERY_ALL_PACKAGES,
START_FOREGROUND_SERVICES_FROM_BACKGROUND, and STATUS_BAR permissions.
These permissions no longer need to be granted because Traceur is now a
platform app.

Bug: 209476712
Test: - Checked that Traceur can still take normal and long traces on
        AOSP userdebug and internal user/userdebug.
      - Checked that bugreports will stop ongoing traces.
      - Checked that the quick settings tile still stops/starts traces.
Change-Id: I33cd317f9fe00e2e08ae205a832237bfe0df044a
Merged-In: I33cd317f9fe00e2e08ae205a832237bfe0df044a
parent 9f165ca3

data/etc/privapp-permissions-platform.xml

+0 −11
Original line number Diff line number Diff line
@@ -519,17 +519,6 @@ applications that come with the platform 
        <permission name="android.permission.INTERACT_ACROSS_USERS"/>

    </privapp-permissions>



    <privapp-permissions package="com.android.traceur">

        <!-- Permissions required to receive BUGREPORT_STARTED intent -->

        <permission name="android.permission.DUMP"/>

        <!-- Permissions required to start/stop tracing -->

        <permission name="android.permission.START_FOREGROUND_SERVICES_FROM_BACKGROUND"/>

        <!-- Permissions required for quick settings tile -->

        <permission name="android.permission.STATUS_BAR"/>

        <!-- Permissions required to query Betterbug -->

        <permission name="android.permission.QUERY_ALL_PACKAGES"/>

    </privapp-permissions>



    <privapp-permissions package="com.android.tv">

        <permission name="android.permission.CHANGE_HDMI_CEC_ACTIVE_SOURCE"/>

        <permission name="android.permission.DVB_DEVICE"/>