Fix encoding of saved content; fixes bug #141050.

2004-04-26  Marco Pesenti Gritti  <marco@gnome.org>

	* embed/mozilla/MozDownload.cpp:

	Fix encoding of saved content; fixes bug #141050.

2 files changed, 29 insertions, 34 deletions

diff --git a/ChangeLog b/ChangeLog
index a89660b3e..a3115e0a1 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,9 @@
+2004-04-26  Marco Pesenti Gritti  <marco@gnome.org>
+
+	* embed/mozilla/MozDownload.cpp:
+
+	Fix encoding of saved content; fixes bug #141050.
+
 2004-04-23  Christian Persch  <chpe@cvs.gnome.org>
 
 	* data/epiphany-1.4.pc.in:
diff --git a/embed/mozilla/MozDownload.cpp b/embed/mozilla/MozDownload.cpp
index e548dd86b..722e7d8a5 100644
--- a/embed/mozilla/MozDownload.cpp
+++ b/embed/mozilla/MozDownload.cpp
@@ -465,7 +465,7 @@ nsresult InitiateMozillaDownload (nsIDOMDocument *domDocument, nsIURI *sourceURI
 	EmbedPersistFlags ephy_flags;
 	ephy_flags = ephy_embed_persist_get_flags (EPHY_EMBED_PERSIST (embedPersist));
 
-	PRBool isHTML = (domDocument && contentType &&
+	PRBool isHTML = (contentType &&
 			(strcmp (contentType, "text/html") == 0 ||
 			 strcmp (contentType, "text/xml") == 0 ||
 			 strcmp (contentType, "application/xhtml+xml") == 0));
@@ -486,7 +486,7 @@ nsresult InitiateMozillaDownload (nsIDOMDocument *domDocument, nsIURI *sourceURI
 
 	PRInt32 flags = nsIWebBrowserPersist::PERSIST_FLAGS_REPLACE_EXISTING_FILES;
 
-	if (!isHTML)
+	if (!domDocument && !isHTML && !(ephy_flags & EMBED_PERSIST_COPY_PAGE))
 	{
 		flags |= nsIWebBrowserPersist::PERSIST_FLAGS_NO_CONVERSION;
 	}
@@ -496,51 +496,40 @@ nsresult InitiateMozillaDownload (nsIDOMDocument *domDocument, nsIURI *sourceURI
 	}
 	webPersist->SetPersistFlags(flags);
 
-	if (!isHTML || ephy_flags & EMBED_PERSIST_COPY_PAGE)
+	if (!domDocument || !isHTML || ephy_flags & EMBED_PERSIST_COPY_PAGE)
 	{
 		rv = webPersist->SaveURI (sourceURI, aCacheKey, nsnull,
 					  postData, nsnull, inDestFile);
 	}
 	else
 	{
-		if (!domDocument) return rv;  /* should never happen */
-    
 		PRInt32 encodingFlags = 0;
 		nsCOMPtr<nsILocalFile> filesFolder;
 
-    		if (!contentType || strcmp (contentType, "text/plain") != 0)
-		{
-			/**
-			 * Construct a directory path to hold the associated files; mozilla
-			 * will create the directory as needed.
-			 */
-
-			filesFolder = do_CreateInstance("@mozilla.org/file/local;1");
-			nsAutoString unicodePath;
-			inDestFile->GetPath(unicodePath);
-			filesFolder->InitWithPath(unicodePath);
-      
-			nsAutoString leafName;
-			filesFolder->GetLeafName(leafName);
-			nsAutoString nameMinusExt(leafName);
-			PRInt32 index = nameMinusExt.RFind(".");
-			if (index >= 0)
-			{
-				nameMinusExt.Left(nameMinusExt, index);
-			}
+		/**
+		 * Construct a directory path to hold the associated files; mozilla
+		 * will create the directory as needed.
+		 */
 
-			nameMinusExt += NS_LITERAL_STRING (" ");
-			nameMinusExt += NS_ConvertUTF8toUTF16 (_("Files"));
- 
-			filesFolder->SetLeafName(nameMinusExt);
-		}
-		else
+		filesFolder = do_CreateInstance("@mozilla.org/file/local;1");
+		nsAutoString unicodePath;
+		inDestFile->GetPath(unicodePath);
+		filesFolder->InitWithPath(unicodePath);
+      
+		nsAutoString leafName;
+		filesFolder->GetLeafName(leafName);
+		nsAutoString nameMinusExt(leafName);
+		PRInt32 index = nameMinusExt.RFind(".");
+		if (index >= 0)
 		{
-			encodingFlags |= nsIWebBrowserPersist::ENCODE_FLAGS_FORMATTED |
-					 nsIWebBrowserPersist::ENCODE_FLAGS_ABSOLUTE_LINKS |
-					 nsIWebBrowserPersist::ENCODE_FLAGS_NOFRAMES_CONTENT;
+			nameMinusExt.Left(nameMinusExt, index);
 		}
 
+		nameMinusExt += NS_LITERAL_STRING (" ");
+		nameMinusExt += NS_ConvertUTF8toUTF16 (_("Files"));
+ 
+		filesFolder->SetLeafName(nameMinusExt);
+
 		rv = webPersist->SaveDocument (domDocument, inDestFile, filesFolder,
 					       contentType, encodingFlags, 80);
 	}