mhansen / profilerpedia

Catalog of profilers, profile data formats, data format converters, and profile analysis UIs.
https://profilerpedia.markhansen.co.nz/
29 stars 2 forks source link

Firefox's built-in dev tools are probably different to Firefox Profiler -- add them? #14

Open mhansen opened 2 years ago

jryans commented 2 years ago

Although it's not yet enabled on Firefox Release, there is a new version of the Firefox DevTools Performance panel in the works which effectively captures and displays page performance using the same UI as the Firefox Profiler for the whole browser, so probably just one entry is sufficient over here.

mhansen commented 2 years ago

Sorry I'm a little confused, the title of this issue says they're different, and so to add them, but the body of the issue says that just one entry is sufficient (merge them?).

On Fri, 28 Jan 2022 at 23:41, J. Ryan Stinnett @.***> wrote:

Although it's not yet enabled https://bugzilla.mozilla.org/show_bug.cgi?id=1693316 on Firefox Release, there is a new version of the Firefox DevTools Performance panel in the works which effectively captures and displays page performance using the same UI as the Firefox Profiler for the whole browser, so probably just one entry is sufficient over here.

— Reply to this email directly, view it on GitHub https://github.com/mhansen/profilerpedia/issues/14#issuecomment-1024178447, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAAZYOIRZMGT77C5JZHUC7DUYKFHNANCNFSM5HAU7GZA . You are receiving this because you authored the thread.Message ID: @.***>

jryans commented 2 years ago

Well, you opened this issue originally, presumably thinking they are different…?

I am trying to say that a single entry should suffice since there will (very soon) only be one UI (Firefox Profiler), which is used for both single page and entire browser profiling.

mhansen commented 2 years ago

Oh, sorry, my bad. This just appeared in my inbox without the context of the first comment and I thought you opened it. I should have clicked through. Thanks. What you're saying makes sense.