Currently when I tap a result off of Google and hit the back button, the search results window is entirely zoomed in. Steps to reproduce i) http://google.ca ii) Search for 'toronto weather' iii) Tap the first result, 'Weather Forecast: Toronto, Ontario...' iv) Hit the browser back button. See screenshot -- LG Nexus 4 (Android 4.2.2) Nightly (03/20)

Kats - Looks like the viewport meta info is not being reset when going back (onpageshow/onpagehide). Any ideas?

I'm able to reproduce but I don't know offhand what's causing this. It looks like it is specific to some search results and not others. For example, when I searched for toronto weather as in comment #0, the first result has this problem but other results do not. (In reply to Mark Finkle (:mfinkle) from comment #1) > Kats - Looks like the viewport meta info is not being reset when going back > (onpageshow/onpagehide). What makes you say this? We don't do anything particular with onpageshow/onpagehide, and never have, as far as I know. So if this is related to onpageshow/onpagehide, then it may be a regression from core gecko code.

Kats, what is your plan for this?

This is fixed now. Bisecting nightlies and inbounds shows it was fixed by bug 856497.

Is there a subset of bug 858969 that we can take to resolve these zoom issues Chris?

Are you sure that just picking up the patch from 856497 doesn't fix the problem? That is what my bisection showed as the fix.

(In reply to Kartikaya Gupta (email:kats@mozilla.com) from comment #8) > Are you sure that just picking up the patch from 856497 doesn't fix the > problem? That is what my bisection showed as the fix. That's a huge patch, so we'd rather not take it in FF22 for the last 3 betas.

I'm thinking that this is likely caused by the changes in browser.js. A patch that would roll back the changes made for bug 716403 in this file ought to help fix this problem without causing any other issues (if the pref is enabled, odd behaviour may occur though) I'm on PTO for 2 weeks, I'm hoping that Margaret or Wes could handle this?

Just glancing at this, it seems like the patch in: https://bug856497.bugzilla.mozilla.org/attachment.cgi?id=734646 is pretty small. I'm going to build it and test locally. A backout of bug 716403 seems much larger to cause problems to me.

Sorry for the delay. Managed to run this through beta, and the patch does fix the problem for me. At this point, I'm more comfortable with the idea of taking the fix than backing out all of the dynamic toolbar and hoping that everything still works. I can try to reduce the patch down a bit to minimize its surface area, but I'm not sure how successful that will be.

wesj - there was a misunderstanding. I thought we were talking about patches from bug 858969, which bug 856497 is duped to. We can move forward with this small patch.

Bug 856497 has medium risk, which means we'd really like to get extra QA verification around the toolbar if/when this lands.

Aaron - can you verify? Thanks!

This is fixed on mozilla-beta (Fx22).