627357 - Scroll indicators are not there for the scrolled iframe when scrolling content inside an iframe

Status: RESOLVED WONTFIX

(Firefox for Android Graveyard :: Panning/Zooming, defect)

Description

[Martijn Wargers (dead)]

Attachment: testcase

See testcase, the iframe content is overflowing, so you should be able to scroll the iframe content by panning it.

Not sure when this regressed, but I can even see this bug on a 2010-08-17 build on Windows Fennec.

Comment 1

[Mark Finkle (:mfinkle) (use needinfo?)]

Bug only recently (yesterday?) enabled scrolling in iframes. This has been broken forever.

Comment 2

[Martijn Wargers (dead)]

I'm not sure what you mean. You mean this has been broken forever on Fennec2.0 builds? With Fennec1.1 builds, this seems to work fine.

Comment 3

[Mark Finkle (:mfinkle) (use needinfo?)]

(In reply to comment #2)
> I'm not sure what you mean. You mean this has been broken forever on Fennec2.0
> builds? With Fennec1.1 builds, this seems to work fine.

Right. This has been broken ever since we implemented e10s.

Comment 4

[Martijn Wargers (dead)]

Ah, yeah, I see it working in today's trunk build (what a coincidence!), but the scroll indicators are at the wrong place.

Comment 5

[Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)]

(In reply to comment #4)
> Ah, yeah, I see it working in today's trunk build (what a coincidence!), but
> the scroll indicators are at the wrong place.

bugmorph?

Comment 6

[Martijn Wargers (dead)]

Ok, steps to reproduce:
- Visit testcase, pan the iframe of the testcase upwards to let the content of the iframe scroll downwards

Expected result:
- A vertical scroll indicator should appear in the iframe, indicating the scroll position of the iframe

Actual result:
- A vertical scroll indicator appears at the right side of the viewport.

Comment 7

[Benjamin Stover (:stechz)]

Bugmorph!

Comment 8

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

odd.  I don't see any scroll indicators for : http://www.htmlcodetutorial.com/frames/_IFRAME.html

Comment 10

[Mark Finkle (:mfinkle) (use needinfo?)]

Is this really needed?

Comment 11

[Mark Finkle (:mfinkle) (use needinfo?)]

Madhava - do we really want scroll indicators inside iframes? I haven't seen any other mobile browsers do this. We could try. It might not be easy.

What's the priority of this feature?

Comment 12

[Martijn Wargers (dead)]

Attachment: testcase2

This testcase also has a scrollable div under the iframe, for which the situation is basically the same.

Comment 13

[Martijn Wargers (dead)]

Btw, I noticed that the stock Android browser just ignores any height (and perhap also width?) set for the iframe. It makes the iframe just as high so there is nothing to scroll. The scrollable div is just not scrollable, there.

Opera Mobile does the same and also for the scrollable div.

Maybe this would be a good solution for Fennec too?

Comment 14

[Mark Finkle (:mfinkle) (use needinfo?)]

REOPEN if needed

Comment 15

[Antonio Gomes (tonikitoo)]

(In reply to comment #13)
> Btw, I noticed that the stock Android browser just ignores any height (and
> perhap also width?) set for the iframe. It makes the iframe just as high so
> there is nothing to scroll. The scrollable div is just not scrollable, there.
> 
> Opera Mobile does the same and also for the scrollable div.
> 
> Maybe this would be a good solution for Fennec too?

It is a WebKit feature called Frame Flattening.