522335 - Fennelectrolysis: nsFrameLoader::TryNewProcess fails because ancestorView->GetFirstChild is null

Status: RESOLVED INVALID

(Core :: IPC, defect)

Description

[Benjamin Smedberg]

Trying to get Fennec + Electrolysis displaying something using joe's asycDrawXULElement patch. After much other interestingness, nsFrameLoader::TryNewProcess is failing because the call ancestorView->GetFirstChild is returning null.

This is really high priority. Assigning to me for the moment, but I don't know my way around this code very well and could use some help.

I'm using attachment 406130 [details] [diff] [review] from bug 505847, plus additional patches for electrolysis and mobile-browser which I will attach. I'm testing on Linux currently; I'll try on Windows in a bit.

Comment 1

[Benjamin Smedberg]

http://mxr.mozilla.org/projects-central/source/electrolysis/content/base/src/nsFrameLoader.cpp#1178

To expand on comment 0, this is probably because the xul:browser element in Fennec is not displayed (visibility:hidden according to #mobile).

Comment 2

[Benjamin Smedberg]

Attachment: Remote xul:browser and hack things not to need a docshell (patch to electrolysis)

Comment 3

[Benjamin Smedberg]

Attachment: Patch to mobile-browser

This is patch against mobile-browser to not use any methods that require a docshell.

Comment 4

[Robert O'Callahan (:roc) (email my personal email if necessary)]

That inner view should have been created and inserted here:
http://mxr.mozilla.org/projects-central/source/electrolysis/layout/generic/nsFrameFrame.cpp#990

Comment 5

[Robert O'Callahan (:roc) (email my personal email if necessary)]

You should also check that your "ourFrame" is an nsSubdocumentFrame.

Comment 6

[Benjamin Smedberg]

Thanks. Part of commenting out the docshell stuff in nsSubDocumentFrame caused us not to create the view. I've rejiggered it and I'm getting closer.