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

List:       kde-commits
Subject:    kdelibs/khtml/ecma
From:       Allan Sandfeld Jensen <kde () carewolf ! com>
Date:       2004-12-11 13:19:36
Message-ID: 20041211131936.236F51BB25 () office ! kde ! org
[Download RAW message or body]

CVS commit by carewolf: 

Seems consensus is to try it out. So hide document.all in default mode.


  M +10 -2     kjs_html.cpp   1.279


--- kdelibs/khtml/ecma/kjs_html.cpp  #1.278:1.279
@@ -322,5 +322,9 @@ Value KJS::HTMLDocument::tryGet(ExecStat
       if ( exec->interpreter()->compatMode() == Interpreter::NetscapeCompat )
         return Undefined();
+      else
+      if ( exec->interpreter()->compatMode() == Interpreter::IECompat )
       return getHTMLCollection(exec,doc.all());
+      else // enabled but hidden
+        return getHTMLCollection(exec,doc.all(), true);
     case Clear:
     case Open:
@@ -1911,5 +1915,9 @@ Value KJS::HTMLElement::getValueProperty
     if ( exec->interpreter()->compatMode() == Interpreter::NetscapeCompat )
       return Undefined();
+    else
+    if ( exec->interpreter()->compatMode() == Interpreter::IECompat )
     return getHTMLCollection(exec,element.all());
+    else // Enabled but hidden by default
+      return getHTMLCollection(exec,element.all(), true);
   // ### what about style? or is this used instead for DOM2 stylesheets?
   }


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