940808 - Move first time search engine options loading from browser to FTU

Status: RESOLVED WONTFIX

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

Description

[Kyle Machulis [:qdot] [:kmachulis] (INACTIVE)]

Currently, we load search engines when the browser app is loaded for the first time. We need to move these to the system app

Comment 1

[Kyle Machulis [:qdot] [:kmachulis] (INACTIVE)]

The search engines are stores in an object store in the browser app indexeddb, so we'll also need to figure out where these go. A new datastore?

Comment 2

[Gregor Wagner [:gwagner]]

I think we talked about storing a small amount of data in plain json files before.

Comment 3

[Kyle Machulis [:qdot] [:kmachulis] (INACTIVE)]

mhenretty pointed out using  https://developer.mozilla.org/en-US/docs/JavaScript_OS.File for possible storage of the json block

Comment 4

[Ben Francis [:benfrancis]]

(In reply to Kyle Machulis [:kmachulis] [:qdot] from comment #0)
> Currently, we load search engines when the browser app is loaded for the
> first time. We need to move these to the system app

The list of search engines should eventually be stored in Rocketbar settings, the design of which is still under discussion.

Why does this block bug 938171 ?

Comment 5

[Kyle Machulis [:qdot] [:kmachulis] (INACTIVE)]

I was just linking in everything around bookmarks to that bug, and it looked like a bookmark issue at the time. Unlinked.

Comment 6

[Kyle Machulis [:qdot] [:kmachulis] (INACTIVE)]

Currently blocked on discussions with product. ni?'ing pdol for fill-in once this is resolved.

Comment 7

[Peter Dolanjski [:pdol]]

(In reply to Kyle Machulis [:kmachulis] [:qdot] from comment #6)
> Currently blocked on discussions with product. ni?'ing pdol for fill-in once
> this is resolved.

Kyle, we discussed this at length and in the release that includes Rocketbar, we won't offer the ability to configure other search engines (though it will likely be there in the future).  Given that, what is the best course of action to deprecate this functionality for the time being?

Comment 8

[Kyle Machulis [:qdot] [:kmachulis] (INACTIVE)]

Seeing this is a feature of the browser, which we'll now have around 'til 1.5, I can just leave it there and it'll be removed when the app itself is removed. Default search engine is stored in the browser's DB, so it's not affecting the rest of the system anyways. I'll go ahead and resolve this as wontfix now. If we want to actually remove this feature from the browser for 1.4, we should make another bug, but I'll leave that up to you.