Skip to content

Conversation

@julienw
Copy link
Contributor

@julienw julienw commented Jan 15, 2025

Updates:

[Nicolas Chevobbe] Adapt Keyboard shortcut dialog in High Contrast Mode. (#5245)
[Nicolas Chevobbe] Fix sidebar-toggle in High Contrast Mode. (#5246)
[Nicolas Chevobbe] Fix timeline selection overlay time / hover line in High Contrast Mode (#5247)
[Zac Spitzer] fix broken link for processed profile format (#5267)
[Greg Tatum] Update the memory allocation docs and add DHAT docs (#5270)
[Markus Stange] Simplify some code related to thread CPU deltas (#5265)
[Greg Tatum] Update dhat convertor to work better with valgrind (#5269)
[Markus Stange] Rename UniqueStringArray to StringTable. (#5283)
[Markus Stange] Use snapshot testing in the symbolicator CLI test. (#5284)
[Markus Stange] Fix two confused upgraders which didn't expect to be run on the serialized format. (#5285)
[Nicolas Chevobbe] Fix timelineSettingsHiddenTracks in High Contrast Mode. (#5250)
[Julien Wajsberg] Fix some test and non-test warnings (#5294)
[Nisarg Jhaveri] Support importing simpleperf trace files from Android Studio (#5212)
[Sean Kim] Add HTTP response status code in the profiler network tab (#5297)
[Markus Stange] Change StringTable API a bit. (#5286)
[Markus Stange] Correctly declare imported simpleperf trace profiles to be of the current version. (#5312)
[Markus Stange] Two small changes (#5313)
[Nazım Can Altınova] Show sample tooltips on sample graph hover (#5298)

Also thanks to our localizers:

en-CA: chutten
es-CL: ravmn
es-CL: ravmn
fr: Théo Chevalier
ia: Melo46
ia: Melo46
sv-SE: Andreas Pettersson
uk: Lobodzets
zh-TW: Olvcpr423

nisargjhaveri and others added 30 commits November 15, 2024 23:58
Co-authored-by: Andreas Pettersson <amvpet@gmail.com>
Make sure the dialog has a border, that kbd elements have a distinct
style and that the close button follows the Firefox HCM guidelines.

Co-authored-by: Nazım Can Altınova <canaltinova@gmail.com>
Make sure the arrow is visible, and fix the hover style

Co-authored-by: Nazım Can Altınova <canaltinova@gmail.com>
#5247)

Co-authored-by: Nazım Can Altınova <canaltinova@gmail.com>
…iler

Co-authored-by: Olvcpr423 <oliverchan86@outlook.com>
…iler

Co-authored-by: chutten <chutten@mozilla.com>
Make it look like a HCM button and actually display the arrow.
Co-authored-by: Lobodzets <Lobodzets@meta.ua>
Co-authored-by: Théo Chevalier <theochevalier@pm.me>
depfu bot and others added 29 commits January 10, 2025 18:15
Fix #5203.

Attempt to add importer for simpleperf trace files. These can be
generated using the `simpleperf report-sample` command or using Profiler
UI in Android Studio.

Here is the file format details:
https://cs.android.com/android/platform/superproject/main/+/main:system/extras/simpleperf/cmd_report_sample.proto
This avoids copies of the underlying array.

It also provides a new "constructor" (StringTable.withBackingArray)
which avoids recreating the map when we need a StringTable for the
same array in different places.
This avoids copies of the underlying array.

It also provides a new "constructor" (StringTable.withBackingArray)
which avoids recreating the map when we need a StringTable for the same
array in different places.
…rent version. (#5312)

This importer uses the Profile type, so it must be maintained to comply
with the types of the latest version.
But if the self-declared version is set to 50, it will go through the
upgraders, which will not work if the profile is already using the
current version.
This saves one level of indentation and feels more straightforward to me these days.
Two small changes in preparation for #5287. No changes in behavior.
…graph

Previously we were always returning the closest sample to the mouse
hover/click location. Which was okay when we were clicking to select the
boxes before. But now we are also showing tooltips on hover which can be
misleading if they were shown when we are not hovering over them. So to
make this experience less confusing, this commit changes the behavior to
only select or hover a specific sample when the mouse is actually on top
of the sample box that we draw.
Previously we were using the exact same algorithm as the activity graph.
Activity graph tests get the value in between this and the next same
time. But for these tests, we actually want the exact sample time.
Fixes #3363.

This was requested by multiple people because sometimes it's not clear
to users what the squares in the sample graph are. For example latest
feedback we got was in #5278. They also said that a tooltip would've
helped.
Updated locales: en-CA, es-CL, fr, ia, sv-SE, uk, zh-TW
@julienw julienw merged commit d879b1b into production Jan 15, 2025
9 of 17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.