Bug 18101: Suppress upload file dialog proxy bypass (linux)

arthuredelstein · Sep 20, 2017 · f657d0c · f657d0c
1 parent 1eeaa47
commit f657d0c
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 13 deletions.
diff --git a/widget/gtk/nsFilePicker.cpp b/widget/gtk/nsFilePicker.cpp
@@ -170,7 +170,6 @@ NS_IMPL_ISUPPORTS(nsFilePicker, nsIFilePicker)
 nsFilePicker::nsFilePicker()
   : mSelectedType(0)
   , mRunning(false)
-  , mAllowURLs(false)
 #if (MOZ_WIDGET_GTK == 3)
   , mFileChooserDelegate(nullptr)
 #endif
@@ -239,13 +238,6 @@ nsFilePicker::InitNative(nsIWidget *aParent,
   mTitle.Assign(aTitle);
 }
 
-NS_IMETHODIMP
-nsFilePicker::AppendFilters(int32_t aFilterMask)
-{
-  mAllowURLs = !!(aFilterMask & filterAllowURLs);
-  return nsBaseFilePicker::AppendFilters(aFilterMask);
-}
-
 NS_IMETHODIMP
 nsFilePicker::AppendFilter(const nsAString& aTitle, const nsAString& aFilter)
 {
@@ -402,9 +394,9 @@ nsFilePicker::Open(nsIFilePickerShownCallback *aCallback)
                                           GTK_RESPONSE_ACCEPT,
                                           GTK_RESPONSE_CANCEL,
                                           -1);
-  if (mAllowURLs) {
-    gtk_file_chooser_set_local_only(GTK_FILE_CHOOSER(file_chooser), FALSE);
-  }
+
+  // Don't allow remote URLs.
+  gtk_file_chooser_set_local_only(GTK_FILE_CHOOSER(file_chooser), TRUE);
 
   if (action == GTK_FILE_CHOOSER_ACTION_OPEN || action == GTK_FILE_CHOOSER_ACTION_SAVE) {
     GtkWidget *img_preview = gtk_image_new();

diff --git a/widget/gtk/nsFilePicker.h b/widget/gtk/nsFilePicker.h
@@ -25,7 +25,6 @@ class nsFilePicker : public nsBaseFilePicker
 
   // nsIFilePicker (less what's in nsBaseFilePicker)
   NS_IMETHOD Open(nsIFilePickerShownCallback *aCallback) override;
-  NS_IMETHOD AppendFilters(int32_t aFilterMask) override;
   NS_IMETHOD AppendFilter(const nsAString& aTitle,
                           const nsAString& aFilter) override;
   NS_IMETHOD SetDefaultString(const nsAString& aString) override;
@@ -62,7 +61,6 @@ class nsFilePicker : public nsBaseFilePicker
   int16_t   mSelectedType;
   int16_t   mResult;
   bool      mRunning;
-  bool      mAllowURLs;
   nsCString mFileURL;
   nsString  mTitle;
   nsString  mDefault;