Closed Bug 635532 Opened 9 years ago Closed 9 years ago

Crash [@ nsHTMLEditor::HideGrabber() ]

Categories

(Core :: Editor, defect, critical)

x86
Windows 7
defect
Not set
critical

Tracking

()

RESOLVED FIXED
Tracking Status
blocking2.0 --- final+

People

(Reporter: scoobidiver, Assigned: ehsan)

References

Details

(Keywords: crash, regression, Whiteboard: [softblocker])

Crash Data

Attachments

(1 file)

It is a new crash signature that first appeared in 4.0b12pre/20110218144935.

One comment says:
"Just quit 3 windows at the same time. Minefield has been rendering pages with odd pixelations since the update. Every page seems rendered improperly."

Signature	nsHTMLEditor::HideGrabber()
UUID	54542787-4602-4b5c-9f95-074232110219
Time 	2011-02-19 10:15:15.437134
Uptime	304
Install Age	304 seconds (5.1 minutes) since version was first installed.
Product	Firefox
Version	4.0b12pre
Build ID	20110219030355
Branch	2.0
OS	Windows NT
OS Version	6.1.7600
CPU	x86
CPU Info	GenuineIntel family 6 model 23 stepping 10
Crash Reason	EXCEPTION_ACCESS_VIOLATION_READ
Crash Address	0x0
App Notes 	AdapterVendorID: 8086, AdapterDeviceID: 29c2, AdapterDriverVersion: 8.15.10.1930

Frame 	Module 	Signature [Expand] 	Source
0 	xul.dll 	nsHTMLEditor::HideGrabber 	editor/libeditor/html/nsHTMLAbsPosition.cpp:307
1 	xul.dll 	nsHTMLEditor::cycleCollection::Unlink 	editor/libeditor/html/nsHTMLEditor.cpp:241
2 	xul.dll 	nsCycleCollector::CollectWhite 	xpcom/base/nsCycleCollector.cpp:1912
3 	xul.dll 	nsCycleCollector::FinishCollection 	xpcom/base/nsCycleCollector.cpp:2720
4 	xul.dll 	nsCycleCollector::Collect 	xpcom/base/nsCycleCollector.cpp:2530
5 	xul.dll 	nsCycleCollector::Shutdown 	xpcom/base/nsCycleCollector.cpp:2779
6 	xul.dll 	nsCycleCollector_shutdown 	xpcom/base/nsCycleCollector.cpp:3505
7 	xul.dll 	mozilla::ShutdownXPCOM 	xpcom/build/nsXPComInit.cpp:694
8 	xul.dll 	ScopedXPCOMStartup::~ScopedXPCOMStartup 	toolkit/xre/nsAppRunner.cpp:1115
9 	xul.dll 	XRE_main 	toolkit/xre/nsAppRunner.cpp:3797
10 	firefox.exe 	wmain 	toolkit/xre/nsWindowsWMain.cpp:128
11 	firefox.exe 	__tmainCRTStartup 	obj-firefox/memory/jemalloc/crtsrc/crtexe.c:591
12 	kernel32.dll 	BaseThreadInitThunk 	
13 	ntdll.dll 	ntdll.dll@0x5b428 	
14 	ntdll.dll 	ntdll.dll@0x5b3fb 

The regression range is:
http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=e95166704b63&tochange=689f55593c70

More reports at:
https://crash-stats.mozilla.com/report/list?product=Firefox&range_value=4&range_unit=weeks&signature=nsHTMLEditor%3A%3AHideGrabber%28%29
Peter, could this be a fallout from bug 633738?
Blocks: 633738
Actually I know what's wrong here.  We should check to make sure that we're not removing those anonymous objects needlessly.  Patch forthcoming.
And I think we need to softblock on this for final.  Although I'd really like us to get this in for b12 if possible.
blocking2.0: --- → final+
Whiteboard: [softblocker]
Attached patch Patch (v1)Splinter Review
Assignee: nobody → ehsan
Status: NEW → ASSIGNED
Attachment #513803 - Flags: review?(roc)
Attachment #513803 - Flags: approval2.0?
Comment on attachment 513803 [details] [diff] [review]
Patch (v1)

This patch is fairly trivial, trying to parallelize review requests!  :-)
Attachment #513803 - Flags: review?(peterv)
Attachment #513803 - Flags: review?(jst)
Attachment #513803 - Flags: review?(Olli.Pettay)
Attachment #513803 - Flags: review?(roc)
Attachment #513803 - Flags: review+
Attachment #513803 - Flags: approval2.0?
Attachment #513803 - Flags: approval2.0+
Attachment #513803 - Flags: review?(jst) → review+
Landed on mozilla-central:

http://hg.mozilla.org/mozilla-central/rev/12056f3beaf2
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Attachment #513803 - Flags: review?(Olli.Pettay) → review+
Attachment #513803 - Flags: review?(peterv)
Thanks Ehsan for cleaning this up.
Crash Signature: [@ nsHTMLEditor::HideGrabber() ]
You need to log in before you can comment on or make changes to this bug.