Closed Bug 1276910 Opened 3 years ago Closed 3 years ago

Hide "Add to home screen" in private browsing

Categories

(Firefox for Android :: Web Apps, enhancement)

49 Branch
ARM
Android
enhancement
Not set

Tracking

()

RESOLVED FIXED
Firefox 49
Tracking Status
firefox48 --- fixed
firefox49 --- fixed

People

(Reporter: TeoVermesan, Assigned: sebastian)

References

Details

Attachments

(3 files)

Steps to reproduce:
1. Visit a site several times in private browsing 

Expected results:
- "Add to home screen" notification is displayed

Actual results:
- Don't show "Add to home screen" in private browsing
This feature should be disabled, but I agree this is a valid bug we would want to prevent from happening if we're going to ship this feature.
Blocks: 1232706
Assignee: nobody → s.kaspari
Status: NEW → ASSIGNED
Comment on attachment 8758606 [details]
MozReview Request: Bug 1276910 - Do not show "Add to home screen" prompt in private browsing. r?margaret

Approval Request Comment

[Feature/regressing bug #]: "Add to home screen" is an experiment that is enabled in Nightly and Aurora. As this code rides the trains we want to enable it in Beta too to get more data (Bug 1277180).

[User impact if declined]: Without this patch we potentially show this prompt in private browsing too. This is something we do not want. So this would block enabling this in Beta.

[Describe test coverage new/current, TreeHerder]: Local testing.

[Risks and why]: Very low. This patch just checks if the tab is private and if so doesn't execute the "Add to home screen" code.

[String/UUID change made/needed]: -
Attachment #8758606 - Flags: approval-mozilla-aurora?
Comment on attachment 8758606 [details]
MozReview Request: Bug 1276910 - Do not show "Add to home screen" prompt in private browsing. r?margaret

https://reviewboard.mozilla.org/r/56860/#review53602
Attachment #8758606 - Flags: review?(margaret.leibovic) → review+
https://hg.mozilla.org/integration/fx-team/rev/faa1661edc002da6a574b08a4b24c9a55f29e6d3
Bug 1276910 - Do not show "Add to home screen" prompt in private browsing. r=margaret
https://hg.mozilla.org/mozilla-central/rev/faa1661edc00
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 49
Comment on attachment 8758606 [details]
MozReview Request: Bug 1276910 - Do not show "Add to home screen" prompt in private browsing. r?margaret

Make sense, taking it
Attachment #8758606 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
has problems to apply to aurora :

grafting 347866:faa1661edc00 "Bug 1276910 - Do not show "Add to home screen" prompt in private browsing. r=margaret"
merging mobile/android/base/java/org/mozilla/gecko/promotion/AddToHomeScreenPromotion.java
warning: conflicts while merging mobile/android/base/java/org/mozilla/gecko/promotion/AddToHomeScreenPromotion.java! (edit, then use 'hg resolve --mark')
abort: unresolved conflicts, can't continue
(use 'hg resolve' and 'hg graft --continue')
Flags: needinfo?(s.kaspari)
Modified patch for Aurora.
Flags: needinfo?(s.kaspari) → needinfo?(cbook)
You need to log in before you can comment on or make changes to this bug.