558108 - Reduce fuzzy page-fit-zoom threshold and make it a pref

Status: RESOLVED FIXED

(Firefox for Android Graveyard :: Panning/Zooming, defect)

Description

[Matt Brubeck (:mbrubeck)]

Attachment: patch

For bug 454456 we added a threshold of 0.2 to page zoom.  That is, we use the "page zoom" level only if the difference from 1.0 is more than 0.2.

After some use, this seems too big - it cuts off large portions of some pages, and leaves large blank areas on others.  This patch reduces the threshold from 0.2 to 0.1, and makes it configurable for further experimentation.

Because prefs can't be floating-point, the new pref is actually the inverse of the old constant.

Comment 1

[Mark Finkle (:mfinkle) (use needinfo?)]

pushed:
http://hg.mozilla.org/mobile-browser/rev/72bf95a744ee

Comment 2

[Aakash Desai [:aakashd]]

Hm, I'm seeing a lot of checkerboards when zooming out with this patch on various sites like mobile support and sqaforums.com using the n900 as well as the n810. Would this patch have something to do with that?

Comment 3

[Matt Brubeck (:mbrubeck)]

No, this shouldn't affect whether checkerboards are displayed.  (In my own quick test, I am seeing equal checkerboards on sqaforums.com with and without the patch.)