Clean up

gfx/layers/ipc/CompositorChild.cpp

@@ -22,6 +22,7 @@
  *
  * Contributor(s):
  *   Benoit Girard <bgirard@mozilla.com>
+ *   Ali Juma <ajuma@mozilla.com>
  *
  * Alternatively, the contents of this file may be used under the terms of
  * either the GNU General Public License Version 2 or later (the "GPL"), or
@@ -59,13 +60,13 @@ CompositorChild::CompositorChild(Thread *aCompositorThread, LayerManager *aLayer
 
 CompositorChild::~CompositorChild()
 {
-  printf("del compositor child\n");
   MOZ_COUNT_DTOR(CompositorChild);
 }
 
 void
 CompositorChild::Destroy()
 {
+  mLayerManager = NULL;
   SendStop();
 }
 
@@ -112,7 +113,6 @@ CompositorChild::AllocPLayers(const LayersBackend &backend, const WidgetDescript
 bool
 CompositorChild::DeallocPLayers(PLayersChild* actor)
 {
-  printf("actor destroy\n");
   delete actor;
   return true;
 }

gfx/layers/ipc/CompositorChild.h

@@ -75,13 +75,12 @@ protected:
 
 private:
   Thread *mCompositorThread;
-  LayerManager *mLayerManager;
-  nsRefPtr<CompositorParent> mCompositorParent;
+  nsRefPtr<LayerManager> mLayerManager;
 
   DISALLOW_EVIL_CONSTRUCTORS(CompositorChild);
 };
 
-} // lauers
+} // layers
 } // mozilla
 
 #endif // mozilla_layers_CompositorChild_h

gfx/layers/ipc/CompositorParent.cpp

@@ -22,6 +22,7 @@
  *
  * Contributor(s):
  *   Benoit Girard <bgirard@mozilla.com>
+ *   Ali Juma <ajuma@mozilla.com>
  *
  * Alternatively, the contents of this file may be used under the terms of
  * either the GNU General Public License Version 2 or later (the "GPL"), or
@@ -45,7 +46,6 @@ namespace mozilla {
 namespace layers {
 
 CompositorParent::CompositorParent()
-  : mLayerManager(NULL)
 {
 
   MOZ_COUNT_CTOR(CompositorParent);
@@ -53,7 +53,6 @@ CompositorParent::CompositorParent()
 
 CompositorParent::~CompositorParent()
 {
-  printf("delete parent\n");
   MOZ_COUNT_DTOR(CompositorParent);
 }
 

gfx/layers/ipc/CompositorParent.h

@@ -22,6 +22,7 @@
  *
  * Contributor(s):
  *   Benoit Girard <bgirard@mozilla.com>
+ *   Ali Juma <ajuma@mozilla.com>
  *
  * Alternatively, the contents of this file may be used under the terms of
  * either the GNU General Public License Version 2 or later (the "GPL"), or

gfx/layers/ipc/ShadowLayersChild.cpp

@@ -47,7 +47,6 @@ namespace layers {
 void
 ShadowLayersChild::Destroy()
 {
-  printf("Destroy shadow layers\n");
   NS_ABORT_IF_FALSE(0 == ManagedPLayerChild().Length(),
                     "layers should have been cleaned up by now");
   PLayersChild::Send__delete__(this);