1283328 - Miscellaneous thread shutdown cleanups

Status: RESOLVED FIXED

(Core :: XPCOM, defect)

Description

[Kyle Huey (Exited; not receiving bugmail, old account, do not use)]

Attachment: Patch

Between AsyncShutdown and unifying Task/Runnable there's some cruft sitting in the tree.

Comment 1

[Nathan Froyd [:froydnj]]

Comment on attachment 8766582 [details] [diff] [review]
Patch

Review of attachment 8766582 [details] [diff] [review]:
-----------------------------------------------------------------

r=me

::: dom/media/systemservices/CamerasChild.cpp
@@ +548,1 @@
>      RefPtr<ShutdownRunnable> runnable = new ShutdownRunnable(event);

Can you fix ShutdownRunnable to not do a bunch of extra refcounting by taking an already_AddRefed instead of a RefPtr?

::: dom/media/systemservices/CamerasUtils.cpp
@@ -13,5 @@
> -#include "nsIIPCBackgroundChildCreateCallback.h"
> -
> -namespace mozilla {
> -namespace camera {
> -

...

Comment 2

[Kyle Huey (Exited; not receiving bugmail, old account, do not use)]

(In reply to Nathan Froyd [:froydnj] from comment #1)
> Comment on attachment 8766582 [details] [diff] [review]
> Patch
> 
> Review of attachment 8766582 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> r=me
> 
> ::: dom/media/systemservices/CamerasChild.cpp
> @@ +548,1 @@
> >      RefPtr<ShutdownRunnable> runnable = new ShutdownRunnable(event);
> 
> Can you fix ShutdownRunnable to not do a bunch of extra refcounting by
> taking an already_AddRefed instead of a RefPtr?

Done.

> ::: dom/media/systemservices/CamerasUtils.cpp
> @@ -13,5 @@
> > -#include "nsIIPCBackgroundChildCreateCallback.h"
> > -
> > -namespace mozilla {
> > -namespace camera {
> > -
> 
> ...

I disemboweled this file in bug 1283327, I just didn't remove the .cpp until I could remove the header too in this bug.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/910670d0db65
Clean up some thread shutdown stuff and remove CameraUtils. r=froydnj

Comment 4

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/910670d0db65