Closed Bug 1219218 Opened 9 years ago Closed 9 years ago

Support add/remove Search provider via addon

Categories

(Firefox OS Graveyard :: Gaia::Browser, defect)

ARM
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: gasolin, Unassigned)

Details

(Keywords: feature, foxfood)

Attachments

(1 file)

As a developer I'd like to add/remove search provider (wikipedia, google translate..) through addon.


In experiment, shared/js/SearchProvider.js will load json file and save the result in SearchProvider providers variable and mozSettings. Addon could change mozSettings but can't change the SearchProvider `providers` variable.

To make Search Providers list properly, we could add addProvider/removeProvider method in SearchProvider.js
Summary: Support add/remove Search provider as addon → Support add/remove Search provider via addon
Comment on attachment 8679985 [details] [review]
[gaia] gasolin:master > mozilla-b2g:master

Dale or Ben, 

I'd like to hear your feedback if its the right approach to add Search Provider. Then I'll add test cases and make sure it works with real addon (the addon needs update mozSettings search providers and call SearchProvider.addProvider on search app to make it works).

I found SearchProvider.ready() will return readyPromise early when it ran once.
It might possible to just add a `forceReload` parameter in SearchProvider.ready() to force reload addons from settings.
Attachment #8679985 - Flags: feedback?(dale)
Attachment #8679985 - Flags: feedback?(bfrancis)
Comment on attachment 8679985 [details] [review]
[gaia] gasolin:master > mozilla-b2g:master

Hi Fred,

I think it's fine if people want to and are able to create addons which add additional search engines if that's possible without modifying the search app too much.

However, we have plans to add native support for adding additional search engines to the search app and UX are currently working on some ideas for how this might work.

I agree this is a crucial piece of functionality but I think we can add general support for any Open Search provider without requiring an addon per provider.

Thanks
Attachment #8679985 - Flags: feedback?(bfrancis) → feedback-
Comment on attachment 8679985 [details] [review]
[gaia] gasolin:master > mozilla-b2g:master

It's great to hear we are trying to address this issue. 

Will set this as wontfix if we can provide a more concrete plan.
Attachment #8679985 - Flags: feedback?(dale)
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: