610646 - 100% CPU hang running crashtest: modules/plugin/test/crashtests/598862.html

Status: RESOLVED FIXED

(Core :: Graphics, defect)

Description

[Mats Palmgren (inactive)]

Running crashtests in my local DEBUG build hangs when running
modules/plugin/test/crashtests/598862.html using 100% CPU.

The odd thing is that although the crashtests.list manifest file says
skip-if(!haveTestPlugin||http.platform!="X11"||!testPluginIsOOP) load 598862.html
it is still run in this tree which is configured with
ac_add_options --disable-ipc
ac_add_options --disable-libxul

Shouldn't that result in "testPluginIsOOP" being false?

Comment 1

[Mats Palmgren (inactive)]

Attachment: stack for the hang

Comment 2

[Karl Tomlinson (:karlt)]

Does changing it to testPluginIsOOP() help?
http://hg.mozilla.org/mozilla-central/annotate/4ef3abd2012c/layout/tools/reftest/reftest.js#l445

Comment 3

[Mats Palmgren (inactive)]

Attachment: fix

Thanks Karl.

Comment 4

[Chris Jones [:cjones] inactive; ni?/f?/r? if you need me]

Thanks Karl!

I checked with sgreenlay and he doesn't know of any reason not to immediately evaluate testPluginIsOOP for reftests.  Lemme see what that patch would entail ...

Comment 5

[Chris Jones [:cjones] inactive; ni?/f?/r? if you need me]

Poked at it, but this arch-pref setup is not ... obvious or well commented and I'm out of time.

Comment 6

[Mats Palmgren (inactive)]

Filed bug 612147 to followup on the suggested syntax change in comment 4.

http://hg.mozilla.org/mozilla-central/rev/b062f63a98fb