From kfm-devel  Thu Oct 30 01:22:54 2003
From: Dirk Mueller <mueller () kde ! org>
Date: Thu, 30 Oct 2003 01:22:54 +0000
To: kfm-devel
Subject: Re: document.write hang
X-MARC-Message: https://marc.info/?l=kfm-devel&m=106747700432101
MIME-Version: 1
Content-Type: multipart/mixed; boundary="--Boundary-00=_ufGo/sgcU36ehRn"


--Boundary-00=_ufGo/sgcU36ehRn
Content-Type: text/plain;
  charset="iso-8859-1"
Content-Transfer-Encoding: 7bit
Content-Disposition: inline

On Wednesday 29 October 2003 20:57, Koos Vriezen wrote:

> it hangs in RenderCanvas::docHeight() because
> layer == layer->nextSibling()

known, this should be the fix. somehow coolo was having problems with it 
though, I don't know which though, so I can't fix them. 


-- 
> Looking for a KDE-related EMail-Alias ? Get one at kdemail.net for FREE! <

--Boundary-00=_ufGo/sgcU36ehRn
Content-Type: application/octet-stream;
  name="dom_docimpl.diff"
Content-Transfer-Encoding: 7bit
Content-Disposition: attachment;
	filename="dom_docimpl.diff"

Index: dom_docimpl.cpp
===================================================================
RCS file: /home/kde/kdelibs/khtml/xml/dom_docimpl.cpp,v
retrieving revision 1.259
diff -u -5 -d -p -r1.259 dom_docimpl.cpp
--- dom_docimpl.cpp	27 Oct 2003 12:45:14 -0000	1.259
+++ dom_docimpl.cpp	28 Oct 2003 15:26:25 -0000
@@ -1098,18 +1098,20 @@ void DocumentImpl::open( bool clearEvent
 
     delete m_tokenizer;
     m_tokenizer = 0;
 
     //{
-        RenderObject* render = m_render;
-        m_render = 0; // indicate destruction mode
-        removeChildren();
+        KHTMLView* view = m_view;
+        detach();
+        delete m_styleSelector;
+        m_styleSelector = 0;
+        m_view = view;
+        attach();
         if (clearEventListeners) {
             QPtrListIterator<RegisteredEventListener> it(m_windowEventListeners);
             for (; it.current();)
                 m_windowEventListeners.removeRef(it.current());
-        m_render = render;
     }
 
     m_tokenizer = createTokenizer();
     m_decoderMibEnum = 0;
     connect(m_tokenizer,SIGNAL(finishedParsing()),this,SIGNAL(finishedParsing()));
@@ -1521,11 +1523,11 @@ bool DocumentImpl::prepareMouseEvent( bo
         bool isInside = m_render->layer()->nodeAtPoint(renderInfo, _x, _y);
         ev->innerNode = renderInfo.innerNode();
 
         if (renderInfo.URLElement()) {
             assert(renderInfo.URLElement()->isElementNode());
-            //qDebug("urlnode: %s  (%d)", getTagName(renderInfo.URLElement()->id()).string().latin1(), renderInfo.URLElement()->id());
+            qDebug("urlnode: %s  (%d)", getTagName(renderInfo.URLElement()->id()).string().latin1(), renderInfo.URLElement()->id());
 
             ElementImpl* e =  static_cast<ElementImpl*>(renderInfo.URLElement());
             DOMString href = khtml::parseURL(e->getAttribute(ATTR_HREF));
             DOMString target = e->getAttribute(ATTR_TARGET);
 

--Boundary-00=_ufGo/sgcU36ehRn--