Closed Bug 1790906 Opened 2 years ago Closed 4 months ago

Identify cause of temporary 262ms improvement to Firefox Focus start up & try to ship it to users

Categories

(Core :: Performance, task, P2)

task

Tracking

()

RESOLVED INVALID
Performance Impact high

People

(Reporter: mcomella, Assigned: alexical)

References

(Blocks 2 open bugs)

Details

Attachments

(3 files)

In our Nightly Firefox Focus cold VIEW start up test (i.e. measuring process launch to navigation start for an app link), we saw an improvement of 262ms on August 11 and a regression back to where we were on August 19. In Fenix Nightly, we noticed a similar trend: 121ms regression by August 11 undone by August 19.

Due to the similarity, we suspect the root cause is in platform code. Our geckoview_example cold ContentfulSpeedIndex test (i.e. the test above plus a page load) on amazon shows something similar: until August 9, we see slow outliers above 1800ms which stops until Aug 21, backing up our conclusion.

The goal of this bug is to identify the root cause of the improvement and determine if we can ship that improvement to users permanently.

I put this in Core::Performance because I'm not sure which component it should go into.

Performance recording (profile)

Profile URL: Haven't taken

System configuration:

  • Fenix & focus tests are taken on my Moto G5
  • geckoview_example tests ran on A51 devices in CI

Sadly the Performance Triage Calculator isn't particularly appropriate for determining an impact for this type of issue. But purely based on the significance of this improvement I'm going to rate this as high for now.

Performance Impact: ? → high

Next step is to run this is geckoview_example, as well as collecting profiles

Assignee: nobody → dothayer
Severity: -- → S3
Priority: -- → P3

The severity field for this bug is set to S3. However, the Performance Impact field flags this bug as having a high impact on the performance.
:dthayer, could you consider increasing the severity of this performance-impacting bug? Alternatively, if you think the performance impact is lower than previously assessed, could you request a re-triage from the performance team by setting the Performance Impact flag to ??

For more information, please visit BugBot documentation.

Flags: needinfo?(dothayer)
Blocks: perf-android
Blocks: 1894804
Severity: S3 → S2
Priority: P3 → P2

I looked deeper into this a while back and it looked overwhelmingly likely that this was a red herring. I don't have my notes from then but I still think we should just close this.

Flags: needinfo?(dothayer)

Closing this bug based on comment 6 - if we find reasons to investigate this in the future, please open a new bug.

Status: NEW → RESOLVED
Closed: 4 months ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: