comment: # (Read https://github.com/mozilla-mobile/fenix#i-want-

In A-C there are some search templates that use http it would be better to use https where possible. The work here would be to verify that the engine accepts queries on the https protocol and then create a pr which changes the http call to https

https://github.com/mozilla-mobile/android-components/search?q=%22template%3D%5C%22http%22&unscoped_q=%22template%3D%5C%22http%22

SearchSuggestionsPromptView is not always displayed during tests as seen in screenshots, which causes both the enable and disable tests to fail.

Test Failure

t =    42.16s Assertion Failure: SearchSuggestionsTest.swift:89: Failed to find "SearchSuggestionsPromptView" Other after 30 seconds.
t =    42.19s Assertion Failure: BaseTestCase.swift:53: Asynchronous wait failed: Exceeded ti

We're having a mix of named function syntax (old style) and non-named function syntax (new style) in our internal JavaScript.

The old style was mostly there for JS stack traces before they were as nice as what we have now. Since that's solved, having named functions has become unnecessary and will just take up JS data space when processing.

We should convert all named function syntax in JSO