1173464 - Markers should have tooltips in MarkerDetails to help explain the marker itself

Status: RESOLVED INVALID

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

Description

[Jordan Santell [:jsantell] [@jsantell]]

We have a handful of markers that developers may be familiar with if they've used Chrome's devtools (recalc style, layout, paint), and additional ones that are either Firefox specific (Cycle Collection) or something Chrome tools does not have (Parse HTML?).

We should have a tooltip in the marker details view (probably a [?] maybe?) that gives a 1-2 sentence description of what this marker represents. Optionally a link to MDN with more information.

Comment 1

[Nick Fitzgerald [:fitzgen] [⏰PST; UTC-8]]

I don't think there is value in duplicating what the MDN docs should already say in a better fashion. We should just ensure that each marker type has its own anchor, and link there.

Additionally, we could pop open a Tooltip view with the MDN page instead of opening it in a new tab.

Comment 2

[Jordan Santell [:jsantell] [@jsantell]]

It's not duplicating -- one is a tooltip for a brief description, no different than tooltips describing the columns in call tree views, whereas the docs[0] can go into further detail.

If we want the tooltips to be like inspector's MDN Tooltips, that should be moderately straight forward, and OK as well if we want to display more than just "tooltip" info.

Also wondering if the MDN Tooltip can cache -- it uses nsIXMLHttpRequest now, which may handle caching on its own, but something to consider in offline modes.

[0] https://developer.mozilla.org/en-US/docs/Tools/Performance/Waterfall#Markers

Comment 3

[Victor Porof [:vporof][:vp]]

Triaging. Filter on ADRENOCORTICOTROPIC (yes).

Comment 4

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

This report is related to the old DevTools profiler implementation, which has been removed. The Performance panel is now based on Firefox Profiler at https://profiler.firefox.com/
Closing as invalid