Closed
Bug 898881
Opened 11 years ago
Closed 11 years ago
OpenSearch autodiscovery disabled in Private Browsing Mode
Categories
(Firefox :: Private Browsing, defect)
Tracking
()
RESOLVED
FIXED
Firefox 25
People
(Reporter: mozilla.org, Assigned: ehsan.akhgari)
Details
Attachments
(1 file)
1.16 KB,
patch
|
jdm
:
review+
|
Details | Diff | Splinter Review |
User Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:17.0) Gecko/20100101 Firefox/17.0 (Beta/Release) Build ID: 20130618114625 Steps to reproduce: Enter Private Browsing Mode. Navigate to a site (ex: http://fr.wikipedia.org/) with an opensearch plugin using a link rel=search tag. Click Firefox search drop-down menu. Actual results: There is no option to add the search plugin. Expected results: There should be an option to add the search plugin.
Comment 1•11 years ago
|
||
Huh, reading code like rebuildPopupDynamic (http://mxr.mozilla.org/mozilla-central/source/browser/components/search/content/search.xml#307), I can't see why this would not be appearing.
Comment 2•11 years ago
|
||
Aha: http://mxr.mozilla.org/mozilla-central/source/browser/base/content/browser.js#3013
Comment 3•11 years ago
|
||
Ehsan, is there any reason we should restrict adding opensearch plugins in private windows?
Status: UNCONFIRMED → NEW
Ever confirmed: true
Flags: needinfo?(ehsan)
Assignee | ||
Comment 4•11 years ago
|
||
Yes, this was done in bug 577685.
Status: NEW → RESOLVED
Closed: 11 years ago
Flags: needinfo?(ehsan)
Resolution: --- → INVALID
I was unaware of bug 577685. It appears to be in conflict with the final comments of bug 566010. Specifically, other permanent modifications to Firefox are permitted, e.g. bookmarks, add-ons, browser upgrades. Also, I can add a search plugin using javascript from within a web page (e.g. Mycroft). Considering that functionality is permitted, can we re-enable the auto-discovery of search engines in Private Browsing mode?
Status: RESOLVED → REOPENED
Resolution: INVALID → ---
Summary: OpenSearch autodiscovery fails in Private Browsing Mode → OpenSearch autodiscovery disabled in Private Browsing Mode
Assignee | ||
Comment 6•11 years ago
|
||
I don't feel very strongly about this. What about you, Gavin?
Flags: needinfo?(gavin.sharp)
Comment 7•11 years ago
|
||
The restriction does seem pretty arbitrary, and I don't see any reason to differ from the bookmarks behavior, so I think we should remove it.
Flags: needinfo?(gavin.sharp)
Assignee | ||
Comment 8•11 years ago
|
||
Updated•11 years ago
|
Attachment #783201 -
Flags: review?(josh) → review+
Assignee | ||
Comment 9•11 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/ded8c8339686
Comment 10•11 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/ded8c8339686
Status: ASSIGNED → RESOLVED
Closed: 11 years ago → 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 25
You need to log in
before you can comment on or make changes to this bug.
Description
•