Last Comment Bug 676316 - Crash Report [@ @0x0 | nsDeviceMotion::DeviceMotionChanged ]
: Crash Report [@ @0x0 | nsDeviceMotion::DeviceMotionChanged ]
Status: RESOLVED FIXED
: regression
Product: Core
Classification: Components
Component: DOM (show other bugs)
: Trunk
: x86 Mac OS X
: -- critical with 1 vote (vote)
: ---
Assigned To: Josh Matthews [:jdm]
:
Mentors:
http://people.mozilla.com/~nhirata/ht...
: 674156 (view as bug list)
Depends on:
Blocks: 673472 676595
  Show dependency treegraph
 
Reported: 2011-08-03 11:09 PDT by Naoki Hirata :nhirata (please use needinfo instead of cc)
Modified: 2011-08-06 02:49 PDT (History)
8 users (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---


Attachments
Reverse the device motion listener removal early return logic. (926 bytes, patch)
2011-08-04 15:24 PDT, Josh Matthews [:jdm]
dougt: review+
Details | Diff | Review

Description Naoki Hirata :nhirata (please use needinfo instead of cc) 2011-08-03 11:09:46 PDT
Build identifier: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.6; rv:8.0a1) Gecko/20110803 Firefox/8.0a1

Mac OS X : 10.6.7
STR: 
1. go to http://people.mozilla.com/~nhirata/html_tp/ball.html
2. while moving the computer, close the window
Expected: no crash
Actual : crash

This bug was filed from the Socorro interface and is 
report bp-ac7017a6-5840-41ba-adf0-2acd92110803 .
============================================================= 
Frame 	Module 	Signature [Expand] 	Source
0 		@0x0 	
1 	XUL 	nsDeviceMotion::DeviceMotionChanged 	dom/system/nsDeviceMotion.cpp:236
2 	XUL 	nsDeviceMotionSystem::UpdateHandler 	dom/system/cocoa/nsDeviceMotionSystem.mm:146
3 	XUL 	nsTimerImpl::Fire 	xpcom/threads/nsTimerImpl.cpp:424
4 	XUL 	nsTimerEvent::Run 	xpcom/threads/nsTimerImpl.cpp:520
5 	XUL 	nsThread::ProcessNextEvent 	xpcom/threads/nsThread.cpp:631
6 	XUL 	NS_ProcessPendingEvents_P 	obj-firefox/x86_64/xpcom/build/nsThreadUtils.cpp:195
7 	XUL 	nsBaseAppShell::NativeEventCallback 	widget/src/xpwidgets/nsBaseAppShell.cpp:130
8 	XUL 	nsAppShell::ProcessGeckoEvents 	widget/src/cocoa/nsAppShell.mm:424
9 	CoreFoundation 	__CFRunLoopDoSources0 	
10 	CoreFoundation 	__CFRunLoopRun 	
11 	CoreFoundation 	CFRunLoopRunSpecific 	
12 	HIToolbox 	HIToolbox@0x2e7ed 	
13 	HIToolbox 	HIToolbox@0x2e550 	
14 	HIToolbox 	HIToolbox@0x2e4ab 	
15 	AppKit 	_DPSNextEvent 	
16 	AppKit 	-[NSApplication nextEventMatchingMask:untilDate:inMode:dequeue:] 	
17 	AppKit 	-[NSApplication run] 	
18 	XUL 	nsAppShell::Run 	widget/src/cocoa/nsAppShell.mm:771
19 	XUL 	nsAppStartup::Run 	toolkit/components/startup/nsAppStartup.cpp:224
20 	XUL 	XRE_main 	toolkit/xre/nsAppRunner.cpp:3577
21 	firefox 	main 	browser/app/nsBrowserApp.cpp:198
22 	firefox 	firefox@0xb73
Comment 2 Marcia Knous [:marcia - use ni] 2011-08-04 10:28:02 PDT
Possible dupe of Bug 673472? Signature seems very similar.
Comment 3 Josh Matthews [:jdm] 2011-08-04 15:24:05 PDT
Created attachment 550857 [details] [diff] [review]
Reverse the device motion listener removal early return logic.

Ugh. I should never be allowed to touch IndexOf, it turns out.
Comment 4 Doug Turner (:dougt) 2011-08-05 12:09:09 PDT
http://hg.mozilla.org/mozilla-central/rev/be090ee1747a
Comment 6 Doug Turner (:dougt) 2011-08-05 13:05:59 PDT
*** Bug 674156 has been marked as a duplicate of this bug. ***
Comment 7 Marco Bonardo [::mak] 2011-08-06 02:49:29 PDT
http://hg.mozilla.org/mozilla-central/rev/899a225fd580

Note You need to log in before you can comment on or make changes to this bug.