1693314 - Enable new Performance panel by default on DevEdition

Status: RESOLVED FIXED

(DevTools :: Performance Tools (Profiler/Timeline), task, P3)

Description

[Jan Honza Odvarko [:Honza] (always need-info? me)]

The new Performance panel is enabled in Nightly be default (bug 1668218, Fx86). Let's wait for one cycle and enable it in 87

Honza

Comment 1

[Pascal Chevrel:pascalc (PTO until April 26)]

Jan, is that targetting 89 now or a later release?

Comment 2

[Jan Honza Odvarko [:Honza] (always need-info? me)]

It should be enabled in 89.

Nicolas, you were enabling the new Performance panel in Nightly, could you please do it for Beta now, thank you!

Honza

Comment 3

[Nicolas Chevobbe [:nchevobbe]]

The performance Team wanted to wait for a bit if I remember correctly. Let's check with Julien

Comment 4

[Julien Wajsberg [:julienw]]

Just checked with Nazim: let's do it :-)
We do have a few known problems but they're being worked on right now. If we get some bad feedbacks we could still rollback later, right?
Thanks for asking!

Comment 5

[Nicolas Chevobbe [:nchevobbe]]

sure, I'll ask for review shortly :)

Comment 6

[Nicolas Chevobbe [:nchevobbe]]

Attachment: Bug 1693314 - [devtools] Enable new Performance panel by default on Beta. r=julienw

Comment 7

[Jan Honza Odvarko [:Honza] (always need-info? me)]

Awesome, thank you Nicolas and Julien for syncing upfront.

Honza

Comment 8

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/c940b088b613
[devtools] Enable new Performance panel by default on Beta. r=julienw

Comment 9

[Andreea Pavel [:apavel]]

https://hg.mozilla.org/mozilla-central/rev/c940b088b613

Comment 10

[Pascal Chevrel:pascalc (PTO until April 26)]

Nicolas, it seems to me that this is not a patch to activate the feature in beta to make it ride the trains but a patch that activates the panel in Dev Edition builds in addition to Nightly builds. Is that the expected outcome?

Comment 11

[Julien Wajsberg [:julienw]]

We don't want that it rides the train yet, so I believe this is the right balance for now.

Comment 12

[Pascal Chevrel:pascalc (PTO until April 26)]

(In reply to Julien Wajsberg [:julienw] from comment #11)

We don't want that it rides the train yet, so I believe this is the right balance for now.

Ok, since this is not going to Firefox beta, but only to DevEdition, I am updating the title to reflect what the patch does.