532076 - XUL iframe content does not repaint when scrolling

Status: RESOLVED INVALID

(Firefox for Android Graveyard :: General, defect)

Description

[testit]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; de; rv:1.9.1.3) Gecko/20090824 Firefox/3.5.3
Build Identifier: fennec-1.0b6pre.en-US.win32.zip from 01-Dec-2009 02:13

XUL iframe is not scrollable. See attached example.

In my opinion this should be classifed as "blocking 1.0" like the very similar bug 460678.

Reproducible: Always

Comment 1

[testit]

Attachment: Testcase

Comment 2

[Fabrice Desré]

I tested by referencing http://planet.mozilla.org/ in a XUL iframe. The HTML content is scrollable, but doesn't repaint has it should.

Comment 3

[Aakash Desai [:aakashd]]

I'm marking this as blocking-fennec? and upgrading the severity.

Comment 4

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

Stuart - I think this should block too

Comment 5

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

It's not just XUL iframes. I see repaint problems with some HTML iframes too.

See:
http://www.samisite.com/test-csb2nf/id43.htm (the Static 2 iframe)

Comment 6

[testit]

In my opinion this is a general problem with frames: XUL iframe, HTML iframe, HTML frame. I've tested these pages, on both scrolling didn't work:

HTML iframe: http://www.w3schools.com/TAGS/tryit.asp?filename=tryhtml_iframe
HTML frame: http://www.w3schools.com/tags/tryit.asp?filename=tryhtml_frame_navigation2

Comment 7

[Fabrice Desré]

I investigated this bug again. Here's what I found :
- HTML frameset are working in current m-b

- HTML iframe is broken because of the HTML scrolling support. I have a very simple patch for this : make http://mxr.mozilla.org/mobile-browser/source/chrome/content/browser.js#1372 be |if (htmlElement && (htmlElement.ownerDocument == target.ownerDocument) && htmlElement instanceof HTMLElement) {|

- XUL iframe is broken because elementFromPoint() in browser.js (http://mxr.mozilla.org/mobile-browser/source/chrome/content/browser.js#1289) does not recognize iframes in the XUL namespace. When adding a test for |element| to be {http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul}:iframe it still doesn't work because |element.contentDocument| is undefined - and I really don't know why...

Comment 8

[Aakash Desai [:aakashd]]

After a discussion with mfinkle, bcombee's iframe test: http://people.mozilla.org/~bcombee/fennec/iframe-test.html scrolls, albeit slowly, when a user is panning within the iframe. 


Yet, this problem happens to me when scrolling a list of google docs as well.

Comment 10

[Aakash Desai [:aakashd]]

This is invalid due to e10s. Filing bug https://bugzilla.mozilla.org/show_bug.cgi?id=586288 for the e10s-related issues.