Closed
Bug 610646
Opened 14 years ago
Closed 14 years ago
100% CPU hang running crashtest: modules/plugin/test/crashtests/598862.html
Categories
(Core :: Graphics, defect)
Tracking
()
RESOLVED
FIXED
mozilla2.0b8
People
(Reporter: MatsPalmgren_bugz, Assigned: MatsPalmgren_bugz)
References
Details
(Keywords: platform-parity, regression, testcase)
Attachments
(2 files)
3.22 KB,
text/plain
|
Details | |
667 bytes,
patch
|
Details | Diff | Splinter Review |
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?
Assignee | ||
Comment 1•14 years ago
|
||
Comment 2•14 years ago
|
||
Does changing it to testPluginIsOOP() help? http://hg.mozilla.org/mozilla-central/annotate/4ef3abd2012c/layout/tools/reftest/reftest.js#l445
Assignee | ||
Comment 3•14 years ago
|
||
Thanks Karl.
Assignee | ||
Updated•14 years ago
|
Keywords: checkin-needed
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 ...
Poked at it, but this arch-pref setup is not ... obvious or well commented and I'm out of time.
Assignee: nobody → matspal
Assignee | ||
Comment 6•14 years ago
|
||
Filed bug 612147 to followup on the suggested syntax change in comment 4. http://hg.mozilla.org/mozilla-central/rev/b062f63a98fb
Keywords: checkin-needed
Target Milestone: --- → mozilla2.0b8
Assignee | ||
Updated•14 years ago
|
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•