1882114 - Remove obsolete Firefox View prefs

Status: RESOLVED FIXED

(Firefox :: Firefox View, defect, P2)

Description

[lagiann]

Steps to reproduce:

Go to about:config and set any of the following options to "false" in Firefox 123
browser.tabs.firefox-view
browser.tabs.firefox-view-newIcon
browser.tabs.firefox-view-next

None of this options apparently works in Firefox 123.

In contrast, "browser.firefox-view.search.enabled" works as expected in 123 and all of them works normally in Firefox 122

Actual results:

Apparently nothing happens

Expected results:

Firefox should honor the option set.

Comment 1

[BugBot [:suhaib / :marco/ :calixte]]

The Bugbug bot thinks this bug should belong to the 'Firefox::Firefox View' component, and is moving the bug to that component. Please correct in case you think the bot is wrong.

Comment 2

[Sarah Clements [:sclements]]

Those prefs were all supposed to be removed in various bugs as part of cleanup - they were intended for feature development only. It looks like they were accidentally added back into the profile.js file in bug 1783521 but they don't control anything anymore.

browser.firefox-view.search.enabled also is for feature development and will be removed.

Comment 3

[Monica Chiorean]

Since this is a task, I think it is ok to mark it's status as New, right?

Comment 4

[Sarah Clements [:sclements]]

(In reply to Monica Chiorean from comment #3)

Since this is a task, I think it is ok to mark it's status as New, right?

Yes, I've updated it.

Comment 5

[Nikki Sharpley (:nikkis) (she/her)]

Attachment: Bug 1882114 - Remove unused Firefox View prefs r=#fxview-reviewers

These were accidentally added back in to the profile.js file in bug 1783521 but don't do anything.

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/0a0c1f9ef23f
Remove unused Firefox View prefs r=fxview-reviewers,kcochrane

Comment 7

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/0a0c1f9ef23f

Comment 8

[BugBot [:suhaib / :marco/ :calixte]]

Set release status flags based on info from the regressing bug 1783521