Closed Bug 484636 Opened 15 years ago Closed 4 years ago

AddSearchProvider() should support data: urls

Categories

(Firefox :: Search, enhancement)

Product:
Firefox
Component:
Search
Platform:
x86
Windows XP
Type:
enhancement
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED WONTFIX

People

(Reporter: timeless, Unassigned)

References

(
URL
)

Details

This search engine isn't supported by Minefield and can't be installed.
data: isn't "blocklisted", removing confusing term from summary. There's a whitelist of allowed schemes.

The only implementation I can find is in mozilla/browser -- what does seamonkey do?

http://hg.mozilla.org/mozilla-central/annotate/ebfe0dc025da/browser/components/sidebar/src/nsSidebar.js#l135
Severity: normal → enhancement
Component: DOM: Mozilla Extensions → Search
Product: Core → Firefox
QA Contact: general → search
Summary: don't blocklist data:, it's not nice → AddSearchProvider() should support data: urls
They have their own nsSidebar:
http://mxr.mozilla.org/comm-central/source/suite/common/sidebar/nsSidebar.js

and just don't support this method. Bug 518929 is about fixing this mess.
I would like to implement this, would that patch be accepted?

Per the recent intent to unship, AddSearchProvider will be going away in bug 1632447/1632448, hence closing as wontfix.

Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.