Opened 3 years ago

Last modified 2 months ago

#4620 new change

Remove unused manifest permissions

Reported by: diegocarloslima Assignee:
Priority: P4 Milestone:
Module: Adblock-Browser-for-Android Keywords:
Cc: Blocked By:
Blocking: Platform: Adblock Browser for Android
Ready: no Confidential: no
Tester: Unknown Verified working: no
Review URL(s):

Description

Background

Currently, we have disabled some features of Firefox that requires a few permissions in the app manifest. It might be interesting to remove these permissions from our manifest, so pre-Marshmallow users will see less permissions requirements when installing our app from the Google Play Store.

What to change

Remove unused permissions from manifest.

Attachments (1)

Screenshot_20170912-155530.png​ (151.3 KB) - added by jwangenheim 2 years ago.

Download all attachments as: .zip

Change History (4)

comment:1 Changed 2 years ago by diegocarloslima

  • Priority changed from Unknown to P4

comment:2 Changed 2 years ago by jwangenheim

With the merge of FENNEC 53 we start using runtime permissions, so the user does not have to grant any permission when installing ABB anymore. Permission now need to be granted as soon as they are needed for the first time (see attached screenshot).

@diego: Do you remember which permissions are obsolete? I think we can remove the following:

  • com.android.browser.permission.READ_HISTORY_BOOKMARKS -> as we don't support to import bookmarks and/or history anymore

I am not 100% sure about

  • uses-permission android:name="android.permission.DOWNLOAD_WITHOUT_NOTIFICATION"

Changed 2 years ago by jwangenheim

comment:3 Changed 6 months ago by Salman86

spam

Last edited 2 months ago by kzar (previous) (diff)
Note: See TracTickets for help on using tickets.