So.. we still have this setting the advanced display settings in Fennec called "Text reflow". In the code this maps to the pref "browser.zoom.reflowOnZoom" which is supposed to control whether text is reflowed to fit the display width after a zoom. This is supposed to take effect after both a pinch-zoom and a double-tap-to-zoom. However, enabling this setting seems to have no effect on a variety of pages I tested. At least it doesn't do what I recall it doing back in the day, where it rewraps text into a narrow column, often in page-layout-breaking ways (although it did help quite a lot with not having to scroll horizontally). A number of bugs have been filed about this in the past (bug 1049063, bug 990821, etc.) which have been WONTFIXed. In general it seems like the feeling is that nobody has time to work on this feature and we can't really get it to the quality bar that we want. So I suggest killing it entirely and deleting the setting and relevant code from browser.js. My main motivation is that removing this code will help with the APZ migration as there will be less hooks into the existing Java-based panning/zooming code. After the APZ migration if we want to reimplement this feature we can do it in C++ with the new APZ code, but that may or may not ever happen.

mfinkle, what do you say?

Actually, never mind. The decision was already made back in bug 1049063, so I'm just going to go ahead and delete the browser.js code. Bug 1059014 is on file for removing the layout-side code but I don't care so much about that for now so I'll leave that bug to track that deletion.

Try push at https://treeherder.mozilla.org/#/jobs?repo=try&revision=3cfcd33516ea

Try is green (the Android 4.0 failures are expected - those jobs are hidden on m-c).

Comment on attachment 8656631 [details] [diff] [review] Delete stuff Looks like you have everything covered.