Closed Bug 520294 Opened 13 years ago Closed 13 years ago

Search Disabled State is Persistent even after Uninstallation


(Firefox for Android Graveyard :: General, defect)

Fennec 1.1
Not set


(Not tracked)



(Reporter: aakashd, Assigned: vingtetun)


(Whiteboard: [fennecb4testday])


(1 file, 1 obsolete file)

Build Id:

Mozilla/5.0 (X11; U; Linux armv7l; en-US; rv:1.9.2b1pre) Gecko/200910002

Steps to Reproduce:
1. Go to
2. Click on Larry and save the search
3. Go to Options and disable the bugzilla search extension
4. Uninstall bugzilla search
5. go back to and install bugzilla search again
6. Go to the addons manager

Actual Results:
the bugzilla search extension is still disabled

Expected Results:
I'd expect the search extension to be set to default values once its installed again. A user is not going to re-install an extension and expect it to be uninstalled.
tracking-fennec: --- → ?
Whiteboard: [fennecb4testday]
Does it work if you close Fennec and restart?
(In reply to comment #1)
> Does it work if you close Fennec and restart?

As I'd be more likely to agree with the blocking-fennec? if it failed after a restart.
Attached patch wip (obsolete) — Splinter Review
Correcting the problem by removing the disabled state once we uninstall a search engine. My question is more, should we remove the disable state once we _uninstall_ a search engine, or once we _install_ a new one?
The second sounds better for me.

Also we have a little bug with the icon of the search engine when we remove it/add it quickly: it is too fast an the icon is not fully loaded before recreating the list which lead to a row in the addons panel with a missing icon.

These code parts are related to the XXX in the code (browser.js#1401/#1407), should I consider patching them also in this bug or open a new one? Again the second sounds more logical to me.
The core issue is bug 341833 - the search service persists "disabled" state across installs/uninstalls. That patch seems like a reasonable workaround.

You should file the other problem as a separate bug.
Comment on attachment 405668 [details] [diff] [review]

Add a comment? Something like:

Make sure the engine isn't hidden before removing it, to make sure it's visible if the user later re-adds it (works around bug 341833)
Attachment #405668 - Flags: review?(mark.finkle) → review+
Attached patch PatchSplinter Review
Adding gavin's comment. I don't think I can made something better than it.
Assignee: nobody → 21
Attachment #405668 - Attachment is obsolete: true
Closed: 13 years ago
Resolution: --- → FIXED
Target Milestone: --- → B5
verified FIXED on builds:

Mozilla/5.0 (X11; U; Linux armv7l; en-US; rv:1.9.2b1pre) Gecko/20091021


Mozilla/5.0 (X11; U; Linux armv6l; en-US; rv:1.9.3a1pre) Gecko/20091021
tracking-fennec: ? → ---
You need to log in before you can comment on or make changes to this bug.