Closed Bug 1263983 Opened 6 years ago Closed 1 year ago

Investigate defaultSearch `null` values in core ping


(Firefox for Android Graveyard :: General, defect)

Not set


(Not tracked)



(Reporter: mcomella, Unassigned)



Continuation of bug 1249288. We expect nulls for some set of conditions (see [1]) but it's not clear if we're getting more than we should expect (~1%). The search engine retrieval code is a bit complex, which is why we haven't looked into this sooner.

See and subsequent comments.

fwiw, if the numbers are insignificant, for now I think it's better for me to prioritize finishing the telemetry work rather than nailing this down.

Also, Sebastian or Margaret might have better context on this code and when the search engines are null so it could be more efficient for them to look into the issue.
We're going to validate this against release and if the numbers don't look too bad, we'll close this.

fwiw, the code to retrieve the search engine is very difficult to follow – it grabs the engine asynchronously, sometimes getting it from over the air, making it hard to figure out where the null might be coming from which is why I'm inclined to sweep it under the rug if the numbers are insignificant. (Practically, I assume it's null when the over-the-air search engine has come in in time, but the docs say we can return null in many places so...)

NI Georg since we're waiting on release data validation.
Flags: needinfo?(gfritzsche)
Blocks: 1251614
No longer blocks: core-ping
Flags: needinfo?(gfritzsche)
We have completed our launch of our new Firefox on Android. The development of the new versions use GitHub for issue tracking. If the bug report still reproduces in a current version of [Firefox on Android nightly]( an issue can be reported at the [Fenix GitHub project]( If you want to discuss your report please use [Mozilla's chat]( server and join the [#fenix]( channel.
Closed: 1 year ago
Resolution: --- → INCOMPLETE
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.