1492638 - JavaScript Error: "1537398183677 Toolkit.Telemetry ERROR TelemetrySession::ReferenceError: PrioEncoder is not defined" {file: "resource://gre/modules/Log.jsm" line: 684}

Status: RESOLVED WORKSFORME

(Toolkit :: Telemetry, defect, P5)

Description

[Treeherder Bug Filer]

Filed by: mozilla [at] noorenberghe.ca

https://treeherder.mozilla.org/logviewer.html#?job_id=200286016&repo=try

https://queue.taskcluster.net/v1/task/cv38SInWRnqi-CCYPygcpw/runs/0/artifacts/public/logs/live_backing.log

Comment 1

[Alessio Placitelli [:Dexter]]

Hi Robert, this looks related to the Prio implementation that landed in bug 1465251. Any chance you could take a look at this?

Comment 2

[Robert Helmer [:rhelmer]]

(In reply to Alessio Placitelli [:Dexter] from comment #1)
> Hi Robert, this looks related to the Prio implementation that landed in bug
> 1465251. Any chance you could take a look at this?

We've seen this before with artifact try pushes and Windows... currently libprio (and the PrioEncoder chrome-only DOM) is disabled on Windows. I don't know why it fails on try but not on opt+debug builds.

Bug 1489691 is the only thing blocking enabling libprio for Windows, so for the moment would you mind doing your try run with a full build and it should be fixed as soon as that lands? :)

Comment 3

[Intermittent Failures Robot]

3 failures in 2902 pushes (0.001 failures/push) were associated with this bug in the last 7 days.

Repository breakdown:
* try: 3

Platform breakdown:
* windows10-64: 3

For more details, see:
https://treeherder.mozilla.org/intermittent-failures.html#/bugdetails?bug=1492638&startday=2018-09-17&endday=2018-09-23&tree=all

Comment 4

[Robert Helmer [:rhelmer]]

I happened to notice https://searchfox.org/mozilla-central/source/browser/components/payments/test/browser/head.js#364 is in place - this isn't still happening, is it? You should be able to safely remove this since bug 1492940 was fixed.

Comment 5

[Alessio Placitelli [:Dexter]]

(In reply to Robert Helmer [:rhelmer] from comment #4)

I happened to notice https://searchfox.org/mozilla-central/source/browser/components/payments/test/browser/head.js#364 is in place - this isn't still happening, is it? You should be able to safely remove this since bug 1492940 was fixed.

Good point! Thanks for flagging me on this, looks like it isn't happening anymore. 0 failures last week :) Closing this!