Closed Bug 635532 Opened 14 years ago Closed 14 years ago

Crash [@ nsHTMLEditor::HideGrabber() ]

Categories

(Core :: DOM: Editor, defect)

x86
Windows 7
defect
Not set
critical

Tracking

()

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

People

(Reporter: scoobidiver, Assigned: ehsan.akhgari)

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+
Status: ASSIGNED → RESOLVED
Closed: 14 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.

Attachment

General

Created:
Updated:
Size: