profiler icon indicating copy to clipboard operation
profiler copied to clipboard

Make innerWindowID columns optional

Open mstange opened this issue 1 year ago • 2 comments

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

mstange avatar May 21 '24 17:05 mstange

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)...

julienw avatar May 23 '24 08:05 julienw

Once we remove the active tab view (#4997), we can just remove this column entirely, see #5443.

mstange avatar May 02 '25 19:05 mstange