[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kfm-devel
Subject:    Re: khtml (fwd)
From:       Koos Vriezen <koos.vriezen () xs4all ! nl>
Date:       2002-02-23 16:00:05
[Download RAW message or body]



On Sat, 23 Feb 2002, Koos Vriezen wrote:

> > 	If i use Konq to view the web page of the National Bank of Canada
> > (www.nbc.ca, or www.bnc.ca) I eather get konq freezing and then crashing on
> > me (kde 3.0 beta 2) or it just shows nothing (kde 2.2.2). Now, back when kde
> > 2.0 came out it could show the page just fine, but after 2.1 and newer, it
> > could no longer show the page. Now, trying to track down the exact bug, i
> > used mozilla (that  can view the web page btw) to download the front page
> > onto my hard drive so i could do some testing. When i tried to view the page
> > from my harddrive with Kong, it viewed fine (besides a little wiredness like
> > duplicated banners .. but that is the fault of mozilla i think). The reason i
> > am bringing this to your attention is this. My girlfriend is the internet
> > consultant for the National Bank, and she is trying VERY hard to have kde
> > supported by the bank (and that would be toooo cool!) , But its kind of hard
> > to do when the main page for the bank will not ever show correctly.
>
> It seems there is a recursion in body.onLoad.

Besides the wrong html, window events aren't clear in a
document.open/clear call. This should fix it:

--- xml/dom_docimpl.cpp 2002/02/16 13:58:15     1.162
+++ xml/dom_docimpl.cpp 2002/02/23 15:52:02
@@ -1021,6 +1021,9 @@ void DocumentImpl::clear()
     m_tokenizer = 0;

     removeChildren();
+    QPtrListIterator<RegisteredEventListener> it(m_windowEventListeners);
+    for (; it.current();)
+        m_windowEventListeners.removeRef(it.current());
 }

 void DocumentImpl::setStyleSheet(const DOM::DOMString &url, const
DOM::DOMString &sheet)

This also renders http://www.nbc.ca/ (after I pressed reload).


>
> I've looked at the main html page and IMHO it is not correct:
>
> <SCRIPT LANGUAGE=javascript>
> function loadframe() {
>         .....
>         document.open();
>         document.write(pag);
>         document.close();
> }
> </SCRIPT>
> <BODY onload="javascript:loadframe();">
> </BODY>
>
> First, the pag string doesn't contain a <html></html> tags.
> Second, document.open overwrites the whole document including the
> javascript code. IIRC IE can have problems with this, showing a
> background wait cursor (arrow with a small hourglass).
> I think this main page should be rewritten to:
>
> function loadframe() {
>   	.....
> 	mainframe.document.open();
>         mainframe.document.write(pag);
>         mainframe.document.close();
> }
> </SCRIPT>
> <FRAMESET onload="javascript:loadframe();">
>   <FRAME NAME='mainframe' SRC="about:blank">
> </FRAMESET>
> </HTML>
>
> I've attached the adjusted page. Please try.
>
> Koos
>

[prev in list] [next in list] [prev in thread] [next in thread]

Configure | About | News | Add a list | Sponsored by KoreLogic