Open Bug 1793132 Opened 2 months ago Updated 24 days ago

App links performs packageManager query on main thread on every page load

Categories

(Fenix :: General, defect)

All
Android
defect

Tracking

(Performance Impact:?)

Performance Impact ?

People

(Reporter: csadilek, Unassigned, NeedInfo)

Details

(Keywords: perf:pageload, top50)

Last we checked in January, this takes 8-30ms on every page load on the Moto G5 (via https://github.com/mozilla-mobile/android-components/issues/7416#issuecomment-1016902556).

NI self to set a performance priority or add to triage once the Performance Impact flag is added to the product (bug 1793646).

Flags: needinfo?(michael.l.comella)

The Performance Impact Calculator has determined this bug's performance impact to be high. If you'd like to request re-triage, you can reset the Performance Impact flag to "?" or needinfo the triage sheriff.

Platforms: Android
Page load impact: Some
Websites affected: Major
[x] Bug affects multiple sites

Note: I ran the calculator by myself, outside of triage. My intuition says "Performance impact: High" seems excessive so I'll leave this in the triage queue.

Next steps

Some suggested next steps, since we filed this so long ago:

  • Verify the delay still occurs and verify how long it takes
  • See if the delay decreases over time, i.e. is it only slow for the first page load or is it slow for all of them? We'd expect the Android system to JIT common code so this step checks if it does. Perhaps this isn't as high of a priority if it's only slow once though we should always consider addressing anything that slows down pageload.
  • If we still consider it to be high priority, address it.
Performance Impact: --- → ?
Flags: needinfo?(michael.l.comella)

The severity field is not set for this bug.
:cpeterson, could you have a look please?

For more information, please visit auto_nag documentation.

Flags: needinfo?(cpeterson)
Severity: -- → S3
You need to log in before you can comment on or make changes to this bug.