Page MenuHomePhabricator

CL support for rolling out Performance Inspector
Closed, InvalidPublic


1. What is the problem

The Performance team created an extension called Performance Inspector[1]. This extension, when enabled, allows the editor (or viewer) of a page to get details about a number of different factors that may influence the performance of the page, such as the number of javascript and CSS files being pulled in, where ResourceLoader modules were loaded from, image sizes, and timing and resource usage from the browser parser. T117411 has additional details about the motivations for the creation of this extension.

Currently the extension is enabled in Beta, and can be activated from the User Preferences/Beta Features tab. When enabled, it appears as an additional link in the "Tools" section of the links panel:

We would like to roll this feature out to Production, but we aren't sure that the current implementation is ideal. In particular, we're wondering whether the addition of a link to the Tools section of the links panel is actually the ideal way to expose this to users, or if there's a better option.

2. What does success of this task look like?

In our view, success is either

  1. Confirmation that the current approach is fine; or
  2. Clarity about what a better approach would be.

Some help understanding any communication/notification expectations would also be appreciated.

3. Is there any goal/program/etc related to this request

T117411 as noted above. This is driven by the Performance team, in order to enable work that community members have been tackling independently.

4. What is your expected timeline? Is there any hard deadline?

There is no hard deadline. This extension has been in Beta for about 18 months now, so clearly there isn't a big rush to get this done.

With that said, work that's complete but not actually deployed is just tech debt, so it would be nice to close the loop.


Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptFeb 1 2018, 8:14 PM
Imarlier moved this task from Inbox to Radar on the Performance-Team board.Feb 5 2018, 9:03 PM
Imarlier edited projects, added Performance-Team (Radar); removed Performance-Team.
Elitre added a subscriber: Elitre.Feb 6 2018, 11:18 AM

Thanks for filing the task. Will triage ASAP.

Elitre assigned this task to Quiddity.Feb 6 2018, 5:44 PM
Quiddity added a comment.EditedFeb 7 2018, 4:13 AM

Hi. The main question is "how to invoke it, i.e. where to place the link that launches it, given that it has to appear in all pages directly" - We talked a lot about these aspects in T129322: How to invoke the performance inspector?. Rather than trying to summarize all of that as part of the discussion here, I'll boldly re-open that task and we can continue refining our understanding there?

Beyond that, here are the top questions I have:

  • Might need a built-in "Info/help" link. This probably depends on the "how to invoke" decision - i.e. if this ends up staying where it is and thus becoming a prominent part of the interface for everyone, then random people will click the link in the sidebar out of curiosity - thus we would need links to the info-page built in to the results overlay (e.g. opposite the cancel button).
  • What is the preferred end-user feedback location? The Help page ( points to the Extension_talk page. That's probably fine, but might lead to bug-reports about random "I thought it was a performance problem" issues being left there. This will also partially depend on the "how to invoke" decision (e.g. if we make it a normal preference, then it will probably need a feature-info link, like mediaviewer does.)
  • The help page might benefit from a screenshot or two, to show the expected/normal results. Otherwise people who are new to it, will have no context to compare the results to, in order to determine whether something is out of the ordinary. E.g. I assume these are normal, but I don't know:
Quiddity triaged this task as Medium priority.Feb 7 2018, 4:13 AM
Quiddity raised the priority of this task from Medium to Needs Triage.
Quiddity triaged this task as High priority.
Quiddity moved this task from Backlog to Jan-Mar-2018 on the Community-Relations-Support board.
Quiddity added a subscriber: Peter.

@Imarlier and @Peter there are a few open questions above. Please add your replies sometime this week. :-)

I've also just updated various bits of
I'll add a note to next week's Tech/News about the upcoming release, in a bit.

(Other ticket now tagged)

Quiddity lowered the priority of this task from High to Medium.Jun 29 2018, 11:46 PM

Note, I'm still available to help with this, but waiting to hear updates on T117411#4304550 et al.

Quiddity removed Quiddity as the assignee of this task.Jul 9 2018, 8:32 PM
Quiddity added a subscriber: Quiddity.

Unassigned from self, but I'm still subscribed. Let me know when work resumes. :-)

Imarlier closed this task as Invalid.Jul 9 2018, 8:34 PM

Thanks Nick. After further consideration, we've changed the way that we're intending to do this, so this task is no longer relevant. Sorry I didn't close it out earlier.

Elitre raised the priority of this task from Medium to High.Jul 13 2018, 3:46 PM