1599132 - isTracker function sometimes fails

Status: NEW

(Toolkit :: Performance Monitoring, defect, P4)

Description

[Erica Wright [:ewright]]

I'm looking at the isTracker function, as I'm implementing some similar functionality. I noticed that the first time isTracker runs it will be incorrect, as asyncClassifyLocalWithFeatures is async, and the tracker will not yet be added to the list.

https://searchfox.org/mozilla-central/source/toolkit/components/aboutperformance/content/aboutPerformance.js#316,322,327

Bug 1527658 added asyncClassifyLocalWithFeatures.

Comment 1

[Erica Wright [:ewright]]

ni: dimi since you implemented Bug 1527658

Comment 2

[Dimi Lee [:dimi]]

I think this is how it was designed in the first place (Bug 1477677).
In Bug 1527658, I change the old API to a new one, but both of the APIs are asynchronous so the behavior should be the same.

I guess we will need some kind of "refresh" mechanism to update UI when we receive the callback?

Comment 3

[BugBot [:suhaib / :marco/ :calixte]]

The priority flag is not set for this bug.
:tarek, could you have a look please?

For more information, please visit auto_nag documentation.