[meta] Update Telemetry JS modules to async function & await

RESOLVED FIXED

Status

()

P3
normal
RESOLVED FIXED
2 years ago
10 months ago

People

(Reporter: gfritzsche, Unassigned)

Tracking

({meta})

Trunk
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox54 affected)

Details

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

(Reporter)

Description

2 years ago
We have `async function` and `await` now:
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Statements/async_function

We should be able to switch all of the `Task.async`, `Task.spawn`, `function*` and `yield` uses over to those:
https://dxr.mozilla.org/mozilla-central/search?q=path%3Atoolkit%2Fcomponents%2Ftelemetry%2F+ext%3Ajs+ext%3Ajsm+Task.&redirect=false

The only case i can think of that doesn't migrate trivially is any `DeferredTask` usage, we could just leave that one for now.
(Reporter)

Updated

2 years ago
Depends on: 1344737
(Reporter)

Updated

2 years ago
Depends on: 1344738
(Reporter)

Updated

2 years ago
Depends on: 1344741
(Reporter)

Updated

2 years ago
Depends on: 1344743
(Reporter)

Updated

2 years ago
Depends on: 1344744
(Reporter)

Comment 1

10 months ago
The dependencies here are resolved, closing.
Status: NEW → RESOLVED
Last Resolved: 10 months ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.