Closed Bug 1062397 Opened 10 years ago Closed 10 years ago

Add support for MozParam url parameters in search plugin parser

Categories

(Firefox for Android Graveyard :: Search Activity, defect)

Product:
Firefox for Android Graveyard
Component:
Search Activity
Platform:
All
Android
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED WONTFIX

People

(Reporter: Margaret, Assigned: Margaret)

References

Details

Right now we just support name/value in <Param> tags:
http://mxr.mozilla.org/mozilla-central/source/mobile/android/search/java/org/mozilla/search/providers/SearchEngine.java#113

We'll need to support <MozParam> tags as well.

mconnor, are there some bits that are more important than others, or do we need everything? The search activity is a bit different than Firefox, since the user can only perform searches with the default engine, so maybe we can cheat and just hard-code some of those parameter values in.

Also, do we want to add anything to indicate that this search came from the search activity?
Flags: needinfo?(mconnor)
Actually, it looks like we already support all the regular expected parameters. We would just have to add support for MozParam if we need it.
Summary: Add support for additional url parameters in search plugin parser → Add support for MozParam url parameters in search plugin parser
Priority: P1 → --
It looks like we don't really need MozParam support for any of the search plugins in the search activity.

I'm going to close this bug, we can open new bugs as we find specific things we need to support.
Status: NEW → RESOLVED
Closed: 10 years ago
Flags: needinfo?(mconnor)
Resolution: --- → WONTFIX
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.