Bug 1369327 - Making reader view users uniform when 'privacy.resistFingerprinting' is true draft
authorJonathan Hao <jhao@mozilla.com>
Wed, 21 Jun 2017 15:11:05 +0800
changeset 597961 3b9a6844ac565fa0ea4b03ed3ab03b7ff90a3b3c
parent 589301 2c6289f56812c30254acfdddabcfec1e149c0336
child 634372 1df64304df975b54d9d4be7eca9e9854fd20a0e6
push id65091
push userbmo:jhao@mozilla.com
push dateWed, 21 Jun 2017 07:24:20 +0000
bugs1369327
milestone55.0a1
Bug 1369327 - Making reader view users uniform when 'privacy.resistFingerprinting' is true MozReview-Commit-ID: 4Bx9Boj2l4L
toolkit/components/reader/ReaderMode.jsm
--- a/toolkit/components/reader/ReaderMode.jsm
+++ b/toolkit/components/reader/ReaderMode.jsm
@@ -65,21 +65,22 @@ this.ReaderMode = {
 
   get isOnLowMemoryPlatform() {
     let memory = Cc["@mozilla.org/xpcom/memory-service;1"].getService(Ci.nsIMemory);
     delete this.isOnLowMemoryPlatform;
     return this.isOnLowMemoryPlatform = memory.isLowMemoryPlatform();
   },
 
   _getStateForParseOnLoad() {
+    let resistFingerprinting = Services.prefs.getBoolPref("privacy.resistFingerprinting");
     let isEnabled = Services.prefs.getBoolPref("reader.parse-on-load.enabled");
     let isForceEnabled = Services.prefs.getBoolPref("reader.parse-on-load.force-enabled");
     // For low-memory devices, don't allow reader mode since it takes up a lot of memory.
     // See https://bugzilla.mozilla.org/show_bug.cgi?id=792603 for details.
-    return isForceEnabled || (isEnabled && !this.isOnLowMemoryPlatform);
+    return isForceEnabled || (!resistFingerprinting && isEnabled && !this.isOnLowMemoryPlatform);
   },
 
   observe(aMessage, aTopic, aData) {
     switch (aTopic) {
       case "nsPref:changed":
         if (aData.startsWith("reader.parse-on-load.")) {
           this.isEnabledForParseOnLoad = this._getStateForParseOnLoad();
         } else if (aData === "reader.parse-node-limit") {