Closed Bug 1230568 Opened 7 years ago Closed 7 years ago

Search suggestions are disabled in normal browsing after going to private browsing without choosing to turn off search suggestions

Categories

(Firefox for Android Graveyard :: General, defect)

ARM
Android
defect
Not set
normal

Tracking

(firefox44+ fixed, firefox45 verified, firefox46 verified, b2g-v2.5 fixed, fennec44+)

VERIFIED FIXED
Firefox 46
Tracking Status
firefox44 + fixed
firefox45 --- verified
firefox46 --- verified
b2g-v2.5 --- fixed
fennec 44+ ---

People

(Reporter: TeoVermesan, Assigned: ahunt)

Details

Attachments

(2 files)

Steps to reproduce:
1. Go to normal browsing and start typing in the URL Bar ("br")
("would you like to turn on search suggestions" message is displayed)
2. Don't choose anything and go to private browsing and start typing in the URL Bar ("br"). See that no search suggestions are displayed
3. Go back to normal browsing and start typing in the URL Bar ("br")

Expected results:
- "would you like to turn on search suggestions" message is displayed

Actual results:
- no message is displayed and no search suggestions are displayed
- search suggestions appear to be disabled

Note:
- no reproducible on 42.0.2 and 43 Beta 8
- reproducible using latest Aurora and Nightly
Hi!
From a quick look, it seems that we should set `mSuggestionsOptInPrompt` to null at [1], so that the showSuggestionsOptIn ([2]) works as it is supposed to. Alternatively (a better approach) we could try to change the view's state from gone to visible. I won't be tackle this immediately. But maybe this could be a good first bug or nth bug.
If nobody takes it by around the 20th, I'll probably take it up!

[1]: https://dxr.mozilla.org/mozilla-central/source/mobile/android/base/home/BrowserSearch.java#793
[2]: https://dxr.mozilla.org/mozilla-central/source/mobile/android/base/home/BrowserSearch.java#749
I think we'll need to fix this before the 20th so I'll take it.
Assignee: nobody → michael.l.comella
tracking-fennec: --- → 44+
Thanks for the thorough analysis, Prabhjyot! :)
Spoke with ahunt – he'll actually take it from here. :) If you need some help around the code, let me know.
Assignee: michael.l.comella → ahunt
Bug 1230568 - Allowing showing search suggestion opt in multiple times r=mcomella
Attachment #8696151 - Flags: review?(michael.l.comella)
Comment on attachment 8696151 [details]
MozReview Request: Bug 1230568 - Allowing showing search suggestion opt in multiple times r=mcomella

https://reviewboard.mozilla.org/r/27285/#review24843

Nice!

::: mobile/android/base/home/BrowserSearch.java:747
(Diff revision 1)
> -        // Return if the ViewStub was already inflated - an inflated ViewStub is removed from the
> +        // Simply make the ViewStub visible again if it has already previously been shown.

mega-nit: Simply -> only

Only means do this and nothing else but simply doesn't have the same implication.
Attachment #8696151 - Flags: review?(michael.l.comella) → review+
Comment on attachment 8696151 [details]
MozReview Request: Bug 1230568 - Allowing showing search suggestion opt in multiple times r=mcomella

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/27285/diff/1-2/
Keywords: checkin-needed
Michael, please request uplift to Beta44 (when ready) as this also impacts FF44. Thanks!
Flags: needinfo?(michael.l.comella)
This is a key scenario, tracked for FF44.
Andrzej, can you request uplift to 44?
Flags: needinfo?(michael.l.comella) → needinfo?(ahunt)
Comment on attachment 8696151 [details]
MozReview Request: Bug 1230568 - Allowing showing search suggestion opt in multiple times r=mcomella

Approval Request Comment
[Feature/regressing bug #]:
[User impact if declined]: Users may not discover the ability to enable search suggestions if the prompt is overseen when first shown - enabling search suggestions in that case requires opening the application settings.
[Describe test coverage new/current, TreeHerder]:
[Risks and why]: low risk - this changes visibility of a view that already exists.
[String/UUID change made/needed]: none
Flags: needinfo?(ahunt)
Attachment #8696151 - Flags: approval-mozilla-aurora?
https://hg.mozilla.org/mozilla-central/rev/97cbdba9513d
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 46
Comment on attachment 8696151 [details]
MozReview Request: Bug 1230568 - Allowing showing search suggestion opt in multiple times r=mcomella

Adding approval request for beta too. Sorry, I got confused by the version changes yesterday, and erroneously thought 44 was still aurora, whereas it's now beta.

Approval Request Comment
[Feature/regressing bug #]:
[User impact if declined]: Users may not discover the ability to enable search suggestions if the prompt is overseen when first shown - enabling search suggestions in that case requires opening the application settings.
[Describe test coverage new/current, TreeHerder]:
[Risks and why]: low risk - this changes visibility of a view that already exists.
[String/UUID change made/needed]: none
Attachment #8696151 - Flags: approval-mozilla-beta?
Teodora, could you please verify this issue is fixed as expected on a latest Nightly build? Thanks!
Flags: needinfo?(teodora.vermesan)
Comment on attachment 8696151 [details]
MozReview Request: Bug 1230568 - Allowing showing search suggestion opt in multiple times r=mcomella

Key scenarios that needs to be fixed soon, Beta44+, Aurora45+
Attachment #8696151 - Flags: approval-mozilla-beta?
Attachment #8696151 - Flags: approval-mozilla-beta+
Attachment #8696151 - Flags: approval-mozilla-aurora?
Attachment #8696151 - Flags: approval-mozilla-aurora+
After returning from private to normal browsing and start typing something in the URL Bar, "Would you like to turn on search suggestions" message is displayed, so:
Verified as fixed using:
Device: Samsung Galaxy Tab S2 (Android 5.0.2)
Build: Firefox for Android 46.0a1 (2015-12-17)
Flags: needinfo?(teodora.vermesan)
This is hitting conflicts uplifting it to beta, likely due to the lack of bug 1107811. Can we get a rebased patch in here to work around it, if this really needs to go to beta?
After returning from private to normal browsing and start typing something in the URL Bar, "Would you like to turn on search suggestions" message is displayed, so:
Verified as fixed using:
Device: Nexus 6 (Android 6.0)
Build: Firefox for Android 45.0a2 (2015-12-23)
Here's the rebased patch!
Flags: needinfo?(ahunt)
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.