Reviewed by Darin.

        - http://bugzilla.opendarwin.org/show_bug.cgi?id=9492
        Fix or remove window.debug.

        It was only available in newly created windows, and became undefined after changing the URL
        or reloading. Let's try removing it; hopefully, no-one will notice.

        * bindings/js/kjs_proxy.cpp:
        (WebCore::KJSProxy::initScriptIfNeeded): Don't add window.debug.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@14912 268f45cc-cd09-0410-ab3c-d52691b4dbfc

diff --git a/WebCore/ChangeLog b/WebCore/ChangeLog
index c59bea70dd04900c841b37ed8795cf9c0630750c..ea0930dafb99697d952b152145e0966dc9279760 100644 (file)
--- a/WebCore/ChangeLog
+++ b/WebCore/ChangeLog
@@ -1,3 +1,16 @@
+2006-06-19  Alexey Proskuryakov  <ap@nypop.com>
+
+        Reviewed by Darin.
+
+        - http://bugzilla.opendarwin.org/show_bug.cgi?id=9492
+        Fix or remove window.debug.
+
+        It was only available in newly created windows, and became undefined after changing the URL
+        or reloading. Let's try removing it; hopefully, no-one will notice.
+
+        * bindings/js/kjs_proxy.cpp:
+        (WebCore::KJSProxy::initScriptIfNeeded): Don't add window.debug.
+
 2006-06-19  Alexey Proskuryakov  <ap@nypop.com>
 
         Reviewed by Darin.

diff --git a/WebCore/bindings/js/kjs_proxy.cpp b/WebCore/bindings/js/kjs_proxy.cpp
index 18a45856e72bc161a76b9d1cd54c2ac176f9b013..e13a152db0c498cead9fd502e2448a484dc716b5 100644 (file)
--- a/WebCore/bindings/js/kjs_proxy.cpp
+++ b/WebCore/bindings/js/kjs_proxy.cpp
@@ -126,25 +126,6 @@ ScriptInterpreter* KJSProxy::interpreter()
   return m_script;
 }
 
-// Implementation of the debug() function
-class TestFunctionImp : public DOMObject {
-public:
-  TestFunctionImp(ExecState*);
-  virtual bool implementsCall() const { return true; }
-  virtual JSValue* callAsFunction(ExecState*, JSObject*, const List& args);
-};
-
-TestFunctionImp::TestFunctionImp(ExecState* exec)
-{
-    setPrototype(exec->lexicalInterpreter()->builtinObjectPrototype());
-}
-
-JSValue *TestFunctionImp::callAsFunction(ExecState* exec, JSObject*, const List& args)
-{
-  fprintf(stderr,"--> %s\n", args[0]->toString(exec).ascii());
-  return jsUndefined();
-}
-
 void KJSProxy::initScriptIfNeeded()
 {
   if (m_script)
@@ -156,7 +137,6 @@ void KJSProxy::initScriptIfNeeded()
 
   // Create a KJS interpreter for this frame
   m_script = new ScriptInterpreter(globalObject, m_frame);
-  globalObject->put(m_script->globalExec(), "debug", new TestFunctionImp(m_script->globalExec()), Internal);
 
   String userAgent = m_frame->userAgent();
   if (userAgent.find("Microsoft") >= 0 || userAgent.find("MSIE") >= 0)