1425202 - nsLayoutStatics Shutdown() never called

Status: UNCONFIRMED

(Core :: Layout, defect, P3)

Description

[Damien Zammit]

User Agent: Mozilla/5.0 (X11; Linux x86_64; rv:54.0) Gecko/20100101 Firefox/54.0
Build ID: 20170628224917

Steps to reproduce:

On default/tip


Actual results:

nsLayoutStatics::Shutdown() is never called and thus important things like cubeb_destroy() is never called causing audio glitches on the host audio server upon firefox shutdown.

I put some printfs in the AddRef and Release functions and somehow the ref counter gets out of sync, ie does not always count in order.

My suspicion is that there are multiple instances of the static ref counter among multiple threads (?)


Expected results:

nsLayoutStatics::Shutdown() should be called unconditionally when firefox is shutting down, or the ref counter needs to count properly.

Comment 1

[Damien Zammit]

Hi, I just built two versions, a debug build and a non-debug build with the printfs in AddRef and Release.  In the debug build, the ref counters count as expected and it shuts down correctly.  In the non-debug build, the counters stop counting down at some point and I miss out on seeing a printf when Shutdown is supposed to be called.

I don't know what is going on, but it appears that it's only non-debug builds don't shut down correctly (?)  Or is it something about the non-debug build itself that makes this whole bug a false alarm (?)