Closed Bug 1543147 Opened 6 years ago Closed 3 years ago

Disable 'Responsiveness' on profiler when creating profiles for raptor

Categories

(Testing :: Raptor, task, P3)

Version 3
task

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: smaug, Unassigned)

References

Details

Because of https://bugzilla.mozilla.org/show_bug.cgi?id=1482240 'Responsiveness' calculations disturb idle detection. Could we disable that for now.

(In reply to Olli Pettay [:smaug] from comment #0)

Because of https://bugzilla.mozilla.org/show_bug.cgi?id=1482240 'Responsiveness' calculations disturb idle detection. Could we disable that for now.

Could you provide details on how to disable these calculations? Also, when you say "for now" does that mean you'll ultimately want these to be re-enabled? Is there value in generating profiles for multiple configurations?

Flags: needinfo?(bugs)

Hopefully mstange could tell how to disable 'Responsiveness'. I know just the profiler UI, and there is a checkbox for responsiveness.
And with "for now" I mean, that until bug 1482240 is fixed in some way.
Right now I don't see much value using 'Responsiveness' during page load ever, since we've moved many
things to use idle queue, and 'Responsiveness' affects to idle detection.

(Responsiveness doesn't also detect input event queue handling, so it doesn't really measure responsiveness but something else)

Flags: needinfo?(bugs) → needinfo?(mstange)

I'm guessing we would just remove 'responsiveness' from GeckoProfiler.start.features [0]

[0] https://searchfox.org/mozilla-central/rev/0376cbf447efa16922c550da3bfd783b916e35d3/testing/raptor/webext/raptor/runner.js#323

That's right.

Flags: needinfo?(mstange)
Type: defect → task
Priority: -- → P3
See Also: → 1545386

Works in browsertime, it doesn't use responsiveness.

Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.