Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Understand Fenix developer needs #2475

Closed
mstange opened this issue Mar 25, 2020 · 3 comments
Closed

Understand Fenix developer needs #2475

mstange opened this issue Mar 25, 2020 · 3 comments
Labels
android Android, GeckoView, or Fenix

Comments

@mstange
Copy link
Contributor

mstange commented Mar 25, 2020

The Fenix team is interested in using the Firefox profiler more. However, as I understand it, they aren't necessarily interested in the Gecko data, they're mostly interested in using the UI, and using it to understand regressions in automation perf tests. The data sources would be Java/Kotlin stacks and manual marker instrumentation in the Java/Kotlin code.

We should find out what we can do to make their lives easier.

The following questions come to mind:

  • What automated performance tests does Fenix use? Which test harnesses do these run in? Which of them support Gecko profiling already? Which of them can capture Android traces? Which of them do we want to get profiles from?
  • What are examples of manual instrumentation / markers that the Fenix team is thinking about adding?
  • How much Gecko data is needed? Do these profiles require having both Java/Kotlin markers and Gecko things on the same timeline? Or should we treat this as an entirely new data source?
  • Apparently nimbledroid automatically generated some kind of profiling data. What type of profile did it generate (example profile?)? What needs to be done to generate similar data in our non-nimbledroid test harnesses?
@digitarald
Copy link
Contributor

@gregtatum gregtatum added the android Android, GeckoView, or Fenix label Apr 21, 2020
@mstange
Copy link
Contributor Author

mstange commented Jun 8, 2022

This issue is quite outdated now. I don't think we've answered the questions above, but things have changed sufficiently in the meantime that you'd want to take a fresh look at this anyway. So I'm closing this issue.

@mstange mstange closed this as completed Jun 8, 2022
@mstange
Copy link
Contributor Author

mstange commented Jun 8, 2022

Ah, we actually have answered most of those questions in the bug that Harald linked to.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
android Android, GeckoView, or Fenix
Projects
None yet
Development

No branches or pull requests

3 participants