804808 - B2G could end up on a blurry page after zooming in on a page and then going to a link

Status: RESOLVED FIXED

(Core :: Graphics, defect, P4)

Description

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

Attachment: screenshot

## Environment :
Otoro phone, build 2012-10-23
Taken from default.xml in b2g-distro: 
* "platform_build" revision= db539a3bd139c93c09b0cd1c3f9396b74d68717c
* "gaia" revision= 0b8ec9b8c16429dc35453dbb7b9342fab3dd18fb 
* "releases-mozilla-aurora" revision= f58edfde05cb708f8a2c440d338f2e429aaf372b
* "gonk-misc" revision= db0c751715f4696515735eb1e0dc5df7a40eb81d

  
## Repro :
1. launch browser on b2g device
2. go to http://people.mozilla.com/~nhirata/html_tp
3. zoom in to be able to read
4. hit the parent directory at the top 

## Expected :
1. a none blurry page

## Actual :
1. blurry

## Note :
related to bug 795657?

Comment 2

[Stefan Arentz | :st3fan | ⏰ EST | he/him]

Just a quick not that this still happens in the 2012-11-28 build.

Comment 3

[Stefan Arentz | :st3fan | ⏰ EST | he/him]

Correction: My report was for the 2012-11-27 build.

Comment 4

[Anthony Jones (:ajones, :kentuckyfriedtakahe, :k17e)]

I can reproduce it using back/forward buttons.

Comment 5

[Anthony Jones (:ajones, :kentuckyfriedtakahe, :k17e)]

Attachment: Fixed missing reset of zoom

Comment 6

[Chris Jones [:cjones] inactive; ni?/f?/r? if you need me]

Comment on attachment 688569 [details] [diff] [review]
Fixed missing reset of zoom

PresShell is supposed to be doing this.  Are we not hitting that code on navigation?

Comment 7

[Anthony Jones (:ajones, :kentuckyfriedtakahe, :k17e)]

We don't hit that code using back/forward unless we are reloading the page.

Comment 8

[Chris Jones [:cjones] inactive; ni?/f?/r? if you need me]

Comment on attachment 688569 [details] [diff] [review]
Fixed missing reset of zoom

Right, we discussed this on IRC yesterday and I forgot to r+.  ftfy

Comment 9

[Mozilla RelEng Bot]

Try run for d89aeaa26d2d is complete.
Detailed breakdown of the results available here:
    https://tbpl.mozilla.org/?tree=Try&rev=d89aeaa26d2d
Results (out of 97 total builds):
    success: 84
    warnings: 13
Builds (or logs if builds failed) available at:
http://ftp.mozilla.org/pub/mozilla.org/firefox/try-builds/[e-mail]-d89aeaa26d2d

Comment 10

[Mozilla RelEng Bot]

Try run for d89aeaa26d2d is complete.
Detailed breakdown of the results available here:
    https://tbpl.mozilla.org/?tree=Try&rev=d89aeaa26d2d
Results (out of 98 total builds):
    success: 85
    warnings: 13
Builds (or logs if builds failed) available at:
http://ftp.mozilla.org/pub/mozilla.org/firefox/try-builds/[e-mail]-d89aeaa26d2d

Comment 11

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/f02032810972

Comment 12

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/f02032810972

Comment 13

[Chris Jones [:cjones] inactive; ni?/f?/r? if you need me]

Comment on attachment 688569 [details] [diff] [review]
Fixed missing reset of zoom

I don't have approval powers but I would approve this.

[Approval Request Comment]
Bug caused by (feature/regressing bug #):
 apzc
User impact if declined:
 back/forward navigation is always messed up
Testing completed (on m-c, etc.):
 manual testing by ajones
Risk to taking this patch (and alternatives if risky):
 zero risk for non-b2g.  \epsilon risk for b2g.
String or UUID changes made by this patch:
 none

Comment 14

[Andrew Overholt [:overholt]]

While we should fix this (and Jonas will approve for uplift), due to the fact that this is easily worked around, we decided during triage we wouldn't block on it.

Comment 15

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-aurora/rev/711f9df3e7bb
https://hg.mozilla.org/releases/mozilla-beta/rev/660f08b60719

Comment 16

[Chris Jones [:cjones] inactive; ni?/f?/r? if you need me]

Oh drat, existing approval-beta+'s for blocking-basecamp bugs need to be going on b2g18 now.

Alex, do you have any bugzilla ninjitsu that could help fix this?

This patch has zero risk on beta so I'll leave it there and uplift to b2g18.

Comment 17

[Chris Jones [:cjones] inactive; ni?/f?/r? if you need me]

https://hg.mozilla.org/releases/mozilla-b2g18/rev/fa9d493d6056

Comment 18

[Ryan VanderMeulen [:RyanVM]]

In the future, don't bother landing separately on b2g18 if it's already on beta. We're merging beta to b2g18 regularly anyway.
https://hg.mozilla.org/releases/mozilla-b2g18/rev/660f08b60719

Comment 19

[Mozilla RelEng Bot]

Try run for 242cbc75ebd3 is complete.
Detailed breakdown of the results available here:
    https://tbpl.mozilla.org/?tree=Try&rev=242cbc75ebd3
Results (out of 83 total builds):
    success: 57
    warnings: 25
    failure: 1
Builds (or logs if builds failed) available at:
http://ftp.mozilla.org/pub/mozilla.org/firefox/try-builds/[e-mail]-242cbc75ebd3

Comment 20

[Mozilla RelEng Bot]

Try run for d89aeaa26d2d is complete.
Detailed breakdown of the results available here:
    https://tbpl.mozilla.org/?tree=Try&rev=d89aeaa26d2d
Results (out of 99 total builds):
    exception: 1
    success: 85
    warnings: 13
Builds (or logs if builds failed) available at:
http://ftp.mozilla.org/pub/mozilla.org/firefox/try-builds/[e-mail]-d89aeaa26d2d