Free QualcommCameraHardware instance when closing camera

The LINK_mm_camera_destroy is being called in QualcommCameraHardware
destructor. If we do not free the QualcommCameraHardware from
camera_device_close the camera may not be deinitialized and drain
the battery when not using camera.

This is especially visible when Skype is installed - which opens the
camera during startup to probe the camera and obtains its parameters
even when the user is not using camera in skype.

QualcommCameraHardware.h

@@ -223,8 +223,7 @@ namespace android {
 class QualcommCameraHardware : public RefBase{
 public:
 
-    //virtual sp<IMemoryHeap> getPreviewHeap() const;
-    //virtual sp<IMemoryHeap> getRawHeap() const;
+    virtual ~QualcommCameraHardware();
 
     void setCallbacks(camera_notify_callback notify_cb,
                             camera_data_callback data_cb,
@@ -283,7 +282,6 @@ class QualcommCameraHardware : public RefBase{
 
 private:
     QualcommCameraHardware();
-    virtual ~QualcommCameraHardware();
     status_t startPreviewInternal();
     status_t setHistogramOn();
     status_t setHistogramOff();

cameraHAL.cpp

@@ -643,6 +643,7 @@ int camera_device_close(hw_device_t* device)
             gCameraHals[dev->cameraid]->release();
             dev->released = true;
         }
+        delete(gCameraHals[dev->cameraid]);
         gCameraHals[dev->cameraid] = NULL;
         gCamerasOpen--;