Open Bug 1165341 Opened 5 years ago Updated 2 months ago

No "engine-default" is notified when removing a default engine

Categories

(Firefox :: Search, defect, P4)

defect

Tracking

()

Tracking Status
firefox41 --- affected

People

(Reporter: Dexter, Unassigned)

References

(Depends on 1 open bug)

Details

(Whiteboard: [fxsearch] [measurement:client])

In bug 1138503, we've noticed that no "browser-search-engine-modified" topic with "engine-default" data field is notified when a default engine is removed using the search preferences panel.

A notification should probably be dispatched in [1], as the default engine is changed internally.

[1] - https://hg.mozilla.org/mozilla-central/annotate/25c5525a1000/toolkit/components/search/nsSearchService.js#l4368
Florian, is there a particular reason why an "engine-default" notification shouldn't be fired when removing a default engine in nsSearchService?
Flags: needinfo?(florian)
See Also: → 1138503
(In reply to Alessio Placitelli [:Dexter] from comment #1)
> Florian, is there a particular reason why an "engine-default" notification
> shouldn't be fired when removing a default engine in nsSearchService?

I don't see any obvious reason for that. If we change it for the default engine, we should likely also change it for the current engine 4 lines above in the code.
Flags: needinfo?(florian)
Priority: -- → P3
Whiteboard: [fxsearch]
Rank: 35
I'm wondering if the issue here isn't that the engine-removed notification isn't fired. There's a patch in bug 1122618 that (among other things) fixes that engine-removed notification.
Whiteboard: [fxsearch] → [fxsearch] [measurement:client]
(In reply to Florian Quèze [:florian] [:flo] from comment #4)
> I'm wondering if the issue here isn't that the engine-removed notification
> isn't fired. There's a patch in bug 1122618 that (among other things) fixes
> that engine-removed notification.

Mh, doesn't seem so unfortunately :( Commenting out [0] still makes the test fail.

[0] - https://dxr.mozilla.org/mozilla-central/rev/e790bba372f14241addda469a4bdb7ab00786ab3/toolkit/components/telemetry/tests/unit/test_TelemetryEnvironment.js#1122
Priority: P3 → P4
Depends on: 1575649
You need to log in before you can comment on or make changes to this bug.