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

Commit 2efeecca authored by Zimuzo's avatar Zimuzo
Browse files

Fix location split-permission targetSdk 

Ia5b3f47b73c9feea924373268a4eee142f555091 introduced a bug where the targetSdk for android.permission.ACCESS_FINE_LOCATION and android.permission.ACCESS_COARSE_LOCATION was set to 28 instead of Q (10000).

Test: CtsAppThatRequestsLocationPermission28.apk requests android.permission.ACCESS_COARSE_LOCATION and android.permission.ACCESS_BACKGROUND_LOCATION
Bug: 118882117
Bug: 111411340
Change-Id: I532379aa2c8a173a516d38e1c8568cff5dbaed33
parent f1a20f5e

data/etc/platform.xml

+4 −2
Original line number Diff line number Diff line
@@ -189,12 +189,14 @@ 
                      targetSdk="16">

        <new-permission name="android.permission.WRITE_CALL_LOG" />

    </split-permission>

    <!-- STOPSHIP(b/118882117): change targetSdk to Q when SDK version finalised -->

    <split-permission name="android.permission.ACCESS_FINE_LOCATION"

                      targetSdk="28">

                      targetSdk="10000">

        <new-permission name="android.permission.ACCESS_BACKGROUND_LOCATION" />

    </split-permission>

    <!-- STOPSHIP(b/118882117): change targetSdk to Q when SDK version finalised -->

    <split-permission name="android.permission.ACCESS_COARSE_LOCATION"

                      targetSdk="28">

                      targetSdk="10000">

        <new-permission name="android.permission.ACCESS_BACKGROUND_LOCATION" />

    </split-permission>