Bug 180127 Inspector destructors unconditionally delete objects

r=caillon sr=bz

extensions/inspector/base/src/inBitmap.cpp

@@ -45,6 +45,7 @@
 ///////////////////////////////////////////////////////////////////////////////
 
 inBitmap::inBitmap()
+  : mBits(nsnull) 
 {
   NS_INIT_ISUPPORTS();
 }

extensions/inspector/base/src/inCSSValueSearch.cpp

@@ -46,18 +46,22 @@
 static NS_DEFINE_CID(kInspectorCSSUtilsCID, NS_INSPECTORCSSUTILS_CID);
 
 ///////////////////////////////////////////////////////////////////////////////
-
 inCSSValueSearch::inCSSValueSearch()
+  : mResults(nsnull),
+    mProperties(nsnull),
+    mLastResult(nsnull),
+    mBaseURL(nsnull),
+    mTextCriteria(nsnull),
+    mResultCount(0),
+    mPropertyCount(0),
+    mIsActive(PR_FALSE),
+    mHoldResults(PR_TRUE),
+    mReturnRelativeURLs(PR_TRUE),
+    mNormalizeChromeURLs(PR_FALSE)
 {
   NS_INIT_ISUPPORTS();
-  
-  mHoldResults = PR_TRUE;
-  mReturnRelativeURLs = PR_FALSE;
-  mNormalizeChromeURLs = PR_FALSE;
-  mResultCount = 0;
 
   mProperties = new nsCSSProperty[100];
-  mPropertyCount = 0;
   mCSSUtils = do_GetService(kInspectorCSSUtilsCID);
 }
 

extensions/inspector/base/src/inCSSValueSearch.h

@@ -63,21 +63,20 @@ public:
   virtual ~inCSSValueSearch();
 
 protected:
-  PRBool mIsActive;
-  PRInt32 mResultCount;
-  PRBool mHoldResults;
-  nsVoidArray* mResults;
-  nsAutoString* mLastResult;
   nsCOMPtr<inISearchObserver> mObserver;
-
+  nsCOMPtr<nsIInspectorCSSUtils> mCSSUtils;
   nsCOMPtr<nsIDOMDocument> mDocument;
+  nsVoidArray* mResults;
+  nsCSSProperty* mProperties;
+  nsAutoString* mLastResult;
   nsAutoString* mBaseURL;
+  nsAutoString* mTextCriteria;
+  PRInt32 mResultCount;
+  PRUint32 mPropertyCount;
+  PRBool mIsActive;
+  PRBool mHoldResults;
   PRBool mReturnRelativeURLs;
   PRBool mNormalizeChromeURLs;
-  nsAutoString* mTextCriteria;
-  nsCSSProperty* mProperties;
-  PRUint32 mPropertyCount;
-  nsCOMPtr<nsIInspectorCSSUtils> mCSSUtils;
 
   nsresult InitSearch();
   nsresult KillSearch(PRInt16 aResult);

extensions/inspector/base/src/inFileSearch.cpp

@@ -45,10 +45,19 @@
 ///////////////////////////////////////////////////////////////////////////////
 
 inFileSearch::inFileSearch()
+  : mSearchLoop(nsnull),
+    mBasePath(nsnull),
+    mTextCriteria(nsnull),
+    mFilenameCriteria(nsnull),
+    mDirsSearched(0),
+    mFilenameCriteriaCount(0),
+    mResultCount(0),
+    mIsActive(PR_FALSE),
+    mHoldResults(PR_FALSE),
+    mReturnRelativePaths(PR_FALSE),
+    mSearchRecursive(PR_FALSE)
 {
   NS_INIT_ISUPPORTS();
-
-  mSearchLoop = 0;
 }
 
 inFileSearch::~inFileSearch()

extensions/inspector/base/src/inFileSearch.h

@@ -59,27 +59,22 @@ public:
   virtual ~inFileSearch();
 
 protected:
-  // inISearchProcess related
-  PRBool mIsActive;
-  PRInt32 mResultCount;
-  nsCOMPtr<nsIFile> mLastResult;
-  nsCOMPtr<nsISupportsArray> mResults;
-  PRBool mHoldResults;
-  nsAutoString* mBasePath;
-  PRBool mReturnRelativePaths;
   nsCOMPtr<inISearchObserver> mObserver;
-
-  // inIFileSearch related
+  nsCOMPtr<nsISupportsArray> mResults;
+  nsCOMPtr<nsISupportsArray> mDirStack;
+  nsCOMPtr<nsIFile> mLastResult;
   nsCOMPtr<nsIFile> mSearchPath;
+  inSearchLoop* mSearchLoop;
+  nsAutoString* mBasePath;
   nsAutoString* mTextCriteria;
   PRUnichar** mFilenameCriteria;
-  PRUint32 mFilenameCriteriaCount;
-  PRBool mSearchRecursive;
   PRUint32 mDirsSearched;
-
-  // asynchronous search related
-  nsCOMPtr<nsISupportsArray> mDirStack;
-  inSearchLoop* mSearchLoop;
+  PRUint32 mFilenameCriteriaCount;
+  PRInt32 mResultCount;
+  PRBool mIsActive;
+  PRBool mHoldResults;
+  PRBool mReturnRelativePaths;
+  PRBool mSearchRecursive;
 
   // life cycle of search
   nsresult InitSearch();