1023689 - Fatal assert in GMPGetVideoDecoder when launching subprocess on Win 8

Status: RESOLVED WORKSFORME

(Core :: Audio/Video, defect)

Description

[Chris Pearce [:cpearce (Not reading bugmail)]]

I get a fatal assert when calling GetGMPVideoDecoderVP8() on Win8:

xul.dll!mozilla::Preferences::InitStaticMembers() Line 410	C++
xul.dll!mozilla::Preferences::GetBool(const char * aPref=0x072b2e64, bool * aResult=0x10b9f21b) Line 1353	C++
xul.dll!mozilla::Preferences::GetBool(const char * aPref=0x072b2e64, bool aDefault=false) Line 109	C++
xul.dll!mozilla::widget::WinTaskbar::GetAppUserModelID(nsAString_internal & aDefaultGroupId={...}) Line 253	C++
xul.dll!mozilla::widget::WinTaskbar::GetDefaultGroupId(nsAString_internal & aDefaultGroupId={...}) Line 326	C++
xul.dll!mozilla::ipc::GeckoChildProcessHost::InitWindowsGroupID() Line 310	C++
xul.dll!mozilla::ipc::GeckoChildProcessHost::PrepareLaunch() Line 262	C++
xul.dll!mozilla::ipc::GeckoChildProcessHost::SyncLaunch(std::vector<std::basic_string<char,std::char_traits<char>,std::allocator<char> >,std::allocator<std::basic_string<char,std::char_traits<char>,std::allocator<char> > > > aExtraOpts={...}, int aTimeoutMs=30000, base::ProcessArchitecture arch=PROCESS_ARCH_I386) Line 325	C++
xul.dll!mozilla::gmp::GMPProcessParent::Launch(int aTimeoutMs=30000) Line 44	C++
xul.dll!mozilla::gmp::GMPParent::LoadProcess() Line 66	C++
xul.dll!mozilla::gmp::GMPParent::EnsureProcessLoaded() Line 198	C++
xul.dll!mozilla::gmp::GMPParent::GetGMPVideoDecoder(mozilla::gmp::GMPVideoDecoderParent * * aGMPVD=0x10b9fa78) Line 208	C++
xul.dll!mozilla::gmp::GeckoMediaPluginService::GetGMPVideoDecoderVP8(GMPVideoHost * * aOutVideoHost=0x0d623e98, GMPVideoDecoder * * aGMPVD=0x0d623e94) Line 209	C++
[...]

widget::WinTaskbar::GetAppUserModelID() is reading a pref from the pref service off main thread, and the pref service fatally asserts that it should not be used off main thread.

This assert was added on 5 March 2014, so I'm not surprised no one else has hit it yet.

Comment 1

[Chris Pearce [:cpearce (Not reading bugmail)]]

Seems to not have been an issue.