Please consider adding Firefox's BuildID string to Marionette's capabilities object

RESOLVED FIXED in Firefox 67

Status

enhancement
RESOLVED FIXED
4 months ago
4 months ago

People

(Reporter: stephend, Assigned: automatedtester)

Tracking

({perf})

Trunk
mozilla67
Points:
---

Firefox Tracking Flags

(firefox67 fixed)

Details

(Whiteboard: [webpagetest])

Attachments

(1 attachment)

tl;dr - would you please consider/look at including Firefox's BuildID, output as a string, in Marionette's capabilities object?

BuildID variable in application.ini https://searchfox.org/mozilla-central/rev/e00ea598e52bbb35f8c45abf9c2eade17962bb5e/build/application.ini.in#29

WebPageTest, via its wptagent[0], gets it from Marionette's capabilities object[1] as the string browserVersion, with a sample value "67.0a1".

Since Firefox 64, navigator.buildID() no longer works, according to https://developer.mozilla.org/en-US/docs/Web/API/Navigator/buildID, so, even if it were to get it via documented means I've come across, we'd get something like this fixed one: "20181001000000".

Perhaps it could be added as a vendor-prefixed string -- moz along with the others, e.g. https://searchfox.org/mozilla-central/rev/e00ea598e52bbb35f8c45abf9c2eade17962bb5e/testing/marionette/capabilities.js#409-416?

For more context, I'm close to submitting my WebPageTest perf-runs to the Mozilla Telemetry Data Pipeline (https://github.com/mozilla/wpt-api/pull/236), and an accurate browserVersion/browser_version is key (no Python-dict pun intended, but I'll take it).

Here's what we have (will have): https://github.com/mozilla/wpt-api/pull/236/files#diff-6d5e2d2375d56dbb4c662b976106321dR65

(Also helpful would be the SHA commit hash for mozilla-central for each build, available via about:buildconfig, and has a value like: https://hg.mozilla.org/mozilla-central/rev/de51545099a617602be187d1c0f68ff2a87d6fb2 but that's totally another bug, another day.)

[0] https://github.com/WPO-Foundation/wptagent/blob/f5226f6fe978f6893eedb2c2552bdefc032405ed/internal/firefox.py#L149-L150
[1] https://searchfox.org/mozilla-central/rev/e00ea598e52bbb35f8c45abf9c2eade17962bb5e/testing/marionette/capabilities.js#388-397

Assignee

Updated

4 months ago
Assignee: nobody → dburns

(In reply to Stephen Donner [:stephend] from comment #0)

BuildID variable in application.ini https://searchfox.org/mozilla-central/rev/e00ea598e52bbb35f8c45abf9c2eade17962bb5e/build/application.ini.in#29

Both buildid and the commit id we should be able to query via Services or a different JSM in Marionette server. That will be safer than trying to access it via that before-mentioned file.

Assignee

Comment 2

4 months ago

Add buildID to returned list of capabilities when a new session is started with
Marionette. This new capability will live in moz:buildID.

Comment 3

4 months ago
Pushed by dburns@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/9092cd81ee1b
Add buildID to returned capabilities list for Marionette; r=ato

Comment 4

4 months ago
bugherder
Status: NEW → RESOLVED
Closed: 4 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla67
You need to log in before you can comment on or make changes to this bug.