1889129 - AndroidDecoderModule::GetSupportedMimeTypes() takes over a 100ms during child process startup.

Status: RESOLVED FIXED

(Core :: Audio/Video, defect, P2)

Description

[Jeff Muizelaar [:jrmuizel]]

See this profile from a pixel 6: https://share.firefox.dev/4cE3MbY

Comment 1

[az [:az]]

I can take a look at this when I have some spare cycles and see if I can improve performance.

Comment 2

[az [:az]]

The call which is causing the delay is can be boiled down to:

new MediaCodecList(MediaCodecList.REGULAR_CODECS).getCodecInfos();

... which is an Android API call. I don't know if there's much that can be done to make this call faster. I chatted with jhlin, and apparently the call reads + parses XML files to get codec information. This information is cached at OS level on a per-process basis, which doesn't help us. It'd probably make the most sense to cache this information in a similar manner to 1551890 - 68ms used to GetHWEncoderCapabilities() during Fenix startup, though I might have to roll a different mechanism for doing so.

Comment 3

[Jeff Muizelaar [:jrmuizel]]

Does Chrome use the same API for codec dection? Why do we need to call it on the main thread during startup?

Comment 4

[az [:az]]

It looks like Chrome uses the same API call. We've been using getCodecInfos() before the about:support changes landed, though I profiled a copy of 109 on my phone and it didn't seem to have the same startup delay. Spinning this off to its own thread might make the most sense. I'm going to investigate a bit more and I'll update again here after.

Comment 5

[az [:az]]

Attachment: Bug 1889129 - Cache Android codec support information directly in HardwareCodecCapabilityUtils to improve performance

Comment 6

[az [:az]]

Between this patch and the general fix alwu put together in Bug 1885671, I think we should be in better shape now.

Comment 7

[az [:az]]

Working on an alternative fix for this.

Comment 8

[az [:az]]

Attachment: Bug 1889129 - Remove Android codec support probe from content process initialization

Comment 9

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/10ac43f0b166
Remove Android codec support probe from content process initialization r=jolin

Comment 10

[Cristian Tuns]

https://hg.mozilla.org/mozilla-central/rev/10ac43f0b166