Open Bug 1698671 Opened 3 years ago Updated 3 years ago

DAMP Perf regression in complicated.debugger.reload (50%) and complicated.inspector.reload (80%)

Categories

(DevTools :: General, task, P3)

task

Tracking

(Not tracked)

People

(Reporter: jdescottes, Unassigned)

References

(Blocks 1 open bug)

Details

Alert: https://treeherder.mozilla.org/perfherder/alerts?id=29154&hideDwnToInv=0
Pushlog: https://hg.mozilla.org/integration/autoland/pushloghtml?fromchange=b332567cbbcaa6e2b70bfe5449410e9cfb8b838f&tochange=56fd5a31695e692b48f22092f9276d62222fce17

The pushlog above improved performance in many subtests (panelsInBackground reload,...), but also created regressions in 2 tests:

  • complicated.debugger.reload
  • complicated.inspector.reload

This is probably related to Bug 1695929 and possibly to the removal of the "close" event but it should be verified. We will focus on performance after the bulk of the DevTools Fission changes have landed.

The Bugbug bot thinks this bug should belong to the 'DevTools::Inspector' component, and is moving the bug to that component. Please revert this change in case you think the bot is wrong.

Component: General → Inspector

I did a quick local check and all the improvements & regressions from the pushlog are actually linked to Bug 1696920 and not to the DevTools change.

Component: Inspector → General
Depends on: 1696920
No longer depends on: 1695929

I am removing the link to Bug 1696920, I was misguided by artifact builds.

Still bisecting, it looks like it might be either:

Waiting for one last try push on top of 7322919d3bbc5ad8b3c86777ceb492f14654b340: https://treeherder.mozilla.org/jobs?repo=try&revision=9b86fed4c9d10bf03a2d7c5df7ef5bd5622389b0

No longer depends on: 1696920
Depends on: 1695917
You need to log in before you can comment on or make changes to this bug.