Closed Bug 1695588 Opened 4 years ago Closed 4 years ago

When a new alias is set, only notify the engine has changed if the alias is different

Categories

(Firefox :: Search, defect, P2)

defect
Points:
2

Tracking

()

RESOLVED FIXED
88 Branch
Iteration:
88.1 - Feb 22 - Mar 7
Tracking Status
firefox88 --- fixed

People

(Reporter: standard8, Assigned: standard8)

References

Details

Attachments

(1 file)

Currently if we attempt to set the alias of an engine to the same value, we'll notify that the engine has changed.

I don't think we should notify - there's no reason for it, and it means that we get extra notifications when removing an application provided engine that we possibly don't need. It also complicates tests.

It also means we're not rewriting the search settings file if the user edits the alias in preferences, and then presses enter with no changes.

Pushed by mbanner@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/1ebfd2d11f33 When a new search engine alias is set, only notify the engine has changed if the alias is different. r=mak
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → 88 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: