1151915 - Make "saved-session" documentation more explicit

Status: RESOLVED FIXED

(Toolkit :: Telemetry, defect)

Description

[Georg Fritzsche [:gfritzsche]]

Attachment: Improve saved-session documentation

Minor documentation improvement.

Comment 1

[Vladan Djeric (:vladan)]

Comment on attachment 8589168 [details] [diff] [review]
Improve saved-session documentation

Review of attachment 8589168 [details] [diff] [review]:
-----------------------------------------------------------------

::: toolkit/components/telemetry/tests/unit/test_PingAPI.js
@@ +18,5 @@
>    // and will be removed with bug 1149754.
>    yield TelemetryPing.setup();
>  
>    const PING_TYPE = "test-ping-api";
> +  let pings = new Map();

What's the motivation for this change?

@@ +25,2 @@
>    let id = yield TelemetryPing.send(PING_TYPE, {});
> +  

trailing space

Comment 2

[Georg Fritzsche [:gfritzsche]]

(In reply to Vladan Djeric (:vladan) -- please needinfo! from comment #1)
> Comment on attachment 8589168 [details] [diff] [review]
> Improve saved-session documentation
> 
> Review of attachment 8589168 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> ::: toolkit/components/telemetry/tests/unit/test_PingAPI.js
> @@ +18,5 @@
> >    // and will be removed with bug 1149754.
> >    yield TelemetryPing.setup();
> >  
> >    const PING_TYPE = "test-ping-api";
> > +  let pings = new Map();
> 
> What's the motivation for this change?

Sorry about that - the test changes are completely unrelated and from a bad patch refresh.
Removing them before landing.

Comment 3

[Georg Fritzsche [:gfritzsche]]

Attachment: Improve saved-session documentation

Comment 4

[Georg Fritzsche [:gfritzsche]]

https://hg.mozilla.org/integration/fx-team/rev/1f446da826ae

Comment 5

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/1f446da826ae