From 1bcf1dabffdacc0f0cbb73ed50d4428d86f7636c Mon Sep 17 00:00:00 2001
From: Arthur Edelstein <arthuredelstein@gmail.com>
Date: Wed, 20 Sep 2017 09:35:38 -0700
Subject: [PATCH] Bug 18101: Suppress upload file dialog proxy bypass (linux)

---
 widget/gtk/nsFilePicker.cpp | 14 +++-----------
 widget/gtk/nsFilePicker.h   |  2 --
 2 files changed, 3 insertions(+), 13 deletions(-)

diff --git a/widget/gtk/nsFilePicker.cpp b/widget/gtk/nsFilePicker.cpp
index 172cb44448494..d1ab963e10122 100644
--- 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
index 2b5042098c654..f7b1bf639bdda 100644
--- a/widget/gtk/nsFilePicker.h
+++ b/widget/gtk/nsFilePicker.h
@@ -25,7 +25,6 @@ public:
 
   // 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 @@ protected:
   int16_t   mSelectedType;
   int16_t   mResult;
   bool      mRunning;
-  bool      mAllowURLs;
   nsCString mFileURL;
   nsString  mTitle;
   nsString  mDefault;
-- 
GitLab