Bug 1225682 - Don't use nsAuto{,C}String as class member variables in netwerk/. r=mcmanus

netwerk/base/BackgroundFileSaver.h

@@ -205,7 +205,7 @@ private:
    * The SHA 256 hash in raw bytes of the downloaded file. This is written
    * by the worker thread but can be read on the main thread.
    */
-  nsAutoCString mSha256;
+  nsCString mSha256;
 
   /**
    * Whether or not to compute the hash. Must be set on the main thread before

netwerk/protocol/about/nsAboutCacheEntry.h

@@ -60,7 +60,7 @@ private:
                    uint32_t *aWriteCount);
 
 private:
-    nsAutoCString mStorageName, mEnhanceId;
+    nsCString mStorageName, mEnhanceId;
     nsCOMPtr<nsILoadContextInfo> mLoadInfo;
     nsCOMPtr<nsIURI> mCacheURI;
 

netwerk/protocol/http/HttpBaseChannel.cpp

@@ -1885,8 +1885,10 @@ class CookieNotifierRunnable : public nsRunnable
 {
 public:
   CookieNotifierRunnable(HttpBaseChannel* aChannel, char const * aCookie)
-    : mChannel(aChannel), mCookie(aCookie)
-  { }
+    : mChannel(aChannel)
+  {
+    CopyASCIItoUTF16(aCookie, mCookie);
+  }
 
   NS_IMETHOD Run()
   {
@@ -1901,7 +1903,7 @@ public:
 
 private:
   RefPtr<HttpBaseChannel> mChannel;
-  NS_ConvertASCIItoUTF16 mCookie;
+  nsString mCookie;
 };
 
 } // namespace