/hg/icedtea-web: Fix JSObject.equals.

[dlila at icedtea.classpath.org]

changeset 43d21942eb2d in /hg/icedtea-web
details: http://icedtea.classpath.org/hg/icedtea-web?cmd=changeset;node=43d21942eb2d
author: Denis Lila <dlila at redhat.com>
date: Thu Mar 31 17:20:17 2011 -0400

	Fix JSObject.equals.


diffstat:

 ChangeLog                                               |   7 +++++++
 plugin/icedteanp/java/netscape/javascript/JSObject.java |  16 +---------------
 2 files changed, 8 insertions(+), 15 deletions(-)

diffs (61 lines):

diff -r fc3bcf836c77 -r 43d21942eb2d ChangeLog
--- a/ChangeLog	Thu Mar 31 17:07:10 2011 -0400
+++ b/ChangeLog	Thu Mar 31 17:20:17 2011 -0400
@@ -1,3 +1,10 @@
+2011-03-31  Denis Lila  <dlila at redhat.com>
+
+	* plugin/icedteanp/java/netscape/javascript/JSObject.java:
+	Fix comments, remove unused imports.
+	(equals): Remove. It was breaking the reflexivity in the
+	equals contract.
+
 2011-03-31  Denis Lila  <dlila at redhat.com>
 
 	* plugin/icedteanp/java/sun/applet/PluginObjectStore.java:
diff -r fc3bcf836c77 -r 43d21942eb2d plugin/icedteanp/java/netscape/javascript/JSObject.java
--- a/plugin/icedteanp/java/netscape/javascript/JSObject.java	Thu Mar 31 17:07:10 2011 -0400
+++ b/plugin/icedteanp/java/netscape/javascript/JSObject.java	Thu Mar 31 17:20:17 2011 -0400
@@ -37,7 +37,7 @@
  *
  * ***** END LICENSE BLOCK ***** */
 
-/* more doc todo:
+/* more doc TODO:
  *  threads
  *  gc
  *
@@ -47,10 +47,8 @@
 package netscape.javascript;
 
 import java.applet.Applet;
-import java.security.AccessControlContext;
 import java.security.AccessControlException;
 import java.security.AccessController;
-import java.security.BasicPermission;
 
 import sun.applet.PluginAppletViewer;
 import sun.applet.PluginDebug;
@@ -193,8 +191,6 @@
         PluginAppletViewer.setSlot(internal, index, value);
     }
 
-    // TODO: toString, finalize.
-
     /**
      * Removes a named member of a JavaScript object.
      */
@@ -267,14 +263,4 @@
         PluginDebug.debug("JSObject.finalize ");
         PluginAppletViewer.JavaScriptFinalize(internal);
     }
-
-    /**
-     * Override java.lang.Object.equals() because identity is not preserved
-     * with instances of JSObject.
-     */
-    public boolean equals(Object obj) {
-        PluginDebug.debug("JSObject.equals " + obj);
-
-        return false;
-    }
 }