Closed Bug 1413508 Opened 3 years ago Closed 3 years ago

1.81 - 2.52% damp (linux64, osx-10-10, windows10-64, windows7-32) regression on push 48ad9a270454 (Tue Oct 31 2017)


(DevTools :: Console, defect)

Not set


(Not tracked)



(Reporter: igoldan, Unassigned)



(Keywords: perf, regression, talos-regression)

Talos has detected a Firefox performance regression from push:

As author of one of the patches included in that push, we need your help to address this regression.


  3%  damp summary linux64 pgo e10s     241.77 -> 247.87
  2%  damp summary osx-10-10 opt e10s   303.13 -> 310.57
  2%  damp summary windows7-32 opt e10s 278.39 -> 284.65
  2%  damp summary linux64 opt e10s     278.84 -> 284.64
  2%  damp summary windows10-64 opt e10s263.00 -> 267.75

You can find links to graphs and comparison views for each of the above tests at:

On the page above you can see an alert for each affected platform as well as a link to a graph showing the history of scores for this test. There is also a link to a treeherder page showing the Talos jobs in a pushlog format.

To learn more about the regressing test(s), please see:

For information on reproducing and debugging the regression, either on try or locally, see:

*** Please let us know your plans within 3 business days, or the offending patch(es) will be backed out! ***

Our wiki page outlines the common responses and expectations:
Component: Untriaged → Developer Tools: Console
:nchevobbe These perf regressions were expected, given the new DAMP test you added.

I would have resolved this as WONTFIX, but I noticed the subtests from Linux.
A 4% regression on damp test was picked up on the OPT build.
Multiple subtests regressions were noticed on the PGO build, via the damp complicated.* and damp simple.* tests, though some here may be related to other changesets. I will do some extra backfilling, to be more precise.

Please take a look over the subtests and state whehter we can address these perf regressions.

To view the subtests comparison, hover over the "damp summary linux64 opt e10s" alert then press the "subtests" link.
Flags: needinfo?(nchevobbe)
Hello Ionuț,

I checked multiple times, and the changes from that patch are never called when the debugger open.
They are related to the console only and I do think the regression we see here is more a result of 
the new DAMP test.
I applied the patch suggested in (which change how damp run tests), and I can't see any regression in subtests at the moment (not done though).

Flags: needinfo?(nchevobbe)
(In reply to Nicolas Chevobbe [:nchevobbe] from comment #2)
> I applied the patch suggested in
> (which change how
> damp run tests), and I can't see any regression in subtests at the moment
> (not done though).
Results have arrived. I don't see any regressions either. I guess we can close this now as WONTFIX.
Closed: 3 years ago
Resolution: --- → WONTFIX
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.