1849918 - Remove unnecessary send of search engine state on SearchService initialisation from ContentSearchParent

Status: RESOLVED FIXED

(Firefox :: Search, task, P2)

Description

[Mark Banner (:standard8)]

ContentSearchService currently listens to the browser-search-service notification for init-complete.

When it gets a notification it gets the current search engine state and sends that to the content process.

This is unnecessary for a few reasons:

• The address bar hand-off code does not use the state at all.
• The legacy in-content search bar will retrieve the search engines when it initialises from the parent process.
  • During the retrieve, the parent process will already wait for the search service to finish initialisation before actioning a GetState.

Comment 1

[Mark Banner (:standard8)]

Attachment: Bug 1849918 - Remove unnecessary send of search engine state on SearchService initialisation from ContentSearchParent. r?#search-reviewers!

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/f1dbd57edb26
Remove unnecessary send of search engine state on SearchService initialisation from ContentSearchParent. r=search-reviewers,mcheang

Comment 3

[Natalia Csoregi [:nataliaCs]]

https://hg.mozilla.org/mozilla-central/rev/f1dbd57edb26