1912071 - Simplify log construction in TelemetryProbesReporter

Status: RESOLVED FIXED

(Core :: Audio/Video, enhancement, P1)

Description

[C.M.Chang[:chunmin]]

1. In nsPrintfCString, we can probably use %lf for logging a double number instead of converting it to %s
2. Append(nsPrintfCString{...}) can be replaced by AppendPrintf(...)

Comment 1

[C.M.Chang[:chunmin]]

Attachment: Bug 1912071 - Remove unnecessary std::to_string in TelemetryProbesReporter

Comment 2

[C.M.Chang[:chunmin]]

Attachment: Bug 1912071 - Replace Append(nsPrintfCString{*}) by AppendPrintf(*)

Depends on D218758

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/23bda0fcbf53
Remove unnecessary std::to_string in TelemetryProbesReporter r=media-playback-reviewers,padenot
https://hg.mozilla.org/integration/autoland/rev/10bf38d18a83
Replace Append(nsPrintfCString{*}) by  AppendPrintf(*) r=media-playback-reviewers,padenot

Comment 4

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/23bda0fcbf53
https://hg.mozilla.org/mozilla-central/rev/10bf38d18a83