search

firefox-devtools / profiler

Firefox Profiler — Web app for Firefox performance analysis
https://profiler.firefox.com
Mozilla Public License 2.0
1.14k stars 372 forks source link

Make innerWindowID columns optional #5006

Open mstange opened 1 month ago

mstange commented 1 month ago

The FrameTable's innerWindowID column is currently required. However it is quite useless for non-Firefox profiles and takes up a lot of space. It would be nice to make it optional.

┆Issue is synchronized with this Jira Task

julienw commented 1 month ago

At one point we tended to make things required so that it would be easier to manage in our codebase. But this is clearly a roadblock for the usage of the profile format outside of Firefox.

I wonder if we could have an external processed format (with optional fields) and an internal processed format (with the optional fields replaced at loading time with sane defaults)...