Closed Bug 940075 Opened 11 years ago Closed 11 years ago

crash in mozilla::dom::PBrowserChild::SendRecordingDeviceEvents(nsString const&, bool const&, bool const&)


(Core :: WebRTC: Audio/Video, defect)

26 Branch
Gonk (Firefox OS)
Not set



Tracking Status
firefox28 --- fixed
b2g-v1.2 --- verified


(Reporter: jsmith, Assigned: schien)



(Keywords: crash, reproducible, Whiteboard: [b2g-crash])

Crash Data

This bug was filed from the Socorro interface and is 
report bp-492bd7cb-5ac0-47fc-a9c6-ddcc12131118.

Saw this when I tried to kill an existing gUM audio process running gUM. I'm working on getting exact STR on this.

Frame 	Module 	Signature 	Source
0 	mozilla::dom::PBrowserChild::SendRecordingDeviceEvents(nsString const&, bool const&, bool const&) 	ipc/chromium/src/chrome/common/ipc_message.h
1 	mozilla::GetUserMediaNotificationEvent::Run() 	dom/media/MediaManager.cpp
2 	nsThread::ProcessNextEvent(bool, bool*) 	xpcom/threads/nsThread.cpp
3 	NS_ProcessNextEvent(nsIThread*, bool) 	xpcom/glue/nsThreadUtils.cpp
4 	mozilla::ipc::MessagePump::Run(base::MessagePump::Delegate*) 	ipc/glue/MessagePump.cpp
5 	mozilla::ipc::MessagePumpForChildProcess::Run(base::MessagePump::Delegate*) 	ipc/glue/MessagePump.cpp
6 	MessageLoop::RunInternal() 	ipc/chromium/src/base/
7 	MessageLoop::Run() 	ipc/chromium/src/base/
8 	nsBaseAppShell::Run() 	widget/xpwidgets/nsBaseAppShell.cpp
9 	XRE_RunAppShell 	toolkit/xre/nsEmbedFunctions.cpp
10 	mozilla::ipc::MessagePumpForChildProcess::Run(base::MessagePump::Delegate*) 	ipc/glue/MessagePump.cpp
11 	MessageLoop::RunInternal() 	ipc/chromium/src/base/
12 	MessageLoop::Run() 	ipc/chromium/src/base/
13 	XRE_InitChildProcess 	toolkit/xre/nsEmbedFunctions.cpp
14 	plugin-container 	main 	ipc/app/MozillaRuntimeMain.cpp
15 	__libc_init 	bionic/libc/bionic/libc_init_dynamic.c
16 		@0xb00045a9
Whiteboard: [b2g-crash]

1. Go to
2. Request audio & accept permissions
3. Go to http://テスト
4. Kill the browser process


The browser process should shutdown cleanly


Crash on shutdown.


Looks like after #3 occurs that the browser process thinks recording is happening, which in fact, there is no recording present.
Keywords: reproducible
Looks like this bug will only reproduce if bug 940045 reproduces first.
Depends on: 940045
Flags: needinfo?(rjesup)
As I think we'll be landing bug 940045 soon, let's retest once that lands.
Flags: needinfo?(rjesup)
|TabChild::GetFrom| returns nullptr under this scenario because the corresponding docshell is detached from window. This means we're unable to notify parent process that recording is being shutdown.
The reason I move |SendRecordingDeviceEvents| from PContent to PBrowser is because I need to get page URL in parent process. Since we are going to get page URL direct from child process, I'll move |SendRecordingDeviceEvents| back to PContent.
I'll merge the required modification to Bug 940045.
Is this fixed now that bug 940045 has landed?
Flags: needinfo?(schien)
Yes, no longer seeing crash in PBrowserChild::SendRecordingDeviceEvents in today's m-c.
Closed: 11 years ago
Flags: needinfo?(schien)
Resolution: --- → FIXED
Keywords: verifyme
QA Contact: jsmith
Assignee: nobody → schien
Target Milestone: --- → mozilla28
Verified on 12/12/2013 1.2 build - no longer seeing a crash.
Keywords: verifyme
You need to log in before you can comment on or make changes to this bug.