Closed Bug 852417 Opened 12 years ago Closed 12 years ago

Add a force override preference for Reader Mode availability on low-mem devices

Categories

(Firefox for Android Graveyard :: Reader View, defect)

20 Branch
All
Android
defect
Not set
normal

Tracking

(firefox20 affected, firefox21 affected, firefox22 affected, firefox23 verified, firefox24 verified)

VERIFIED FIXED
Firefox 23
Tracking Status
firefox20 --- affected
firefox21 --- affected
firefox22 --- affected
firefox23 --- verified
firefox24 --- verified

People

(Reporter: bgvaughan, Assigned: kats)

References

Details

Attachments

(1 file)

User Agent: Mozilla/5.0 (X11; Linux x86_64; rv:19.0) Gecko/20100101 Firefox/19.0 Build ID: 20130311191316 Steps to reproduce: Visited pages, such as news sites, that had clearly defined body text, but considerable clutter, with the release (19), Beta (20), and Aurora (21) versions of Firefox Mobile. For example, http://www.cnn.com/2013/03/18/world/americas/canada-prison-escape/index.html?hpt=hp_inthenews Actual results: For 20 and 21, the Reader Mode icon does not appear in the location bar when page finishes loading. Encapsulating the URL, as with "about:reader?url=www.example.com&readingList=1", led to a white page with a message, "Loading...", but that remains static, and doesn't error out. Expected results: When the page completed loading, the Reader Mode icon should have appeared in the location bar, and tapping it should have reloaded the page in reader mode. This worked correctly with the release (19) of Firefox Mobile.
Works for me as-is / no special encapsulation ... GS3 4.1.1 ... w/19.0, 20.0, nightly 22.01a-3/17.
What device are you using? Reader mode is disabled on low-memory platforms, the threshold for which is around ~384 megabytes of RAM (as reported by /proc/meminfo).
I'm using a T-Mobile G2; i.e., http://wiki.cyanogenmod.org/w/Vision_Info $ cat /proc/meminfo MemTotal: 376792 kB MemFree: 8976 kB Buffers: 22676 kB Cached: 141320 kB SwapCached: 0 kB Active: 143416 kB Inactive: 191624 kB Active(anon): 79400 kB Inactive(anon): 92272 kB Active(file): 64016 kB Inactive(file): 99352 kB Unevictable: 268 kB Mlocked: 0 kB SwapTotal: 0 kB SwapFree: 0 kB Dirty: 16 kB Writeback: 0 kB AnonPages: 171324 kB Mapped: 44000 kB Shmem: 360 kB Slab: 10608 kB SReclaimable: 3364 kB SUnreclaim: 7244 kB KernelStack: 4640 kB PageTables: 10764 kB NFS_Unstable: 0 kB Bounce: 0 kB WritebackTmp: 0 kB CommitLimit: 188396 kB Committed_AS: 1860720 kB VmallocTotal: 262144 kB VmallocUsed: 107612 kB VmallocChunk: 112644 kB
Yeah, reader mode is going to be disabled on your device, since MemTotal is less than 393216 kB. If you feel like your device is up to it, we could put in a pref to force-enable it. Based on the "MemFree" value in your output though I don't think you'll have a good experience with reader mode. :/
Well, in my case anyway, I did have the use of Reader Mode for a month or two at least on this same device, and performance was fine. I'm not sure why so much RAM is in use, though.
(In reply to Kartikaya Gupta (email:kats@mozilla.com) from comment #4) > Yeah, reader mode is going to be disabled on your device, since MemTotal is > less than 393216 kB. If you feel like your device is up to it, we could put > in a pref to force-enable it. Based on the "MemFree" value in your output > though I don't think you'll have a good experience with reader mode. :/ I'm more inclined to just offer the feature and work through performance then to deny it. Do we have any active bugs being worked on that are fine-tuning reader-mode?
I don't believe anybody is working on reader mode performance at the moment. I think that depending on how people use Firefox on low-mem devices (e.g. number of tabs open at any given time) this might impact some people more than others, so I'm a bit hesitant to turn it on unconditionally.
Blocks: 792603
Status: UNCONFIRMED → NEW
Ever confirmed: true
So this is the same as bug 828124 which was checked into 19 beta at the time but I guess did not take affect since bug 792603 was 20 only.
Not sure what resolution to close this as tbh
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → WONTFIX
Reader Mode is still disabled in the latest Firefox 21 beta with my device (Xperia Ray), whereas it worked fine in earlier versions of Firefox (16-19). Denying this feature for phones with small high-res screens (for example) is a big step backwards in functionality, in my opinion. For the record, I never noticed any performance-related issues with Reader Mode in previous versions.
Likewise. I've been puzzled in that part of the utility of Reader Mode was that in eliminating most of the graphics, scripts, etc., on a page, the browser seemed to become more responsive. I would have assumed that Reader Mode required less memory for displaying a Web page than the default display.
MemTotal reported by my phone is: 343396 KB. While this may seem low, Reader Mode appeared to run just fine beforehand. A hidden pref would probably be a decent compromise for people willing to use this feature regardless of limited memory resources. Though a normal (visible) preference would be better still.
Status: RESOLVED → REOPENED
Flags: needinfo?(mark.finkle)
Resolution: WONTFIX → ---
Attached patch Add a pref (deleted) — Splinter Review
Assignee: nobody → bugmail.mozilla
Attachment #744107 - Flags: review?(mark.finkle)
Flags: needinfo?(mark.finkle)
Attachment #744107 - Flags: review?(mark.finkle) → review+
OS: Linux → Android
Hardware: x86_64 → All
Summary: Reader Mode is unavailable in Beta (20) or Aurora (21) → Add a force override preference for Rreader Mode availability on low-mem devices
Summary: Add a force override preference for Rreader Mode availability on low-mem devices → Add a force override preference for Reader Mode availability on low-mem devices
Status: REOPENED → RESOLVED
Closed: 12 years ago12 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 23
I can confirm that Reader Mode works again in the latest nightly. Thanks!
I'm using the latest nightly, and Reader Mode is still not available. Checking about:config, there is no reader.force_allow setting. Adding this setting, setting it to TRUE, and reloading pages or restarting Firefox doesn't have any apparent effect.
My mistake. The name of the setting changed to: reader.parse-on-load.force-enabled When I toggled that to TRUE, Reader Mode is available and works as expected.
Status: RESOLVED → VERIFIED
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: