Bug 1142364: Store instances of |Bluetooth*HALResultRunnable| in ref-counted pointers, r=shuang

Result runnables in Bluetooth's HAL backend ar now protected by ref-counted
pointers. This prevent memory leaks if dispatching to the main thread fails.

dom/bluetooth/bluedroid/BluetoothA2dpHALInterface.cpp

@@ -26,7 +26,7 @@ DispatchBluetoothA2dpHALResult(
 {
   MOZ_ASSERT(aRes);
 
-  nsRunnable* runnable;
+  nsRefPtr<nsRunnable> runnable;
 
   if (aStatus == STATUS_SUCCESS) {
     runnable = new BluetoothA2dpHALResultRunnable(aRes, aMethod);

dom/bluetooth/bluedroid/BluetoothAvrcpHALInterface.cpp

@@ -26,7 +26,7 @@ DispatchBluetoothAvrcpHALResult(
 {
   MOZ_ASSERT(aRes);
 
-  nsRunnable* runnable;
+  nsRefPtr<nsRunnable> runnable;
 
   if (aStatus == STATUS_SUCCESS) {
     runnable = new BluetoothAvrcpHALResultRunnable(aRes, aMethod);

dom/bluetooth/bluedroid/BluetoothHALInterface.cpp

@@ -79,7 +79,7 @@ DispatchBluetoothHALResult(BluetoothResultHandler* aRes,
 {
   MOZ_ASSERT(aRes);
 
-  nsRunnable* runnable;
+  nsRefPtr<nsRunnable> runnable;
 
   if (aStatus == STATUS_SUCCESS) {
     runnable = new BluetoothHALResultRunnable(aRes, aMethod);

dom/bluetooth/bluedroid/BluetoothHandsfreeHALInterface.cpp

@@ -26,7 +26,7 @@ DispatchBluetoothHandsfreeHALResult(
 {
   MOZ_ASSERT(aRes);
 
-  nsRunnable* runnable;
+  nsRefPtr<nsRunnable> runnable;
 
   if (aStatus == STATUS_SUCCESS) {
     runnable = new BluetoothHandsfreeHALResultRunnable(aRes, aMethod);

dom/bluetooth/bluedroid/BluetoothSocketHALInterface.cpp

@@ -37,7 +37,7 @@ DispatchBluetoothSocketHALResult(
 {
   MOZ_ASSERT(aRes);
 
-  nsRunnable* runnable;
+  nsRefPtr<nsRunnable> runnable;
 
   if (aStatus == STATUS_SUCCESS) {
     runnable = new BluetoothSocketHALIntResultRunnable(aRes, aMethod, aArg);
@@ -60,7 +60,7 @@ DispatchBluetoothSocketHALResult(
 {
   MOZ_ASSERT(aRes);
 
-  nsRunnable* runnable;
+  nsRefPtr<nsRunnable> runnable;
 
   if (aStatus == STATUS_SUCCESS) {
     runnable = new BluetoothSocketHALIntStringIntResultRunnable(