1149185 - Flame: Loop crashes on debug build

Status: RESOLVED FIXED

(Core :: Graphics: Layers, defect)

Description

[No-Jun Park [:njpark]]

STR:
Load the debug build, start loop, enter a room where no one else is in yet
Actual:
Loop app crashes
Expected:
Loop app does not crash

Following Assert failures were seen:
I/Gecko   ( 2536): [Child 2536] ###!!! ASSERTION: Child frames aren't sorted correctly: '(!mFrames.IsEmpty() && mFrames.FirstChild()->GetContent()->GetContainingShadow()) || nsIFrame::IsFrameListSorted<IsOrderLEQWithDOMFallback>(mFrames)', file ../../../gecko/layout/generic/nsFlexContainerFrame.cpp, line 1906
F/MOZ_Assert( 2536): Assertion failure: aForwarder && aForwarder->GetMessageLoop() == mAllocator->GetMessageLoop(), at ../../../gecko/gfx/layers/client/TextureClient.cpp:257

Crash stat:
https://crash-stats.mozilla.org/report/index/74a5848e-84b1-4ce8-8e47-bbdc92150330

Version info:
Build ID               20150330023007
Gaia Revision          be25b16efa19bab8d54be08f8fe45dcc93bf93d0
Gaia Date              2015-03-29 10:19:00
Gecko Revision         https://hg.mozilla.org/integration/b2g-inbound/rev/ee5a8ba64485
Gecko Version          39.0a1
Device Name            flame
Firmware(Release)      4.4.2
Firmware(Incremental)  eng.cltbld.20150330.055351
Firmware Date          Mon Mar 30 05:54:00 EDT 2015
Bootloader             L1TC000118D0

Comment 1

[Sotaro Ikeda [:sotaro]]

I am going to take a look.

Comment 2

[Sotaro Ikeda [:sotaro]]

Attachment: patch - override GetMessageLoop()

The patch fixed the problem on my flame.

Comment 3

[Sotaro Ikeda [:sotaro]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/5f7a58ce3767

Comment 4

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/5f7a58ce3767