1677868 - Rename "Java Main Thread" to "AndroidUI (JVM)"

Status: RESOLVED FIXED

(Core :: Gecko Profiler, task, P2)

Description

[Markus Stange [:mstange]]

The Java stacks in Android profiles come from what's commonly called the "UI thread". Furthermore, as of bug 1661172 we now have an "AndroidUI" thread registered with the regular (C++) profiler. That thread is actually the same thread as the Java thread. So we should give both the same name, and make it clear that they're just displaying different types of stacks.

Comment 1

[Markus Stange [:mstange]]

Attachment: Bug 1677868 - Rename "Java Main Thread" to "AndroidUI (Java)" in the profiler. r=canaltinova

Comment 2

[Markus Stange [:mstange]]

Example profile with patch: https://share.firefox.dev/3kOl3Dj

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/0ecfa354c74c
Rename "Java Main Thread" to "AndroidUI (Java)" in the profiler. r=canaltinova

Comment 4

[Razvan Maries]

Backed out for perma failures.

Push with failure: https://treeherder.mozilla.org/jobs?repo=autoland&resultStatus=pending%2Crunning%2Csuccess%2Ctestfailed%2Cbusted%2Cexception&searchStr=os%2Cx%2C10.14%2Cwebrender%2Copt%2Cmochitests%2Ctest-macosx1014-64-qr%2Fopt-mochitest-browser-chrome-e10s%2Cbc1&revision=e43e56bfbfc6261a2372c3e5071fd29cdc135617&selectedTaskRun=IF8DULzzQwKBM67YebK8oA.0

Log: https://treeherder.mozilla.org/logviewer?job_id=322230379&repo=autoland&lineNumber=1853

Backout: https://hg.mozilla.org/integration/autoland/rev/a8772cfec92ca24a80b373a3eb028de88bc1b11b

Comment 5

[Markus Stange [:mstange]]

The failure is most probably bug 1677396, so I'm relanding just this bug.

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/a7db0b6f568c
Rename "Java Main Thread" to "AndroidUI (Java)" in the profiler. r=canaltinova

Comment 7

[Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)]

https://hg.mozilla.org/mozilla-central/rev/a7db0b6f568c733578309902c402301ff6cee229