safari patch splitup: redundant code

Alexander Kellett lypanov at kde.org
Thu Jan 9 13:35:15 GMT 2003


(patch two)

looks like some old code wasn't removed during merge of isURLAllowed

mvg,
Alex

-- 
"[...] Konqueror open source project. Weighing in at less than
            one tenth the size of another open source renderer"
Apple,  Jan 2003 (http://www.apple.com/safari/)
-------------- next part --------------
Index: html/html_baseimpl.cpp
===================================================================
RCS file: /home/kde/kdelibs/khtml/html/html_baseimpl.cpp,v
retrieving revision 1.173
diff -u -p -B -w -r1.173 html_baseimpl.cpp
--- html/html_baseimpl.cpp	8 Jan 2003 09:03:55 -0000	1.173
+++ html/html_baseimpl.cpp	9 Jan 2003 13:35:58 -0000
@@ -613,16 +613,6 @@ void HTMLIFrameElementImpl::attach()
     assert(parentNode());
 
     KHTMLView* w = getDocument()->view();
-    // avoid endless recursion
-    KURL u;
-    if (!url.isEmpty()) u = getDocument()->completeURL( url.string() );
-    bool selfreference = false;
-    for (KHTMLPart* part = w->part(); part; part = part->parentPart())
-        if (part->url() == u) {
-            selfreference = true;
-            break;
-        }
-
     KHTMLPart *part = w->part();
 
     RenderStyle* _style = getDocument()->styleSelector()->styleForElement(this);


More information about the kfm-devel mailing list