"ASSERTION: Caller forgot to check ReadyToExecuteScripts()" involving modal networking error dialog

RESOLVED WORKSFORME

Status

()

Core
DOM
P2
normal
RESOLVED WORKSFORME
11 years ago
10 years ago

People

(Reporter: Jesse Ruderman, Assigned: sicking)

Tracking

(Blocks: 2 bugs, {assertion})

Trunk
x86
Mac OS X
assertion
Points:
---
Dependency tree / graph
Bug Flags:
wanted1.9.1 +

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

11 years ago
Created attachment 304125 [details]
stack trace for the assertion

I have a hard-to-reduce testcase that triggers:

###!!! ASSERTION: Caller forgot to check ReadyToExecuteScripts(): 'ReadyToExecuteScripts() && nsContentUtils::IsSafeToRunScript()', file /Users/jruderman/trunk/mozilla/content/base/src/nsScriptLoader.cpp, line 519

The testcase does something like:
  xulEditor.setAttribute('src', "foop:yaz");
to trigger a networking error dialog.

The assertion was added in bug 343730.  My build includes the patch for bug 401155, which modifies the assertion to include the second condition.  (Shouldn't it be two separate assertions?)
Need to test this after bug 395609 and bug 420415
Assignee: nobody → jonas
Flags: wanted1.9.1+
Priority: -- → P2
(Reporter)

Comment 2

10 years ago
I haven't seen this assertion for a while.  I retested my old "hard-to-reduce testcase" and didn't see the assertion there either.
Status: NEW → RESOLVED
Last Resolved: 10 years ago
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.