752947 - Flour missing from pancake recipe

Status: RESOLVED WORKSFORME

(Core :: Layout: Text and Fonts, defect, P1)

Description

[Eitan Isaacson [:eeejay]]

Attachment: Screenshots

Web page or screen you were on when you saw the issue: 
http://allrecipes.com/recipe/good-old-fashioned-pancakes/

Steps to reproduce:
1. Go to website with Fennec Aurora

What you expected: 
The top ingredient should be flour, but it does not appear. It was there for a second as the page was loading and then disappeared. Most pancakes require some form of flour :)

Crash report ID (if applicable):

Comment 1

[Aaron Train [:aaronmt]]

Thread summary and issue deliver!

This WFM on Nightly (05/08); text-size default at: Medium in Settings. Checking Aurora (05/08), I also don't see this.

Which phone, text-size and Aurora build are you running?  If anything, it sounds like a font-inflation issue, but I can't reproduce.

Comment 2

[Eitan Isaacson [:eeejay]]

14.a2 Aurora 2012-05-08
Samsung Galaxy S 2 running Cyanogenmod 9 nightly.

Comment 3

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

STR: 
1. go to http://allrecipes.com/recipe/good-old-fashioned-pancakes/
2. pinch zoom in.

Expected: all text remains on page
Actual: flour disappears

Plugins tap to play
text size medium

Note: if you set text size to tiny, flour will not disappear.
1. Galaxy S 2 running Android OS 2.3.3
2. Nightly 5/8/2012
3. also occurs on Aurora 5/8/2012

Comment 4

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

nightly : 2/15 occurs
nightly : 3/15 occurs

Looks like this bug has been occurring for a while.

the website uses iframes, the ingredients doesn't look like it has anything special other than being placed in a div.

It's not because of just the div.  Possibly due to the position in iframes as well?  still trying to break it down.

Adding mw22 for breakdown of page.

Comment 5

[Kevin Brosnan [Ex-Mozilla]]

This happens on the Galaxy Nexus as well.

Comment 6

[Scott Johnson (:jwir3)]

Shortened link.

Comment 7

[Scott Johnson (:jwir3)]

Strange... if you don't tap to play the plugin, flour remains visible.

Comment 8

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

qawanted should not have been removed.  QA should break down the page.

Comment 9

[Chris Lord [:cwiiis]]

(In reply to Scott Johnson (:jwir3) from comment #7)
> Strange... if you don't tap to play the plugin, flour remains visible.

I see it on initial render, then it disappears when the page fully loads.

For the record, I also see odd disappearing paragraphs and images on the article pages of phoronix.com, under the same conditions (appear while loading, disappear once loading progresses).

Disabling font inflation fixes both sites for me.

Comment 10

[Vladimir Vukicevic [:vlad] [:vladv] (needinfo me, slow to respond)]

Flour shows up fine on today's nightly.  Theory is that it might have been fixed by bug 753742

Comment 11

[Chris Lord [:cwiiis]]

(In reply to Vladimir Vukicevic (:vlad) from comment #10)
> Flour shows up fine on today's nightly.  Theory is that it might have been
> fixed by bug 753742

This seems unlikely to me. All that patch did was force layer building on scroll frames with a display-port.

Comment 12

[Vladimir Vukicevic [:vlad] [:vladv] (needinfo me, slow to respond)]

Yup, which fixed a bunch of issues with overflow:hidden elements on mobile.  The element here has overflow:hidden set, as do some ancestor elements..

Comment 13

[Chris Lord [:cwiiis]]

(In reply to Vladimir Vukicevic (:vlad) from comment #12)
> Yup, which fixed a bunch of issues with overflow:hidden elements on mobile. 
> The element here has overflow:hidden set, as do some ancestor elements..

right, but for the patch to have made a difference, that overflow:hidden needed to occur on the root document node (the one we set the displayport on) - if it does, then fair enough :)

Comment 14

[Xiao Meng Wei :xwei]

Needs a testcase.

Comment 15

[Eitan Isaacson [:eeejay]]

Flour is back in Aurora! Looks like it may have been fixed.

Comment 16

[JP Rosevear [:jpr]]

Closing as per #15

Comment 17

[Scott Johnson (:jwir3)]

(In reply to Eitan Isaacson [:eeejay] from comment #15)
> Flour is back in Aurora! Looks like it may have been fixed.

It would be nice to know what fixed it, though. Any chance we might be able to get a bisect to see what caused this to appear again?

Comment 18

[Kevin Brosnan [Ex-Mozilla]]

This still reproduces with a current nightly (2012-5-24).

Comment 19

[Aaron Train [:aaronmt]]

I'm unable to reproduce on Nightly (05/29, Galaxy Nexus 4.0.4) on default 'minTwips' settings @ 120 ("Medium") all content pertaining to the recipe is shown in my viewport.

Comment 20

[Vladimir Vukicevic [:vlad] [:vladv] (needinfo me, slow to respond)]

(In reply to Kevin Brosnan [:kbrosnan] from comment #18)
> This still reproduces with a current nightly (2012-5-24).

What device do you see it on?

Comment 21

[Kevin Brosnan [Ex-Mozilla]]

Galaxy Nexus, today's nightly.

Comment 22

[David Bolter [:davidb] (NeedInfo me for attention)]

I can't recreate on my galaxy nexus running today's nightly (same font inflation minTwips as Aaron).

Comment 23

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

This seems likely to be a case where font inflation is just changing the sizes of things such that we trigger some other cause, rather than the actual cause.  It would be useful to have a more reliable testcase here.

Comment 24

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

In order to reproduce, you have to tap on the flash ad on the right and then zoom in.

STR: 
1. go to http://allrecipes.com/recipe/good-old-fashioned-pancakes/
2. tap on the plugin on the right
3. pinch zoom in.

Expected: all text remains on page
Actual: flour disappears

Plugins tap to play
text size medium

Comment 25

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

Trying to reduce the test case, I'm not so sure it's font inflation that's causing this issue any more.  I copied the html over to a people account in it's entire page and the issue is not occurring.  I verified that I can still reproduce the issue.

I think it may have something to do with the js script that's being used towards the bottom of the html page?  Not sure at this moment in time without looking closely at the html code.

Comment 26

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

Attachment: reduced webpage

Reduced the webpage, there's still some more that can be reduced.

Comment 27

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

Weird part was that it replicated when I used my Mac as the hosting webpage.  Using the bugzilla... it doesn't seem to replicate the issue?  Not sure what's going on.  Need to look at the scripts in the webpage closer I suppose.

Comment 28

[Ryan VanderMeulen [:RyanVM]]

Doesn't reproduce anymore AFAICT. Feel free to reopen if you still can.