Closed Bug 987333 Opened 6 years ago Closed 5 years ago

Document BrowserUITelemetry JSON blob format

Categories

(Firefox :: Toolbars and Customization, defect)

x86_64
All
defect
Not set

Tracking

()

RESOLVED FIXED
Firefox 37

People

(Reporter: mconley, Assigned: Unfocused)

References

Details

(Whiteboard: [Australis:P-])

Attachments

(1 file, 1 obsolete file)

This has come up a number of times in various discussions, and most recently when talking to Ilana Segall, who is working on processing the BrowserUITelemetry stuff we're gathering for Australis.

We really should document the format we're using here to make it easier to interpret what we're recording, what we're _not_ recording, and how we're doing it.
Mike, I just landed some of this with bug 991757. Can you have a look at it and see what is still missing / could be improved? I think it could probably do (at the least) with a better explanation about the buckets, which I will happily admit I don't fully understand myself... :-)
Flags: needinfo?(mconley)
Whoa! We're doing Sphinx now? THANK GOODNESS! :) That's awesome.

The docs look good - though we might want to add something about buckets. Blair, do you think you'd have a few minutes to extend that documentation with the bucket stuff?
Flags: needinfo?(mconley) → needinfo?(bmcbride)
Assignee: nobody → bmcbride
Status: NEW → ASSIGNED
Attachment #8542439 - Flags: review?(mconley)
/r/1797 - Bug 987333 - Add documentation for UITour and buckets to BrowserUITelemetry docs.

Pull down this commit:

hg pull review -r 20d82a2b517fdff78445865617e8c219610e6aa7
Oh, and it seems the existing UITelemetry document wasn't ever included in the generated docs:
https://gecko.readthedocs.org/en/latest/browser/browser/index.html
So I've fixed that too.
Flags: needinfo?(bmcbride)
Attachment #8542439 - Flags: review?(mconley) → review+
https://reviewboard.mozilla.org/r/1795/#review1243

So good to have documentation. Thanks for this, Blair. :)

::: browser/docs/UITelemetry.rst
(Diff revision 1)
> +- ``bucket_<NAME>|<INTERVAL>`` - Expiring buckets, which similar to a countdown timer. The ``<INTERNAL>`` in the bucket ID describes the time interval the recorded event happened in. The intervals are ``1m`` (one minute), ``3m`` (three minutes), ``10m`` (ten minutes), and ``1h`` (one hour). After one hour, the ``__DEFAULT__`` bucket is automatically used again.

"which similar" -> "which are similar",
"<INTERNAL>" -> "<INTERVAL>"

::: browser/docs/UITelemetry.rst
(Diff revision 1)
> -The UI Tour has its own section in the UI Telemetry output, outside of the ``toolbars`` section.
> +The UITour API provides ways for pages on trusted domains to safely interact with the browser UI and request it perform ations such as opening menus and showing highlights over the browser chrome - for the purposes of interactive tours. We track some usage of this API via the ``UITour`` object in the UI Telemetry output.

"request it perform" -> "request it to perform"
https://hg.mozilla.org/mozilla-central/rev/9477cb31bf80
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Flags: in-testsuite-
Resolution: --- → FIXED
Target Milestone: --- → Firefox 37
Attachment #8542439 - Attachment is obsolete: true
Attachment #8618101 - Flags: review+
You need to log in before you can comment on or make changes to this bug.