Fix PDT+ bug # 40199: default to ISO-8859-1 encoding instead of UTF-8. Review: waterson

xpfe/components/search/src/nsInternetSearchService.cpp

@@ -653,7 +653,6 @@ void
 InternetSearchDataSource::FireTimer(nsITimer* aTimer, void* aClosure)
 {
 	InternetSearchDataSource *search = NS_STATIC_CAST(InternetSearchDataSource *, aClosure);
-	nsresult rv;
 	if (!search)	return;
 
 #ifndef	REPEATING_TIMERS
@@ -662,6 +661,9 @@ InternetSearchDataSource::FireTimer(nsITimer* aTimer, void* aClosure)
 		mTimer->Cancel();
 		mTimer = nsnull;
 	}
+
+	nsresult rv;
+
 	mTimer = do_CreateInstance("component://netscape/timer", &rv);
 	if (NS_FAILED(rv) || (!mTimer)) return;
 	mTimer->Init(InternetSearchDataSource::FireTimer, search,
@@ -3060,8 +3062,8 @@ InternetSearchDataSource::MapEncoding(const nsString &numericEncoding, nsString
 		{	nsnull, nsnull		}
 	};
 
-	// make "UTF-8" as the default
-	stringEncoding.AssignWithConversion("UTF-8");
+	// make "ISO-8859-1" as the default (not "UTF-8")
+	stringEncoding= NS_ConvertASCIItoUCS2("ISO-8859-1");
 
 	PRUint32	loop = 0;
 	while (encodingList[loop].numericEncoding != nsnull)