Bug 1400489: Restrict global sharing to desktop Firefox. r?tcampbell draft
authorKris Maglione <maglione.k@gmail.com>
Fri, 15 Sep 2017 21:24:36 -0700
changeset 665881 2767b054ed6c3b0bce1b7368023edfed6fd86e49
parent 665880 b020722d08c09d44e64c3f89a0108820de76b676
child 731920 eb3e64d069eae207c514eac669c0c97d50d31792
push id80212
push usermaglione.k@gmail.com
push dateSat, 16 Sep 2017 04:24:53 +0000
reviewerstcampbell
bugs1400489
milestone57.0a1
Bug 1400489: Restrict global sharing to desktop Firefox. r?tcampbell MozReview-Commit-ID: Gese366rNCH
browser/app/profile/firefox.js
modules/libpref/init/all.js
--- a/browser/app/profile/firefox.js
+++ b/browser/app/profile/firefox.js
@@ -506,16 +506,18 @@ pref("browser.bookmarks.openInTabClosesM
 
 // Scripts & Windows prefs
 pref("dom.disable_open_during_load",              true);
 pref("javascript.options.showInConsole",          true);
 #ifdef DEBUG
 pref("general.warnOnAboutConfig",                 false);
 #endif
 
+pref("jsloader.shareGlobal", true);
+
 // This is the pref to control the location bar, change this to true to
 // force this - this makes the origin of popup windows more obvious to avoid
 // spoofing. We would rather not do it by default because it affects UE for web
 // applications, but without it there isn't a really good way to prevent chrome
 // spoofing, see bug 337344
 pref("dom.disable_window_open_feature.location",  true);
 // prevent JS from setting status messages
 pref("dom.disable_window_status_change",          true);
--- a/modules/libpref/init/all.js
+++ b/modules/libpref/init/all.js
@@ -5245,17 +5245,17 @@ pref("dom.idle-observers-api.fuzz_time.d
 // they are handled separately. This pref is only read once at startup:
 // a restart is required to enable a new value.
 pref("network.activity.blipIntervalMilliseconds", 0);
 
 // If true, reuse the same global for (almost) everything loaded by the component
 // loader (JS components, JSMs, etc). This saves memory, but makes it possible
 // for the scripts to interfere with each other.  A restart is required for this
 // to take effect.
-pref("jsloader.shareGlobal", true);
+pref("jsloader.shareGlobal", false);
 
 // When we're asked to take a screenshot, don't wait more than 2000ms for the
 // event loop to become idle before actually taking the screenshot.
 pref("dom.browserElement.maxScreenshotDelayMS", 2000);
 
 // Whether we should show the placeholder when the element is focused but empty.
 pref("dom.placeholder.show_on_focus", true);