Closed Bug 1126139 Opened 7 years ago Closed 7 years ago

Add an android:label for the reading list provider

Categories

(Firefox for Android Graveyard :: Data Providers, defect)

All
Android
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
Firefox 38

People

(Reporter: rnewman, Assigned: rnewman)

References

Details

Attachments

(1 file)

This is necessary for it to show a meaningful checkbox in Android's account settings.
Pretty trivial, but wanted to make sure this wasn't only in my head.

This shows "Reading List" in the account settings.
Attachment #8555009 - Flags: review?(nalexander)
Comment on attachment 8555009 [details] [diff] [review]
Add an android:label for the reading list provider. v1

Review of attachment 8555009 [details] [diff] [review]:
-----------------------------------------------------------------

::: mobile/android/base/AndroidManifest.xml.in
@@ +427,5 @@
>                    android:permission="@ANDROID_PACKAGE_NAME@.permissions.BROWSER_PROVIDER"/>
>  
>          <provider android:name="org.mozilla.gecko.db.ReadingListProvider"
>                    android:authorities="@ANDROID_PACKAGE_NAME@.db.readinglist"
> +                  android:label="@string/reading_list_title"

Let's make this android:exported="false" while we're here.
Attachment #8555009 - Flags: review?(nalexander) → review+
https://hg.mozilla.org/mozilla-central/rev/1028a620d607
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 38
Blocks: 1128189
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.