Update data reporting notification to point to privacy preference screen

VERIFIED FIXED in Firefox 45

Status

()

VERIFIED FIXED
3 years ago
3 years ago

People

(Reporter: Margaret, Assigned: Margaret)

Tracking

35 Branch
Firefox 45
Points:
---

Firefox Tracking Flags

(firefox45 verified)

Details

Attachments

(2 attachments)

(Assignee)

Description

3 years ago
Right now it points to the vendor screen, since that's where data choices are, but that's changing with bug 1216257.
(Assignee)

Updated

3 years ago
Duplicate of this bug: 1221559
(Assignee)

Comment 2

3 years ago
Created attachment 8683152 [details]
MozReview Request: Bug 1220833 - Update data reporting notification to point to privacy preference screen. r=sebastian

Bug 1220833 - Update data reporting notification to point to privacy preference screen. r=sebastian
Attachment #8683152 - Flags: review?(s.kaspari)
(Assignee)

Comment 3

3 years ago
Created attachment 8683153 [details]
MozReview Request: Bug 1220833 - (Part 2) Properly select header on tablet when launching preference screen directly. r=sebastian

Bug 1220833 - (Part 2) Properly select header on tablet when launching preference screen directly. r=sebastian
Attachment #8683153 - Flags: review?(s.kaspari)
(Assignee)

Comment 4

3 years ago
Somewhat surprisingly, I found that `activity.isMultiPane()` returns true on my N5x, so I had to add a null check to `switchToHeader`, since it could end up being called on a device that doesn't actually have headers to show.

Also, note that the problem for my second patch is an old pre-existing issue (I could reproduce on release), so this is just making things better :)
(Assignee)

Comment 5

3 years ago
Comment on attachment 8683153 [details]
MozReview Request: Bug 1220833 - (Part 2) Properly select header on tablet when launching preference screen directly. r=sebastian

Bug 1220833 - (Part 2) Properly select header on tablet when launching preference screen directly. r=sebastian
Attachment #8683152 - Flags: review?(s.kaspari) → review+
Comment on attachment 8683152 [details]
MozReview Request: Bug 1220833 - Update data reporting notification to point to privacy preference screen. r=sebastian

https://reviewboard.mozilla.org/r/24199/#review21827
Attachment #8683153 - Flags: review?(s.kaspari) → review+
Comment on attachment 8683153 [details]
MozReview Request: Bug 1220833 - (Part 2) Properly select header on tablet when launching preference screen directly. r=sebastian

https://reviewboard.mozilla.org/r/24201/#review21917

::: mobile/android/base/preferences/GeckoPreferences.java:516
(Diff revision 2)
> +        while (iterator.hasNext()) {

nit: As the iterator is only used for looping, maybe a for-each looper would be easier to read?

for (Header header : mHeaders) {
..
}

::: mobile/android/base/preferences/GeckoPreferences.java:518
(Diff revision 2)
> +            if (header.id == id) {
> +                switchToHeader(header);
> +            }

Should we return after finding the header?

::: mobile/android/base/preferences/GeckoPreferences.java:518
(Diff revision 2)
> +            if (header.id == id) {
> +                switchToHeader(header);
> +            }

Should we return after finding the header?
(Assignee)

Comment 8

3 years ago
https://hg.mozilla.org/integration/fx-team/rev/b51d63ae93a5241edeb8791d00b9cf202944f712
Bug 1220833 - Update data reporting notification to point to privacy preference screen. r=sebastian

https://hg.mozilla.org/integration/fx-team/rev/4e951da373d703bce17600cea9ac9b0f509c84e5
Bug 1220833 - (Part 2) Properly select header on tablet when launching preference screen directly. r=sebastian
(Assignee)

Comment 10

3 years ago
https://hg.mozilla.org/integration/fx-team/rev/e097f75e306eaf010266b61e19529454c87fa252
Bug 1220833 - (Part 2) Properly select header on tablet when launching preference screen directly. r=sebastian

Comment 11

3 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/b51d63ae93a5
Status: NEW → RESOLVED
Last Resolved: 3 years ago
status-firefox45: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 45
Tapping "Choose what to share" button, opens "Privacy" sub-menu, so:
Verified as fixed using:
Device: Nexus 9 (Android 6.0) and Xiaomi Mi i4 (Android 5.0.2)
Status: RESOLVED → VERIFIED
status-firefox45: fixed → verified
You need to log in before you can comment on or make changes to this bug.