Bug 1369327 - Remove low memory platform checks in reader mode. r?Gijs draft
authorJonathan Hao <jhao@mozilla.com>
Wed, 19 Jul 2017 14:00:21 +0800
changeset 611028 400ed07def7f7f2598ed9ddb933d349096824eee
parent 610973 1b065ffd8a535a0ad4c39a912af18e948e6a42c1
child 638050 f3858ec0b9ea5366fae3b648081584e21859a344
push id69103
push userbmo:jhao@mozilla.com
push dateWed, 19 Jul 2017 06:01:05 +0000
reviewersGijs
bugs1369327
milestone56.0a1
Bug 1369327 - Remove low memory platform checks in reader mode. r?Gijs MozReview-Commit-ID: 1VUprtntsK0
toolkit/components/reader/ReaderMode.jsm
--- a/toolkit/components/reader/ReaderMode.jsm
+++ b/toolkit/components/reader/ReaderMode.jsm
@@ -58,28 +58,20 @@ this.ReaderMode = {
     delete this.isEnabledForParseOnLoad;
 
     // Listen for future pref changes.
     Services.prefs.addObserver("reader.parse-on-load.", this);
 
     return this.isEnabledForParseOnLoad = this._getStateForParseOnLoad();
   },
 
-  get isOnLowMemoryPlatform() {
-    let memory = Cc["@mozilla.org/xpcom/memory-service;1"].getService(Ci.nsIMemory);
-    delete this.isOnLowMemoryPlatform;
-    return this.isOnLowMemoryPlatform = memory.isLowMemoryPlatform();
-  },
-
   _getStateForParseOnLoad() {
     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 || isEnabled;
   },
 
   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") {