862602 - enable unlocking of not visible images in the current tab on Fennec

Status: RESOLVED FIXED

(Core :: Layout: Images, Video, and HTML Frames, defect)

Description

[Timothy Nikkel (:tnikkel)]

Bug 689623 was only enabled on desktop, with mobile and b2g being followups due to the notion of visibility being different there. We should adapt and generalize so we use less memory for images on Fennec.

Comment 1

[Timothy Nikkel (:tnikkel)]

Attachment: flip the pref

I did some basic testing of this. It seems to work fine on timecube, nytimes, slashdot, google, google images, imgur, facebook.

I also tested that it prevents us from OOMing on my favourite images page http://congressoamericano.blogspot.ca/p/fotos-do-congresso-americano-iii.html

google maps seems to be broken on nightly right now (with the pref on or off), but turning the pref on in aurora seems to work fine. It briefly seemed like turning the pref on made google maps on nightly worse, but now I see the same broken behaviour on nightly with the pref on or off.

kats, do you have any suggestions for further testing before enabling this for all nightly users?

Comment 2

[Kartikaya Gupta (email:[e-mail])]

Comment on attachment 808928 [details] [diff] [review]
flip the pref

Review of attachment 808928 [details] [diff] [review]:
-----------------------------------------------------------------

The only thing I'd want to explicitly ensure testing for beyond desktop are some mobile-specific scenarios involving the meta-viewport tag. I don't know how much of this is covered by your basic testing above, but I think the testing should include at least:
(1) a page with no meta-viewport tag,
(2) a page with a meta-viewport tag set to width=device-width (or something similarly small) but that actually has wide content so you can zoom out,
(3) a page that starts off with no meta-viewport tag and later adds one (i.e. goes from case 1 to case 2) and
(4) a page that starts off with a meta-viewport tag and removes it (i.e. goes from case 2 to case 1).

If all those behave well, then I'm all thumbs-up to get this landed! :)

Comment 3

[Timothy Nikkel (:tnikkel)]

Ok, I tested those cases. No problems detected.

Comment 4

[Timothy Nikkel (:tnikkel)]

https://hg.mozilla.org/integration/mozilla-inbound/rev/2c1d2d4368f7

Comment 5

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/2c1d2d4368f7