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

RESOLVED FIXED in mozilla2.0b8

Status

()

Core
Graphics
RESOLVED FIXED
7 years ago
7 years ago

People

(Reporter: mats, Assigned: mats)

Tracking

({pp, regression, testcase})

Trunk
mozilla2.0b8
x86_64
Linux
pp, regression, testcase
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

(Assignee)

Description

7 years ago
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

7 years ago
Created attachment 489142 [details]
stack for the hang
Does changing it to testPluginIsOOP() help?
http://hg.mozilla.org/mozilla-central/annotate/4ef3abd2012c/layout/tools/reftest/reftest.js#l445
(Assignee)

Comment 3

7 years ago
Created attachment 489286 [details] [diff] [review]
fix

Thanks Karl.
(Assignee)

Updated

7 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

7 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

7 years ago
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.