Open Bug 1599742 Opened 2 years ago Updated 2 years ago

Review fallback for purpose parameter to EngineURL.getSubmission

Categories

(Firefox :: Search, task, P3)

task
Points:
2

Tracking

()

People

(Reporter: standard8, Unassigned)

References

Details

EngineURL.getSubmission currently takes a purpose parameter. This falls back to searchbar in a couple of instances:

  1. If the parameter is not supplied.
  2. If there are no matching purposes in the associated parameters.

From looking at the callers to getSubmission, then apart from the mobile code, it looks like everyone is supplying a purpose.

As the mobile code may be going away, we should review this and possibly make purpose non-optional and remove the need for the first fallback.

Points: --- → 2
You need to log in before you can comment on or make changes to this bug.