1595482 - Change name of responsiveness field

Status: RESOLVED FIXED

(Core :: Gecko Profiler, task, P1)

Description

[Randell Jesup [:jesup] (needinfo me)]

This lets the front-end code know whether to create an event delay track.

Comment 1

[Randell Jesup [:jesup] (needinfo me)]

Attachment: Bug 1595482: change "responsiveness" field to "eventDelay" in profiler r=canaltinova

Comment 2

[Nazım Can Altınova [:canova][:canaltinova on phabricator]]

Thanks for the patch. Front-end changes should be merged first to be able to handle the upgrading. See here: https://github.com/firefox-devtools/profiler/pull/2303

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/70738ac7ba9b
change "responsiveness" field to "eventDelay" in profiler r=canaltinova

Comment 4

[Narcis Beleuzu [:NarcisB]]

Backed out for xpcshell failures on test_responsiveness.js

Backout link: https://hg.mozilla.org/integration/autoland/rev/76886efb06b40b8b9934f15e34150335c46c9c82
Log link: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=276029199&repo=autoland&lineNumber=2693

Comment 5

[Nazım Can Altınova [:canova][:canaltinova on phabricator]]

Ah, we have a test for responsiveness here: https://searchfox.org/mozilla-central/rev/cac340f10816707e91c46db6d285f80259420f07/tools/profiler/tests/xpcshell/test_responsiveness.js#26. We should also change that to eventDelay.

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/9afdb269ce04
change "responsiveness" field to "eventDelay" in profiler r=canaltinova

Comment 7

[Natalia Csoregi [:nataliaCs]]

https://hg.mozilla.org/mozilla-central/rev/9afdb269ce04