SVN commit 852517 by orlovich:

Make sourceIndex work --- neither assert-fail in release builds, 
nor always return 0 on non-empty documents with debug builds,
nor loop infinitely on empty documents.


 M  +4 -4      kjs_dom.cpp  


--- branches/KDE/4.1/kdelibs/khtml/ecma/kjs_dom.cpp #852516:852517
@@ -335,11 +335,11 @@
     if (doc->isHTMLDocument()) {
       HTMLCollectionImpl all(doc, HTMLCollectionImpl::DOC_ALL);
       unsigned long i = 0;
-      DOM::NodeImpl* n = all.firstItem();
-      for ( ; !n && n != impl(); n = all.nextItem() )
+      for (DOM::NodeImpl* n = all.firstItem(); n; n = all.nextItem() ) {
+        if (n == impl())
+            return jsNumber(i);
         ++i;
-      Q_ASSERT( n ); // node not in document.all !?
-      return jsNumber(i);
+      }
     }
     return jsUndefined();
   }