473678 - Crash [@ nsCSSFrameConstructor::ProcessOneRestyle] on reload with binding and textZoom and display: none on root

Status: VERIFIED FIXED

(Core :: Layout, defect, P2)

Description

[Martijn Wargers (dead)]

Attachment: testcase (uses enhanced privileges)

See testcase (which uses enhanced privileges), which crashes current trunk build on reload. This is a regression from bug 283686.

http://crash-stats.mozilla.com/report/index/82f79c35-2bc7-45e8-9af5-0b92b2090114?p=1
0  	 	@0x1ad43df  	
1 	xul.dll 	xul.dll@0x316e89 	
2 	xul.dll 	nsCSSFrameConstructor::ProcessOneRestyle 	layout/base/nsCSSFrameConstructor.cpp:13226
3 	xul.dll 	nsCSSFrameConstructor::ProcessPendingRestyles 	layout/base/nsCSSFrameConstructor.cpp:13332
4 	xul.dll 	PresShell::DoFlushPendingNotifications 	layout/base/nsPresShell.cpp:4453
5 	xul.dll 	PresShell::FlushPendingNotifications 	layout/base/nsPresShell.cpp:4411
6 	xul.dll 	nsCSSFrameConstructor::RestyleEvent::Run 	layout/base/nsCSSFrameConstructor.cpp:13417
7 	xul.dll 	nsThread::ProcessNextEvent 	xpcom/threads/nsThread.cpp:510
8 	xul.dll 	nsBaseAppShell::Run 	widget/src/xpwidgets/nsBaseAppShell.cpp:170
9 	nspr4.dll 	PR_GetEnv 	
10 	firefox.exe 	wmain 	toolkit/xre/nsWindowsWMain.cpp:87
11 	firefox.exe 	firefox.exe@0x2197 	
12 	kernel32.dll 	BaseProcessStart

Comment 1

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

I crashed every time without my patch to bug 473042 but I don't crash with that patch.

Comment 2

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Fixed by landing of bug 473042.

Comment 3

[Martijn Wargers (dead)]

Verified fixed, using:
Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.2a1pre) Gecko/20090126 Minefield/3.2a1pre