574683 - Local XUL pages (about:config) are not rendering

Status: VERIFIED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

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

Attachment: patch

Bug 574260 removed too much. For local XUL pages (browser.contentDocument is not null) we need to keep code that fires MozScrolledAreaChanged or the content will not render.

See about:config

Works with this patch, which adds back just the minimum code. It also checks for a null contentDocument.

Comment 1

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

Comment on attachment 454039 [details] [diff] [review]
patch

>+    // Local XUL documents are not firing MozScrolledAreaChanged
>+    let doc = browser.contentDocument;
>+    if (doc && doc instanceof XULDocument) {
>+       let width = doc.documentElement.scrollWidth;
>+       let height = doc.documentElement.scrollHeight;
>+       BrowserView.Util.ensureMozScrolledAreaEvent(browser, width, height);
>+    }
>   },

The indentation looks strange.

Comment 2

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

fixed indent

pushed:
http://hg.mozilla.org/mobile-browser/rev/d34f915d9d6a

Comment 4

[Aakash Desai [:aakashd]]

verified FIXED on builds:

Mozilla/5.0 (X11; U; Linux armv71; Nokia N900; en-US; rv:2.0b2pre) Gecko/2010630 Namoroka/4.0b2pre Fennec/2.0a1pre


and latest-electrolysis-maemo5-gtk

Comment 5

[Dão Gottwald [:dao]]

bugspam