822703 - s/ShutdownTime/TelemetryData/ where appropriate

Status: RESOLVED FIXED

(Toolkit :: Telemetry, defect)

Description

[Rafael Ávila de Espíndola (:espindola) (not reading bugmail)]

Attachment: rename

In bug 814765 we will be reading more than just the shutdown time, but the extra data (the late write stacks) can reuse exactly the same logic for when to read and how to cache as the shutdown time.

This patch goes on top of 822695 and just renames some fields and methods to reduce the noise on the patch for 814765.

Comment 1

[Rafael Ávila de Espíndola (:espindola) (not reading bugmail)]

Attachment: In which I am reminded that doing a build is not enough for JS :-(

Comment 2

[Vladan Djeric (:vladan)]

Comment on attachment 693539 [details] [diff] [review]
In which I am reminded that doing a build is not enough for JS :-(

As long as we're on the topic of renaming vars, what do you think of changing "Read" to "Fetch" in variable & function names? It seems more clear that we are doing I/O. Just an idea

Comment 3

[Rafael Ávila de Espíndola (:espindola) (not reading bugmail)]

 https://hg.mozilla.org/integration/mozilla-inbound/rev/9331979adc34

Comment 4

[Ed Morley [:emorley]]

https://hg.mozilla.org/mozilla-central/rev/9331979adc34