953239 - Embedded YouTube videos cut off and pannable

Status: RESOLVED DUPLICATE of bug 911574

(Firefox for Android Graveyard :: Toolbar, defect)

Description

[Brad Lassey [:blassey] (use needinfo?)]

Attachment: tmp_Screenshot_2013-12-27-02-35-51-1162956996.png

Comment 1

[Daniel Holbert [:dholbert] (PTO until 2/24)]

I ran into this with an embedded YouTube video yesterday, too (though at the time, I wasn't sure whether it was a legitimately embedded youtube video, or some custom 3rd-party-site hackery).

Comment 2

[Daniel Holbert [:dholbert] (PTO until 2/24)]

(Though in my case, I don't think the video was actually cut off -- the full poster frame was visible, and when I clicked and dragged the poster upwards, it panned the poster out of view and a black region into view (into the video's clip-region).)

Comment 3

[Brad Lassey [:blassey] (use needinfo?)]

tracking 29 as a place-holder. Please renom once we have a regression window so we can track for the appropriate release

Comment 4

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

Not sure who's supposed to be getting the regression window here since it's in the Layout component but manifests (I assume) only in Fennec. Aaron, do you know?

Comment 5

[Aaron Train [:aaronmt]]

It's unclear that this is a regression.

Comment 6

[Aaron Train [:aaronmt]]

Flaviu, can you check to see if you can reproduce and determine whether this is a regression?

Comment 7

[Brad Lassey [:blassey] (use needinfo?)]

(In reply to Kartikaya Gupta (email:[e-mail]) from comment #4)
> Not sure who's supposed to be getting the regression window here since it's
> in the Layout component but manifests (I assume) only in Fennec. Aaron, do
> you know?

Sounds like it could be related to bug 943206, which is reproducible on Mac

Comment 8

[Flaviu Cos]

I can not find the regression range. The bug is reproducible even on 2011 builds.
Website I tested on: readwrite.com/2012/09/02/top_10_youtube_videos_of_all_time 
Device: Samsung Galaxy Nexus (Android 4.2.1).

Comment 9

[Sylvestre Ledru [:Sylvestre]]

Not a regression, so, not tracking for now.
Please resubmit if you disagree.

Comment 10

[Brad Lassey [:blassey] (use needinfo?)]

(In reply to Sylvestre Ledru [:sylvestre] from comment #9)
> Not a regression, so, not tracking for now.
> Please resubmit if you disagree.

This has existed on Android for a long time (perhaps forever), but Mac only started seeing this behavior in 29. Since bug 943206 has better STR for Mac, I nominated that for tracking 29 instead.

Comment 11

[Sylvestre Ledru [:Sylvestre]]

OK, bug 943206 is worth tracking. Thanks for bringing to my attention.

Comment 13

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

This bug spawned bug 943206 which got really confusing because it conflated things on Mac with things on Fennec, so then I forked the Fennec issue from that into bug 988991. After some investigation it turned out the issue was not as general as I thought, and is quite specific to iframes that have overflow:hidden which Fennec doesn't respect. So I duped that bug back here, and put together a patch that fixes this issue as well as bug 911574 and it is posted on bug 911574.

Comment 15

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

Should be fixed by bug 911574. Please reopen if that is not the case.

Comment 16

[Kevin Brosnan [Ex-Mozilla]]

Verify this dupe

Comment 17

[Flaviu Cos]

Verified as fixed in builds:
- 31.0a1 (2014-04-09);
Device: Samsung Galaxy Nexus (Android 4.2.1);

The bug is still reproducible on builds:
- 30.0a2 (2014-04-09);
- 29 beta 6;
- 28.