Closed Bug 1627499 Opened 5 years ago Closed 5 years ago

Showing top site items/Recently visited items in Private Mode is inconsistent in Address Bar

Categories

(Firefox :: Address Bar, defect, P2)

defect
Points:
2

Tracking

()

RESOLVED FIXED
Firefox 78
Iteration:
78.2 - May 18 - May 31
Tracking Status
firefox-esr68 --- unaffected
firefox74 --- unaffected
firefox75 --- wontfix
firefox76 --- wontfix
firefox77 --- wontfix
firefox78 --- fixed

People

(Reporter: mehmet.sahin, Assigned: mak)

References

(Blocks 1 open bug, Regression)

Details

(Keywords: blocked-ux, privacy, regression)

Attachments

(1 file, 1 obsolete file)

Nightly 76.0a1 (2020-04-04)
macOS 10.15.4

STR:
1.) Open a Private Window
2.) Visit a page
3.) Focus the Address Bar

Result: No Top site items/No Recently visited items are shown in Address Bar (probabaly okay, because this is an Private Mode Window)

4.) But now clear the Address Bar

Result: Top SItes items/Recently visited items are shown in Address Bar

Not sure if this is intended. But this feels inconsistent. Maybe they shouldn't be shown at all in Private Mode or show them also when focussing the Address Bar.

Thanks.

Flags: needinfo?(dao+bmo)
Regressed by: 1617408
Has Regression Range: --- → yes
Flags: needinfo?(dao+bmo)
Flags: needinfo?(dao+bmo)
Priority: -- → P1

Depending on the result of bug 1628997, we may or may not want to fix this...

Blocks: 1630275
No longer blocks: urlbar-update-1
Priority: P1 → P2
Summary: [Megabar] Showing top site items/Recently visited items in Private Mode is inconsitent in Address Bar → Showing top site items/Recently visited items in Private Mode is inconsitent in Address Bar
Points: --- → 2
Summary: Showing top site items/Recently visited items in Private Mode is inconsitent in Address Bar → Showing top site items/Recently visited items in Private Mode is inconsistent in Address Bar

For now let's fix this so the result is consistent, then we'll decide what to do for bug 1628997 and we can eventually undo this.

Assignee: nobody → mak
Iteration: --- → 77.2 - Apr 20 - May 3
Flags: needinfo?(dao+bmo)

Depends on D73279

Attachment #9144703 - Attachment is obsolete: true

The problem is distinguishing an effective empty search (the user types a space) from a text removal bringing to an empty search.
The patch I made also disallowed effective empty searches, and it's unclear what the behavior should be if openViewOnFocus is disabled.
In the new world where the pref states "don't show Top Sites", it would be easier, we just don't show anything for an empty search.
So the expected outcome of browser/components/urlbar/tests/browser/browser_empty_search.js is unclear
And there is also https://searchfox.org/mozilla-central/rev/7fd1c1c34923ece7ad8c822bee062dd0491d64dc/browser/components/extensions/test/browser/browser_ext_urlbar.js#235-236

This is a lot more complicate than expected without the final pref.

Assignee: mak → nobody
Iteration: 77.2 - Apr 20 - May 3 → ---
Depends on: 1627858
Assignee: nobody → mak
Status: NEW → ASSIGNED
Iteration: --- → 78.2 - May 18 - May 31
Pushed by mak77@bonardo.net: https://hg.mozilla.org/integration/autoland/rev/ab17874438ee Address Bar should not show Top Sites in Private Browsing when clearing input. r=harry
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 78
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: