Closed Bug 1644365 Opened 4 years ago Closed 4 years ago

MOZ_DIAGNOSTIC_ASSERT(nsContentUtils::IsSafeToRunScript() || mOwnerContent->OwnerDoc()->IsStaticDocument()) (FrameLoader should never be initialized during document update or reflow!)

Categories

(Core :: DOM: Content Processes, defect, P2)

Product:
Core
Component:
DOM: Content Processes
Platform:
Unspecified
Windows 10
Type:
defect

Tracking

()

Status:
RESOLVED WORKSFORME
Tracking Flags:
Tracking Status
firefox-esr68 --- unaffected
firefox-esr78 --- unaffected
firefox77 --- unaffected
firefox78 --- unaffected
firefox79 --- fixed

People

(Reporter: achronop, Assigned: kmag)

References

(Regression)

Details

(Keywords: crash, regression)

Crash Data

This bug is for crash report bp-8da6b6fd-9c5f-447f-8a0f-c35a40200608.

Top 10 frames of crashing thread:

0 xul.dll xul.dll@0x18deec2 
1 xul.dll xul.dll@0x2d34296 
2 dwmapi.dll DwmDefWindowProc 
3  @0xdac89fd56f 
4 xul.dll xul.dll@0x5f067ff 
5 xul.dll xul.dll@0x18de63d 
6 xul.dll xul.dll@0x2b2905a 
7 xul.dll xul.dll@0x4bed04 
8 xul.dll xul.dll@0x2fec35 
9 mozglue.dll mozglue.dll@0x922b

Low traffic crash. I am not able to find a clear pattern of why this happens. I open this bug according to Bug 1595936 which introduced the assert. Feel free to redirect.

Regressed by: 1595936
Has Regression Range: --- → yes

Crash reason: MOZ_DIAGNOSTIC_ASSERT(nsContentUtils::IsSafeToRunScript() || mOwnerContent->OwnerDoc()->IsStaticDocument()) (FrameLoader should never be initialized during document update or reflow!)

Assigning to kmag because he said he will investigate. He added the assertion. This crash reason is the same as IsSafeToRunScript bug 1612431. Same crash reason as MaybeCreateDocShell() crashes.

P2 because the crash volume is low. Looks like this crash is new in 79 Nightly.

Is someone trying to run JS on a print preview document?

Assignee: nobody → kmaglione+bmo
Severity: -- → S2
status-firefox77: --- → unaffected
status-firefox78: --- → unaffected
status-firefox79: --- → affected
status-firefox-esr68: --- → unaffected
status-firefox-esr78: --- → unaffected
Priority: -- → P2
See Also: → https://bugzilla.mozilla.org/show_bug.cgi?id=1612431
Summary: Crash in [@ xul.dll | DwmDefWindowProc] → MOZ_DIAGNOSTIC_ASSERT(nsContentUtils::IsSafeToRunScript() || mOwnerContent->OwnerDoc()->IsStaticDocument()) (FrameLoader should never be initialized during document update or reflow!)

Looks like these went away in Nightly builds after 20200612145655.

Status: NEW → RESOLVED
Closed: 4 years ago
status-firefox79: affected → fixed
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.