Always save a page's content if there i content to be saved. Remove the

2003-04-23  David Bordoley <bordoley@msu.edu>

	* embed/ephy-embed-single.c: (ephy_embed_single_show_file_picker):
	* embed/ephy-embed-single.h:
	* embed/ephy-embed-utils.c: (ephy_embed_utils_save):
	* embed/ephy-embed-utils.h:
	* embed/mozilla/FilePicker.cpp: (GFilePicker::GFilePicker),
	(GFilePicker::Show), (GFilePicker::HandleFilePickerR):
	* embed/mozilla/FilePicker.h:
	* embed/mozilla/mozilla-embed-persist.cpp: (impl_save):
	* embed/mozilla/mozilla-embed-single.cpp: (impl_show_file_picker):
	* src/window-commands.c: (window_cmd_file_open):

	Always save a page's content if there i content to be saved.
	Remove the checkbox from the save as dialog.

	Patch from Christian Persch <chpe+gnomebugz@stud.uni-saarland.de>

4 files changed, 7 insertions, 52 deletions

diff --git a/embed/mozilla/FilePicker.cpp b/embed/mozilla/FilePicker.cpp
index 8ae5429f7..fb8a82313 100644
--- a/embed/mozilla/FilePicker.cpp
+++ b/embed/mozilla/FilePicker.cpp
@@ -76,9 +76,7 @@
 /* Implementation file */
 NS_IMPL_ISUPPORTS1(GFilePicker, nsIFilePicker)
 
-GFilePicker::GFilePicker(PRBool aShowContentCheck, FileFormat *aFileFormats) :
-			mShowContentCheck(aShowContentCheck),
-			mSaveContentCheck(NULL),
+GFilePicker::GFilePicker(FileFormat *aFileFormats) :
 			mFileFormats(aFileFormats)
 {
 	NS_INIT_ISUPPORTS();
@@ -246,24 +244,6 @@ NS_IMETHODIMP GFilePicker::Show(PRInt16 *_retval)
 		gtk_window_set_transient_for(GTK_WINDOW(mFileSelector),
 					     GTK_WINDOW(mParentWidget));
 
-	if (mShowContentCheck)
-	{
-		GtkWidget *bbox = gtk_hbutton_box_new ();
-		gtk_button_box_set_layout(GTK_BUTTON_BOX(bbox),
-					  GTK_BUTTONBOX_END);
-		gtk_box_set_spacing(GTK_BOX(bbox), 0);
-        	gtk_box_pack_end(GTK_BOX(GTK_FILE_SELECTION(mFileSelector)->action_area),
-				 bbox, TRUE, TRUE, 0);
-
-		mSaveContentCheck = 
-			gtk_check_button_new_with_label(_("Save with content"));
-
-		gtk_box_pack_start(GTK_BOX(bbox), mSaveContentCheck,
-				   FALSE, FALSE, 0);
-
-		gtk_widget_show_all(bbox);
-	}
-
 	if (mFileFormats)
 	{
 		mFormatChooser = gtk_option_menu_new();
@@ -482,11 +462,7 @@ NS_METHOD GFilePicker::HandleFilePickerResult(PRInt16 *retval)
 		}
 	}
 
-	if (GTK_IS_TOGGLE_BUTTON(mSaveContentCheck))
-		*retval = gtk_toggle_button_get_active(GTK_TOGGLE_BUTTON(mSaveContentCheck)) ?
-			  returnOKSaveContent : returnOK;
-	else
-		*retval = returnOK;
+	*retval = returnOK;
 
 	return NS_OK;
 }
diff --git a/embed/mozilla/FilePicker.h b/embed/mozilla/FilePicker.h
index 8f8fdf571..c478ce7a1 100644
--- a/embed/mozilla/FilePicker.h
+++ b/embed/mozilla/FilePicker.h
@@ -52,11 +52,9 @@ class GFilePicker : public nsIFilePicker
 	NS_DECL_NSIFILEPICKER
 	enum {  returnOK = nsIFilePicker::returnOK,
 		returnCancel = nsIFilePicker::returnCancel,
-		returnReplace = nsIFilePicker::returnReplace,
-		returnOKSaveContent = 256 };
+		returnReplace = nsIFilePicker::returnReplace };
 
-	GFilePicker(PRBool aShowContentCheck = PR_FALSE,
-		    FileFormat *aFileFormats = nsnull);
+	GFilePicker(FileFormat *aFileFormats = nsnull);
 	virtual ~GFilePicker();
 
 	/* additional members */
@@ -80,12 +78,9 @@ class GFilePicker : public nsIFilePicker
 
 	PRInt16 mMode;
 
-	PRBool mShowContentCheck;
-
 	GtkWidget *mParentWidget;	
 	GtkWidget *mFileSelector;
 	GtkWidget *mFormatChooser;
-	GtkWidget *mSaveContentCheck;
 
 	FileFormat *mFileFormats;
 };
diff --git a/embed/mozilla/mozilla-embed-persist.cpp b/embed/mozilla/mozilla-embed-persist.cpp
index 2b19bcc69..5668e14fb 100644
--- a/embed/mozilla/mozilla-embed-persist.cpp
+++ b/embed/mozilla/mozilla-embed-persist.cpp
@@ -190,7 +190,7 @@ impl_save (EphyEmbedPersist *persist)
         if (flags & EMBED_PERSIST_SAVE_CONTENT)
 	{
 		char *datapath;
-		datapath = g_strconcat (filename, "content", NULL);
+		datapath = g_strconcat (filename, ".content", NULL);
                 NS_NewLocalFile(NS_ConvertUTF8toUCS2(datapath), PR_TRUE, getter_AddRefs(path));
 		g_free (datapath);
 	}
diff --git a/embed/mozilla/mozilla-embed-single.cpp b/embed/mozilla/mozilla-embed-single.cpp
index 20ccdfdca..355e33583 100644
--- a/embed/mozilla/mozilla-embed-single.cpp
+++ b/embed/mozilla/mozilla-embed-single.cpp
@@ -108,7 +108,6 @@ impl_show_file_picker (EphyEmbedSingle *shell,
 		       const char *file, 
 		       FilePickerMode mode,
                        char **ret_fullpath, 
-		       gboolean *ret_save_content, 
                        FileFormat *file_formats, 
 		       int *ret_file_format);
 
@@ -887,20 +886,12 @@ impl_show_file_picker (EphyEmbedSingle *shell,
 		       const char *file, 
 		       FilePickerMode mode,
                        char **ret_fullpath, 
-		       gboolean *ret_save_content, 
                        FileFormat *file_formats, 
 		       int *ret_file_format)
 {
-	PRBool showContentCheck;
-        gchar *expanded_directory;
+	gchar *expanded_directory;
 
-        if (ret_save_content == NULL)
-                showContentCheck = PR_FALSE;
-        else
-                showContentCheck = PR_TRUE;
-
-        GFilePicker *filePicker = new GFilePicker (showContentCheck, 
-                                                   file_formats);
+        GFilePicker *filePicker = new GFilePicker (file_formats);
 
 	/* FIXME sane path: expand tilde ... */
         expanded_directory = g_strdup (directory);
@@ -926,13 +917,6 @@ impl_show_file_picker (EphyEmbedSingle *shell,
         PRInt16 retval;
         filePicker->Show (&retval);
 
-        if (ret_save_content != NULL)
-        {
-                if (retval == GFilePicker::returnOKSaveContent)
-                        *ret_save_content = TRUE;
-                else
-                        *ret_save_content = FALSE;
-        }
         if (ret_file_format != NULL)
         {
                 *ret_file_format = filePicker->mSelectedFileFormat;